On 08/18/2017 01:23 AM, Bart Van Assche wrote: > Signed-off-by: Bart Van Assche <bart.vanassche@xxxxxxx> > Cc: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx> > Cc: Roger Pau Monn303251 <roger.pau@xxxxxxxxxx> > Cc: xen-devel@xxxxxxxxxxxxxxxxxxxx > --- > drivers/block/xen-blkback/blkback.c | 1 + > drivers/block/xen-blkback/xenbus.c | 3 ++- > 2 files changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/block/xen-blkback/blkback.c b/drivers/block/xen-blkback/blkback.c > index 68157a84bf4d..5f3a813e7ae0 100644 > --- a/drivers/block/xen-blkback/blkback.c > +++ b/drivers/block/xen-blkback/blkback.c > @@ -1251,6 +1251,7 @@ static int dispatch_rw_block_io(struct xen_blkif_ring *ring, > break; > case BLKIF_OP_WRITE_BARRIER: > drain = true; > + /* fall through */ > case BLKIF_OP_FLUSH_DISKCACHE: > ring->st_f_req++; > operation = REQ_OP_WRITE; > diff --git a/drivers/block/xen-blkback/xenbus.c b/drivers/block/xen-blkback/xenbus.c > index 792da683e70d..88eaea6475d7 100644 > --- a/drivers/block/xen-blkback/xenbus.c > +++ b/drivers/block/xen-blkback/xenbus.c > @@ -810,7 +810,8 @@ static void frontend_changed(struct xenbus_device *dev, > xenbus_switch_state(dev, XenbusStateClosed); > if (xenbus_dev_is_online(dev)) > break; > - /* fall through if not online */ > + /* fall through */ > + /* if not online */ > case XenbusStateUnknown: > /* implies xen_blkif_disconnect() via xen_blkbk_remove() */ > device_unregister(&dev->dev); > Oh well, gcc again. Reviewed-by: Hannes Reinecke <hare@xxxxxxxx> Cheers, Hannes -- Dr. Hannes Reinecke Teamlead Storage & Networking hare@xxxxxxx +49 911 74053 688 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: F. Imendörffer, J. Smithard, J. Guild, D. Upmanyu, G. Norton HRB 21284 (AG Nürnberg)
