diff options
| author | Nathan Chancellor | 2018-12-20 17:23:44 +0100 |
|---|---|---|
| committer | Jens Axboe | 2018-12-20 17:51:31 +0100 |
| commit | 5816a0932b4fd74257b8cc5785bc8067186a8723 (patch) | |
| tree | 8264ce26871ddbe2b14b548e13b09c523c9ca577 /drivers/block/drbd/drbd_state.c | |
| parent | drbd: Avoid Clang warning about pointless switch statment (diff) | |
| download | kernel-qcow2-linux-5816a0932b4fd74257b8cc5785bc8067186a8723.tar.gz kernel-qcow2-linux-5816a0932b4fd74257b8cc5785bc8067186a8723.tar.xz kernel-qcow2-linux-5816a0932b4fd74257b8cc5785bc8067186a8723.zip | |
drbd: Change drbd_request_detach_interruptible's return type to int
Clang warns when an implicit conversion is done between enumerated
types:
drivers/block/drbd/drbd_state.c:708:8: warning: implicit conversion from
enumeration type 'enum drbd_ret_code' to different enumeration type
'enum drbd_state_rv' [-Wenum-conversion]
rv = ERR_INTR;
~ ^~~~~~~~
drbd_request_detach_interruptible's only call site is in the return
statement of adm_detach, which returns an int. Change the return type of
drbd_request_detach_interruptible to match, silencing Clang's warning.
Reported-by: Nick Desaulniers <ndesaulniers@google.com>
Reviewed-by: Nick Desaulniers <ndesaulniers@google.com>
Signed-off-by: Nathan Chancellor <natechancellor@gmail.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'drivers/block/drbd/drbd_state.c')
| -rw-r--r-- | drivers/block/drbd/drbd_state.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/block/drbd/drbd_state.c b/drivers/block/drbd/drbd_state.c index 883155657273..2b4c0db5d867 100644 --- a/drivers/block/drbd/drbd_state.c +++ b/drivers/block/drbd/drbd_state.c @@ -688,11 +688,9 @@ request_detach(struct drbd_device *device) CS_VERBOSE | CS_ORDERED | CS_INHIBIT_MD_IO); } -enum drbd_state_rv -drbd_request_detach_interruptible(struct drbd_device *device) +int drbd_request_detach_interruptible(struct drbd_device *device) { - enum drbd_state_rv rv; - int ret; + int ret, rv; drbd_suspend_io(device); /* so no-one is stuck in drbd_al_begin_io */ wait_event_interruptible(device->state_wait, |