On Thu, 2016-09-08 at 17:12 -0400, Josef Bacik wrote: > In preparation for some future changes, change a few of the state bools over to > normal bits to set/clear properly. [] > diff --git a/drivers/block/nbd.c b/drivers/block/nbd.c [] > @@ -41,8 +41,12 @@ > > #include <linux/nbd.h> > > +#define NBD_TIMEDOUT 0 > +#define NBD_DISCONNECT_REQUESTED 1 > + > struct nbd_device { > u32 flags; > + unsigned long runtime_flags; Better to use DECLARE_BITMAP > @@ -626,7 +627,7 @@ static int __nbd_ioctl(struct block_device *bdev, struct nbd_device *nbd, > return -EINVAL; > } > > - nbd->disconnect = true; > + set_bit(NBD_DISCONNECT_REQUESTED, &nbd->runtime_flags); And remove the & from runtime_flags here -- To unsubscribe from this list: send the line "unsubscribe linux-block" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html