[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [RFC PATCH v3 65/78] hw/nvme: add fallthrough pseudo-keyword
From: |
Klaus Jensen |
Subject: |
Re: [RFC PATCH v3 65/78] hw/nvme: add fallthrough pseudo-keyword |
Date: |
Wed, 15 Nov 2023 10:30:13 +0100 |
On Oct 13 11:46, Emmanouil Pitsidianakis wrote:
> In preparation of raising -Wimplicit-fallthrough to 5, replace all
> fall-through comments with the fallthrough attribute pseudo-keyword.
>
> Signed-off-by: Emmanouil Pitsidianakis <manos.pitsidianakis@linaro.org>
> ---
> hw/nvme/ctrl.c | 24 ++++++++++++------------
> hw/nvme/dif.c | 4 ++--
> 2 files changed, 14 insertions(+), 14 deletions(-)
>
> diff --git a/hw/nvme/ctrl.c b/hw/nvme/ctrl.c
> index f026245d1e..acb2012fb9 100644
> --- a/hw/nvme/ctrl.c
> +++ b/hw/nvme/ctrl.c
> @@ -1918,7 +1918,7 @@ static uint16_t nvme_zrm_finish(NvmeNamespace *ns,
> NvmeZone *zone)
> case NVME_ZONE_STATE_IMPLICITLY_OPEN:
> case NVME_ZONE_STATE_EXPLICITLY_OPEN:
> nvme_aor_dec_open(ns);
> - /* fallthrough */
> + fallthrough;
> case NVME_ZONE_STATE_CLOSED:
> nvme_aor_dec_active(ns);
>
> @@ -1929,7 +1929,7 @@ static uint16_t nvme_zrm_finish(NvmeNamespace *ns,
> NvmeZone *zone)
> }
> }
>
> - /* fallthrough */
> + fallthrough;
> case NVME_ZONE_STATE_EMPTY:
> nvme_assign_zone_state(ns, zone, NVME_ZONE_STATE_FULL);
> return NVME_SUCCESS;
> @@ -1946,7 +1946,7 @@ static uint16_t nvme_zrm_close(NvmeNamespace *ns,
> NvmeZone *zone)
> case NVME_ZONE_STATE_IMPLICITLY_OPEN:
> nvme_aor_dec_open(ns);
> nvme_assign_zone_state(ns, zone, NVME_ZONE_STATE_CLOSED);
> - /* fall through */
> + fallthrough;
> case NVME_ZONE_STATE_CLOSED:
> return NVME_SUCCESS;
>
> @@ -1961,7 +1961,7 @@ static uint16_t nvme_zrm_reset(NvmeNamespace *ns,
> NvmeZone *zone)
> case NVME_ZONE_STATE_EXPLICITLY_OPEN:
> case NVME_ZONE_STATE_IMPLICITLY_OPEN:
> nvme_aor_dec_open(ns);
> - /* fallthrough */
> + fallthrough;
> case NVME_ZONE_STATE_CLOSED:
> nvme_aor_dec_active(ns);
>
> @@ -1971,12 +1971,12 @@ static uint16_t nvme_zrm_reset(NvmeNamespace *ns,
> NvmeZone *zone)
> }
> }
>
> - /* fallthrough */
> + fallthrough;
> case NVME_ZONE_STATE_FULL:
> zone->w_ptr = zone->d.zslba;
> zone->d.wp = zone->w_ptr;
> nvme_assign_zone_state(ns, zone, NVME_ZONE_STATE_EMPTY);
> - /* fallthrough */
> + fallthrough;
> case NVME_ZONE_STATE_EMPTY:
> return NVME_SUCCESS;
>
> @@ -2017,7 +2017,7 @@ static uint16_t nvme_zrm_open_flags(NvmeCtrl *n,
> NvmeNamespace *ns,
> case NVME_ZONE_STATE_EMPTY:
> act = 1;
>
> - /* fallthrough */
> + fallthrough;
>
> case NVME_ZONE_STATE_CLOSED:
> if (n->params.auto_transition_zones) {
> @@ -2040,7 +2040,7 @@ static uint16_t nvme_zrm_open_flags(NvmeCtrl *n,
> NvmeNamespace *ns,
> return NVME_SUCCESS;
> }
>
> - /* fallthrough */
> + fallthrough;
>
> case NVME_ZONE_STATE_IMPLICITLY_OPEN:
> if (flags & NVME_ZRM_AUTO) {
> @@ -2049,7 +2049,7 @@ static uint16_t nvme_zrm_open_flags(NvmeCtrl *n,
> NvmeNamespace *ns,
>
> nvme_assign_zone_state(ns, zone, NVME_ZONE_STATE_EXPLICITLY_OPEN);
>
> - /* fallthrough */
> + fallthrough;
>
> case NVME_ZONE_STATE_EXPLICITLY_OPEN:
> if (flags & NVME_ZRM_ZRWA) {
> @@ -3582,7 +3582,7 @@ static uint16_t nvme_do_write(NvmeCtrl *n, NvmeRequest
> *req, bool append,
> return NVME_INVALID_PROT_INFO | NVME_DNR;
> }
>
> - /* fallthrough */
> + fallthrough;
>
> case NVME_ID_NS_DPS_TYPE_2:
> if (piremap) {
> @@ -3737,7 +3737,7 @@ static uint16_t nvme_offline_zone(NvmeNamespace *ns,
> NvmeZone *zone,
> switch (state) {
> case NVME_ZONE_STATE_READ_ONLY:
> nvme_assign_zone_state(ns, zone, NVME_ZONE_STATE_OFFLINE);
> - /* fall through */
> + fallthrough;
> case NVME_ZONE_STATE_OFFLINE:
> return NVME_SUCCESS;
> default:
> @@ -4914,7 +4914,7 @@ static uint16_t nvme_cmd_effects(NvmeCtrl *n, uint8_t
> csi, uint32_t buf_len,
> switch (NVME_CC_CSS(ldl_le_p(&n->bar.cc))) {
> case NVME_CC_CSS_NVM:
> src_iocs = nvme_cse_iocs_nvm;
> - /* fall through */
> + fallthrough;
> case NVME_CC_CSS_ADMIN_ONLY:
> break;
> case NVME_CC_CSS_CSI:
> diff --git a/hw/nvme/dif.c b/hw/nvme/dif.c
> index 01b19c3373..00dd96bdb3 100644
> --- a/hw/nvme/dif.c
> +++ b/hw/nvme/dif.c
> @@ -161,7 +161,7 @@ static uint16_t nvme_dif_prchk_crc16(NvmeNamespace *ns,
> NvmeDifTuple *dif,
> break;
> }
>
> - /* fallthrough */
> + fallthrough;
> case NVME_ID_NS_DPS_TYPE_1:
> case NVME_ID_NS_DPS_TYPE_2:
> if (be16_to_cpu(dif->g16.apptag) != 0xffff) {
> @@ -229,7 +229,7 @@ static uint16_t nvme_dif_prchk_crc64(NvmeNamespace *ns,
> NvmeDifTuple *dif,
> break;
> }
>
> - /* fallthrough */
> + fallthrough;
> case NVME_ID_NS_DPS_TYPE_1:
> case NVME_ID_NS_DPS_TYPE_2:
> if (be16_to_cpu(dif->g64.apptag) != 0xffff) {
> --
> 2.39.2
>
>
Reviewed-by: Klaus Jensen <k.jensen@samsung.com>
signature.asc
Description: PGP signature
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [RFC PATCH v3 65/78] hw/nvme: add fallthrough pseudo-keyword,
Klaus Jensen <=