Messages in this thread | | | From | Dexuan Cui <> | Subject | RE: [PATCH 3/5] Drivers: hv: kvp: Fix the recent regression caused by incorrect clean-up | Date | Wed, 17 Oct 2018 17:56:15 +0000 |
| |
> From: devel <driverdev-devel-bounces@linuxdriverproject.org> On Behalf Of > KY Srinivasan > Sent: Tuesday, October 16, 2018 23:02 > > > --- a/drivers/hv/hv_kvp.c > > > +++ b/drivers/hv/hv_kvp.c > > > @@ -353,6 +353,9 @@ static void process_ib_ipinfo(void *in_msg, void > > *out_msg, int op) > > > > > > out->body.kvp_ip_val.dhcp_enabled = in- > > >kvp_ip_val.dhcp_enabled; > > > > > > + __attribute__ ((fallthrough)); > > > > The comment should be sufficient for this, right? I haven't seen many > > uses of this attribute before, how common is it? > > Yes; a common should be sufficient.
You all are right. Right now, I realized the gcc warning can also be suppressed by a simple line of comment:
/* fallthrough */
It looks gcc treats this comment specially.
If I add something more in the comment, like /* add fallthrough */ , the warning can not be suppressed. :-)
I'll do a new version for KY.
Thank you all!
-- Dexuan
| |