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