patches for DPDK stable branches
 help / color / mirror / Atom feed
From: Christian Ehrhardt <christian.ehrhardt@canonical.com>
To: Ferruh Yigit <ferruh.yigit@intel.com>
Cc: stable@dpdk.org
Subject: Re: [PATCH 19.11 v2] igb_uio: fix build for switch fall through
Date: Fri, 17 Dec 2021 08:07:15 +0100
Message-ID: <CAATJJ0J-5yzrb_FLwcvtm5y3uuf7VpG4WnxS_=ncT_eQuPfG=Q@mail.gmail.com> (raw)
In-Reply-To: <20211216123611.2125507-1-ferruh.yigit@intel.com>

On Thu, Dec 16, 2021 at 1:36 PM Ferruh Yigit <ferruh.yigit@intel.com> wrote:
>
> Linux is using '-Wimplicit-fallthrough=5' compiler option, which doesn't
> take any fall through comments into account but only uses compiler
> 'fallthrough' attribute to document fall through action is intended.
>
> "falls through" comment was used in the code which is causing a build
> error now, this patch converts comment to the 'fallthrough' macro
> defined in the Linux.
>
> To cover the case where Linux version doesn't have the macro, defined it
> in the compatibility header too.

Thank you Ferruh, I've applied it to the WIP branch for test builds.
If nothing regresses I'll keep it and it will go into -rc2 later.

> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> ---
> Cc: Christian Ehrhardt <christian.ehrhardt@canonical.com>
>
> v2:
> * Add both dummy and attribute fallthrough macro definition to
>   compatibility header.
> ---
>  kernel/linux/igb_uio/compat.h  | 14 ++++++++++++++
>  kernel/linux/igb_uio/igb_uio.c |  6 +++---
>  2 files changed, 17 insertions(+), 3 deletions(-)
>
> diff --git a/kernel/linux/igb_uio/compat.h b/kernel/linux/igb_uio/compat.h
> index 8dbb896ae118..850b359f3643 100644
> --- a/kernel/linux/igb_uio/compat.h
> +++ b/kernel/linux/igb_uio/compat.h
> @@ -152,3 +152,17 @@ static inline bool igbuio_kernel_is_locked_down(void)
>         return false;
>  #endif
>  }
> +
> +#ifndef fallthrough
> +
> +#ifndef __has_attribute
> +#define __has_attribute(x) 0
> +#endif
> +
> +#if __has_attribute(__fallthrough__)
> +#define fallthrough    __attribute__((__fallthrough__))
> +#else
> +#define fallthrough    do {} while (0)
> +#endif
> +
> +#endif
> diff --git a/kernel/linux/igb_uio/igb_uio.c b/kernel/linux/igb_uio/igb_uio.c
> index 039f5a5f6354..57d0c58ab1b8 100644
> --- a/kernel/linux/igb_uio/igb_uio.c
> +++ b/kernel/linux/igb_uio/igb_uio.c
> @@ -236,7 +236,7 @@ igbuio_pci_enable_interrupts(struct rte_uio_pci_dev *udev)
>                 }
>  #endif
>
> -       /* falls through - to MSI */
> +       fallthrough;
>         case RTE_INTR_MODE_MSI:
>  #ifndef HAVE_ALLOC_IRQ_VECTORS
>                 if (pci_enable_msi(udev->pdev) == 0) {
> @@ -255,7 +255,7 @@ igbuio_pci_enable_interrupts(struct rte_uio_pci_dev *udev)
>                         break;
>                 }
>  #endif
> -       /* falls through - to INTX */
> +       fallthrough;
>         case RTE_INTR_MODE_LEGACY:
>                 if (pci_intx_mask_supported(udev->pdev)) {
>                         dev_dbg(&udev->pdev->dev, "using INTX");
> @@ -265,7 +265,7 @@ igbuio_pci_enable_interrupts(struct rte_uio_pci_dev *udev)
>                         break;
>                 }
>                 dev_notice(&udev->pdev->dev, "PCI INTX mask not supported\n");
> -       /* falls through - to no IRQ */
> +       fallthrough;
>         case RTE_INTR_MODE_NONE:
>                 udev->mode = RTE_INTR_MODE_NONE;
>                 udev->info.irq = UIO_IRQ_NONE;
> --
> 2.33.1
>


-- 
Christian Ehrhardt
Staff Engineer, Ubuntu Server
Canonical Ltd

      reply	other threads:[~2021-12-17  7:07 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-12-15 19:00 [PATCH 19.11] " Ferruh Yigit
2021-12-16 12:36 ` [PATCH 19.11 v2] " Ferruh Yigit
2021-12-17  7:07   ` Christian Ehrhardt [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAATJJ0J-5yzrb_FLwcvtm5y3uuf7VpG4WnxS_=ncT_eQuPfG=Q@mail.gmail.com' \
    --to=christian.ehrhardt@canonical.com \
    --cc=ferruh.yigit@intel.com \
    --cc=stable@dpdk.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

patches for DPDK stable branches

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://inbox.dpdk.org/stable/0 stable/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 stable stable/ http://inbox.dpdk.org/stable \
		stable@dpdk.org
	public-inbox-index stable

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.stable


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git