[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <fc8f0ae6-de4f-0468-201f-298abf21e91a@nvidia.com>
Date: Wed, 25 Jan 2023 10:28:47 +0200
From: Gal Pressman <gal@...dia.com>
To: Jakub Kicinski <kuba@...nel.org>
Cc: Bar Shapira <bar.shapira.work@...il.com>,
Rahul Rameshbabu <rrameshbabu@...dia.com>,
Jacob Keller <jacob.e.keller@...el.com>,
Saeed Mahameed <saeed@...nel.org>,
"David S. Miller" <davem@...emloft.net>,
Paolo Abeni <pabeni@...hat.com>,
Eric Dumazet <edumazet@...gle.com>,
Saeed Mahameed <saeedm@...dia.com>, netdev@...r.kernel.org,
Tariq Toukan <tariqt@...dia.com>,
Vincent Cheng <vincent.cheng.xh@...esas.com>,
Richard Cochran <richardcochran@...il.com>
Subject: Re: [net-next 03/15] net/mlx5: Add adjphase function to support
hardware-only offset control
On 25/01/2023 2:48, Jakub Kicinski wrote:
> On Tue, 24 Jan 2023 11:15:31 -0800 Richard Cochran wrote:
>> On Tue, Jan 24, 2023 at 12:33:05PM +0200, Bar Shapira wrote:
>>> I guess this expectation should be part of the documentation too, right? Are
>>> there more expectations when calling adjphase?
>>
>> I'll gladly ack improvements to the documentation. I myself won't
>> spend time on that, because it will only get ignored, even when it is
>> super clear. Like ptp_clock_register(), for example.
>
> IMHO it is a responsibility of maintainers to try to teach, even if not
> everyone turns out to be a diligent listener/reader. I've looked for
> information about this callback at least twice in the last 6 months.
>
> nVidia folks, could you please improve the doc, in that case?
> I think you also owe me the docs for your "debug" configuration
> of hairpin queues in debugfs.
I have patches converting the hairpin debugfs to devlink params, the
write debugfs files will be reverted.
Powered by blists - more mailing lists