[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <1e1da86d-d544-425d-b1f6-fcc88bc49eed@linux.dev>
Date: Wed, 9 Jul 2025 14:20:17 +0100
From: Vadim Fedorenko <vadim.fedorenko@...ux.dev>
To: Simon Horman <horms@...nel.org>, Gur Stavi <gur.stavi@...wei.com>
Cc: andrew+netdev@...n.ch, christophe.jaillet@...adoo.fr, corbet@....net,
davem@...emloft.net, edumazet@...gle.com, gongfan1@...wei.com,
guoxin09@...wei.com, helgaas@...nel.org, jdamato@...tly.com,
kuba@...nel.org, lee@...ger.us, linux-doc@...r.kernel.org,
linux-kernel@...r.kernel.org, luosifu@...wei.com, meny.yossefi@...wei.com,
mpe@...erman.id.au, netdev@...r.kernel.org, pabeni@...hat.com,
przemyslaw.kitszel@...el.com, shenchenyang1@...ilicon.com,
shijing34@...wei.com, sumang@...vell.com, wulike1@...wei.com,
zhoushuai28@...wei.com, zhuyikai1@...artners.com
Subject: Re: [PATCH net-next v06 5/8] hinic3: TX & RX Queue coalesce
interfaces
On 09/07/2025 12:56, Simon Horman wrote:
> On Wed, Jul 09, 2025 at 11:26:20AM +0300, Gur Stavi wrote:
>>> On 27/06/2025 07:12, Fan Gong wrote:
>>>> Add TX RX queue coalesce interfaces initialization.
>>>> It configures the parameters of tx & tx msix coalesce.
>>>>
>>>> Co-developed-by: Xin Guo <guoxin09@...wei.com>
>>>> Signed-off-by: Xin Guo <guoxin09@...wei.com>
>>>> Co-developed-by: Zhu Yikai <zhuyikai1@...artners.com>
>>>> Signed-off-by: Zhu Yikai <zhuyikai1@...artners.com>
>>>> Signed-off-by: Fan Gong <gongfan1@...wei.com>
>>>> ---
>>>> .../net/ethernet/huawei/hinic3/hinic3_main.c | 61 +++++++++++++++++--
>>>> .../ethernet/huawei/hinic3/hinic3_nic_dev.h | 10 +++
>>>> 2 files changed, 66 insertions(+), 5 deletions(-)
>>>>
>>>
>>> Reviewed-by: Vadim Fedorenko <vadim.fedorenko@...ux.dev>
>>
>> Procedural question about submissions:
>> Are we allowed (or expected) to copy the "Reviewed-by" above to future
>> submissions as long as we do not modify this specific patch?
>
> Vadim is free to offer his own guidance, as it's his tag.
>
> But in principle tags should be carried forward into future submissions
> unless there is a material change. In this case I think that would
> mean a material change this patch.
>
> But if in doubt, please ask.
> Thanks for doing so.
As Simon said, in general if there are no changes in the patch you can
carry forward Rb tag.
Powered by blists - more mailing lists