| Message ID | 20231107123825.61051-1-linyunsheng@huawei.com (mailing list archive) |
|---|---|
| State | Superseded |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | [net,v2] page_pool: Add myself as page pool reviewer in MAINTAINERS | expand |
On Tue, 7 Nov 2023 20:38:24 +0800 Yunsheng Lin wrote: > I have added frag support for page pool, made some improvement > for it recently, and reviewed some related patches too. > > So add myself as reviewer so that future patch will be cc'ed > to my email. Not sure what to do about this, it feels somewhat wrong to add as a reviewer someone who seem to not follow our basic process requirements: https://www.kernel.org/doc/html/next/process/maintainer-netdev.html :(
On 2023/11/8 1:49, Jakub Kicinski wrote: > On Tue, 7 Nov 2023 20:38:24 +0800 Yunsheng Lin wrote: >> I have added frag support for page pool, made some improvement >> for it recently, and reviewed some related patches too. >> >> So add myself as reviewer so that future patch will be cc'ed >> to my email. > > Not sure what to do about this, it feels somewhat wrong to add > as a reviewer someone who seem to not follow our basic process > requirements: > > https://www.kernel.org/doc/html/next/process/maintainer-netdev.html I reread the above doc in order not to miss anything obvious again:( I suppose basic process requirements mean: 1. designate your patch to a tree - [PATCH net] or [PATCH net-next] 2. don't repost your patches within one 24h period For 1, somehow I got the impression that changing to MAINTAINERS can be targetted to net branch, so that the change can flow to other main trees quickly, maybe I was wrong? For 2, yes, maybe I should stick to the rule even if it is a simple patch and obivous format error. Please correct me if there is anything else I missed. > > :( > > . >
Hi Jakub, On Wed, 8 Nov 2023 at 05:31, Yunsheng Lin <linyunsheng@huawei.com> wrote: > > On 2023/11/8 1:49, Jakub Kicinski wrote: > > On Tue, 7 Nov 2023 20:38:24 +0800 Yunsheng Lin wrote: > >> I have added frag support for page pool, made some improvement > >> for it recently, and reviewed some related patches too. > >> > >> So add myself as reviewer so that future patch will be cc'ed > >> to my email. > > > > Not sure what to do about this, it feels somewhat wrong to add > > as a reviewer someone who seem to not follow our basic process > > requirements: > > > > https://www.kernel.org/doc/html/next/process/maintainer-netdev.html > > I reread the above doc in order not to miss anything obvious again:( > > I suppose basic process requirements mean: > 1. designate your patch to a tree - [PATCH net] or [PATCH net-next] > 2. don't repost your patches within one 24h period > > For 1, somehow I got the impression that changing to MAINTAINERS > can be targetted to net branch, so that the change can flow to other > main trees quickly, maybe I was wrong? > > For 2, yes, maybe I should stick to the rule even if it is a simple > patch and obivous format error. > > Please correct me if there is anything else I missed. FWIW Yunsheng has been helping a lot with patch reviews. So once the procedural issues are resolved, I support having him as a designated reviewer Thanks /Ilias > > > > > > :( > > > > . > >
On Wed, 8 Nov 2023 11:31:45 +0800 Yunsheng Lin wrote: > For 2, yes, maybe I should stick to the rule even if it is a simple > patch and obivous format error. Yes, maybe you should.
On 2023/11/9 0:47, Jakub Kicinski wrote: > On Wed, 8 Nov 2023 11:31:45 +0800 Yunsheng Lin wrote: >> For 2, yes, maybe I should stick to the rule even if it is a simple >> patch and obivous format error. > > Yes, maybe you should. Thanks for clarifying. Maybe I should be targetting the net-next branch for the repost after merge window open, in order not to cause any confusion. > . >
diff --git a/MAINTAINERS b/MAINTAINERS index 14e1194faa4b..67817d80c9cc 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -16242,6 +16242,7 @@ F: mm/truncate.c PAGE POOL M: Jesper Dangaard Brouer <hawk@kernel.org> M: Ilias Apalodimas <ilias.apalodimas@linaro.org> +R: Yunsheng Lin <linyunsheng@huawei.com> L: netdev@vger.kernel.org S: Supported F: Documentation/networking/page_pool.rst
I have added frag support for page pool, made some improvement for it recently, and reviewed some related patches too. So add myself as reviewer so that future patch will be cc'ed to my email. Signed-off-by: Yunsheng Lin <linyunsheng@huawei.com> CC: Jesper Dangaard Brouer <hawk@kernel.org> CC: Ilias Apalodimas <ilias.apalodimas@linaro.org> CC: David S. Miller <davem@davemloft.net> CC: Jakub Kicinski <kuba@kernel.org> CC: Paolo Abeni <pabeni@redhat.com> CC: Netdev <netdev@vger.kernel.org> --- V2: add missing ":" as pointed out by Jesper --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+)