On Sat, 29 May 2021 11:22:14 +0800 Huazhong Tan wrote:
> diff --git a/Documentation/networking/ethtool-netlink.rst b/Documentation/networking/ethtool-netlink.rst
> index 25131df..8e8c6b3 100644
> --- a/Documentation/networking/ethtool-netlink.rst
> +++ b/Documentation/networking/ethtool-netlink.rst
> @@ -937,6 +937,8 @@ Kernel response contents:
> ``ETHTOOL_A_COALESCE_TX_USECS_HIGH`` u32 delay (us), high Tx
> ``ETHTOOL_A_COALESCE_TX_MAX_FRAMES_HIGH`` u32 max packets, high Tx
> ``ETHTOOL_A_COALESCE_RATE_SAMPLE_INTERVAL`` u32 rate sampling interval
> + ``ETHTOOL_A_COALESCE_USE_CQE_TX`` bool timer reset in CQE, Tx
> + ``ETHTOOL_A_COALESCE_USE_CQE_RX`` bool timer reset in CQE, Rx
> =========================================== ====== =======================
>
> Attributes are only included in reply if their value is not zero or the
> @@ -975,6 +977,8 @@ Request contents:
> ``ETHTOOL_A_COALESCE_TX_USECS_HIGH`` u32 delay (us), high Tx
> ``ETHTOOL_A_COALESCE_TX_MAX_FRAMES_HIGH`` u32 max packets, high Tx
> ``ETHTOOL_A_COALESCE_RATE_SAMPLE_INTERVAL`` u32 rate sampling interval
> + ``ETHTOOL_A_COALESCE_USE_CQE_TX`` bool timer reset in CQE, Tx
> + ``ETHTOOL_A_COALESCE_USE_CQE_RX`` bool timer reset in CQE, Rx
> =========================================== ====== =======================
>
> Request is rejected if it attributes declared as unsupported by driver (i.e.
Did you provide the theory of operation for CQE vs EQE mode somewhere,
as I requested?
> + [ETHTOOL_A_COALESCE_USE_CQE_MODE_TX] = { .type = NLA_U8 },
> + [ETHTOOL_A_COALESCE_USE_CQE_MODE_RX] = { .type = NLA_U8 },
Why not NLA_POLICY_MAX(NLA_U8, 1) ?
Any chance you could split the patch into adding the new parameter
to the callback and adding new attributes?
