Re: [PATCH net-next v2] net: dlink: add support for reporting stats via `ethtool -S`

From: Moon Yeounsu
Date: Sun Nov 10 2024 - 09:26:07 EST


First of all, I would like to extend my apologies.
In the previous patch, the `get_ethtool_rmon_stats()` function should
have called the `get_stats()` function to update the stats
information,
but I missed this. I will include this part in the next patch.

On Fri, Nov 8, 2024 at 1:09 PM Jakub Kicinski <kuba@xxxxxxxxxx> wrote:
> Do these macro wrappers really buy you anything?
> They make the code a lot harder to follow :(
I fully understand that these macros can be difficult to comprehend at
first glance.
However, I wanted to avoid code duplication with this structure.
For now, I will write the code in a way that minimizes the use of
macros as much as possible.

> nit: multiple empty lines, checkpatch --strict should catch this
In the next patch, I will use the `checkpatch --strict` option to fix
as many warnings and checks as possible before submitting the patch.
I also confirmed from patchwork[1] that there were additional warning messages.
I will pay attention to and correct these warnings in the next patch.
Thank you for pointing that out.

> We've been getting conversion patches replacing such code with
> ethtool_puts() lately, let's use it from the start.
I will do so.

[1]: https://patchwork.kernel.org/project/netdevbpf/patch/20241107151929.37147-5-yyyynoom@xxxxxxxxx/