On Tue, 13 Aug 2024 21:13:05 +0000 Mina Almasry wrote:
> +             if (NL_REQ_ATTR_CHECK(info->extack, attr, tb,
> +                                   NETDEV_A_QUEUE_ID)) {
> +                     err = -EINVAL;
> +                     goto err_unlock;
> +             }
> +
> +             if (NL_REQ_ATTR_CHECK(info->extack, attr, tb,
> +                                   NETDEV_A_QUEUE_TYPE)) {
> +                     err = -EINVAL;
> +                     goto err_unlock;
> +             }
> +
> +             if (nla_get_u32(tb[NETDEV_A_QUEUE_TYPE]) !=
> +                 NETDEV_QUEUE_TYPE_RX) {
> +                     err = -EINVAL;
> +                     goto err_unlock;
> +             }

nit: you can || these three into as single if statement

Reply via email to