Re: [RFC PATCH v2 net-next 1/2] net: dcb: add new pcp selector to app object
From: Petr Machata <petrm@nvidia.com>
Date: 2022-09-19 10:03:50
Daniel Machon [off-list ref] writes:
quoted hunk ↗ jump to hunk
diff --git a/include/uapi/linux/dcbnl.h b/include/uapi/linux/dcbnl.h index a791a94013a6..8eab16e5bc13 100644 --- a/include/uapi/linux/dcbnl.h +++ b/include/uapi/linux/dcbnl.h@@ -217,6 +217,7 @@ struct cee_pfc { #define IEEE_8021QAZ_APP_SEL_DGRAM 3 #define IEEE_8021QAZ_APP_SEL_ANY 4 #define IEEE_8021QAZ_APP_SEL_DSCP 5 +#define IEEE_8021QAZ_APP_SEL_PCP 255 /* This structure contains the IEEE 802.1Qaz APP managed object. This * object is also used for the CEE std as well.
One more thought: please verify how this behaves with openlldpad. It's a fairly major user of this API. I guess it is OK if it refuses to run or bails out in face of the PCP APP entries. On its own it will never introduce them, so this clear and noisy diagnostic when a user messes with the system through a different channels is OK IMHO. But it shouldn't silently reinterpret the 255 to mean something else.