Re: [PATCH 027/141] drbd: Fix fall-through warnings for Clang
On 4/20/21 16:23, Jens Axboe wrote: > Applied, thanks. Awesome. :) Thanks -- Gustavo
Re: [PATCH 027/141] drbd: Fix fall-through warnings for Clang
On 4/20/21 2:25 PM, Gustavo A. R. Silva wrote: > Hi all, > > Friendly ping: who can take this, please? Applied, thanks. -- Jens Axboe
Re: [PATCH 027/141] drbd: Fix fall-through warnings for Clang
Hi all, Friendly ping: who can take this, please? Thanks -- Gustavo On 11/20/20 12:28, Gustavo A. R. Silva wrote: > In preparation to enable -Wimplicit-fallthrough for Clang, fix a couple > of warnings by explicitly adding a break statement instead of just > letting the code fall through to the next, and by adding a fallthrough > pseudo-keyword in places whre the code is intended to fall through. > > Link: https://github.com/KSPP/linux/issues/115 > Signed-off-by: Gustavo A. R. Silva > --- > drivers/block/drbd/drbd_receiver.c | 1 + > drivers/block/drbd/drbd_req.c | 1 + > 2 files changed, 2 insertions(+) > > diff --git a/drivers/block/drbd/drbd_receiver.c > b/drivers/block/drbd/drbd_receiver.c > index dc333dbe5232..c19bb74ac935 100644 > --- a/drivers/block/drbd/drbd_receiver.c > +++ b/drivers/block/drbd/drbd_receiver.c > @@ -5863,6 +5863,7 @@ static int got_NegRSDReply(struct drbd_connection > *connection, struct packet_inf > switch (pi->cmd) { > case P_NEG_RS_DREPLY: > drbd_rs_failed_io(device, sector, size); > + break; > case P_RS_CANCEL: > break; > default: > diff --git a/drivers/block/drbd/drbd_req.c b/drivers/block/drbd/drbd_req.c > index 330f851cb8f0..9f212a923a3c 100644 > --- a/drivers/block/drbd/drbd_req.c > +++ b/drivers/block/drbd/drbd_req.c > @@ -750,6 +750,7 @@ int __req_mod(struct drbd_request *req, enum > drbd_req_event what, > > case WRITE_ACKED_BY_PEER_AND_SIS: > req->rq_state |= RQ_NET_SIS; > + fallthrough; > case WRITE_ACKED_BY_PEER: > /* Normal operation protocol C: successfully written on peer. >* During resync, even in protocol != C, >
[PATCH 027/141] drbd: Fix fall-through warnings for Clang
In preparation to enable -Wimplicit-fallthrough for Clang, fix a couple of warnings by explicitly adding a break statement instead of just letting the code fall through to the next, and by adding a fallthrough pseudo-keyword in places whre the code is intended to fall through. Link: https://github.com/KSPP/linux/issues/115 Signed-off-by: Gustavo A. R. Silva --- drivers/block/drbd/drbd_receiver.c | 1 + drivers/block/drbd/drbd_req.c | 1 + 2 files changed, 2 insertions(+) diff --git a/drivers/block/drbd/drbd_receiver.c b/drivers/block/drbd/drbd_receiver.c index dc333dbe5232..c19bb74ac935 100644 --- a/drivers/block/drbd/drbd_receiver.c +++ b/drivers/block/drbd/drbd_receiver.c @@ -5863,6 +5863,7 @@ static int got_NegRSDReply(struct drbd_connection *connection, struct packet_inf switch (pi->cmd) { case P_NEG_RS_DREPLY: drbd_rs_failed_io(device, sector, size); + break; case P_RS_CANCEL: break; default: diff --git a/drivers/block/drbd/drbd_req.c b/drivers/block/drbd/drbd_req.c index 330f851cb8f0..9f212a923a3c 100644 --- a/drivers/block/drbd/drbd_req.c +++ b/drivers/block/drbd/drbd_req.c @@ -750,6 +750,7 @@ int __req_mod(struct drbd_request *req, enum drbd_req_event what, case WRITE_ACKED_BY_PEER_AND_SIS: req->rq_state |= RQ_NET_SIS; + fallthrough; case WRITE_ACKED_BY_PEER: /* Normal operation protocol C: successfully written on peer. * During resync, even in protocol != C, -- 2.27.0