Thread (28 messages) 28 messages, 4 authors, 2022-11-08

Re: [PATCH net-next v9 0/9] Implement devlink-rate API and extend it

From: Wilczynski, Michal <hidden>
Date: 2022-11-08 18:11:38


On 11/8/2022 5:54 PM, Jacob Keller wrote:

On 11/8/2022 12:08 AM, Wilczynski, Michal wrote:
quoted

On 11/7/2022 7:31 PM, Jakub Kicinski wrote:
quoted
On Mon, 7 Nov 2022 19:18:10 +0100 Wilczynski, Michal wrote:
quoted
I provided some documentation in v10 in ice.rst file.
Unfortunately there is no devlink-rate.rst as far as I can
tell and at some point we even discussed adding this with Edward,
but honestly I think this could be added in a separate patch
series to not unnecessarily prolong merging this.
You can't reply to email and then immediately post a new version :/
How am I supposed to have a conversation with you? Extremely annoying.
I'm sorry if you find this annoying, however I can't see any harm here ?
I fixed some legit issues that you've pointed in v9, wrote some
documentation and basically said, "I wrote some documentation in
the next patchset, is it enough ?". I think it's better to get feedback
for smaller commits faster, this way I send the updated patchset
quickly.
From 
https://docs.kernel.org/process/maintainer-netdev.html?highlight=netdev
quoted
2. netdev FAQ¶
2.1. tl;dr¶
designate your patch to a tree - [PATCH net] or [PATCH net-next]

for fixes the Fixes: tag is required, regardless of the tree

don’t post large series (> 15 patches), break them up

don’t repost your patches within one 24h period

reverse xmas tree
Giving everyone at least 24 hours per posting (if not more) helps 
ensure that limited reviewer time doesn't get overloaded by constantly 
re-reviewing the same code. It also helps ensure that everyone has a 
chance to look at the patches.
Again, I'm sorry if my actions were annoying, or seemed overly impatient.
My question would be then, when I am allowed to re-post v11 ?
Technically I didn't re-post my series during 24h period, I just responded
to comments after re-sending a new version :(.
I guess I should wait for Kuba response now, and only after getting a
response re-send a v11  ?

Thanks,
Michał

Thanks,
Jake
  
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help