netdev - Re: [PATCH iproute2-next] iproute2: add 'ip monitor mcaddr' support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CADXeF1FivD14xVJ-12dMDO-b9TPX7Ya8H_1iBBgCNd_6Mu1+yg@mail.gmail.com>
Date: Tue, 19 Nov 2024 13:25:48 +0900
From: Yuyang Huang <yuyanghuang@...gle.com>
To: Andrew Lunn <andrew@...n.ch>
Cc: Hangbin Liu <liuhangbin@...il.com>, "David S. Miller" <davem@...emloft.net>, 
	Eric Dumazet <edumazet@...gle.com>, Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>, 
	Simon Horman <horms@...nel.org>, David Ahern <dsahern@...nel.org>, roopa@...ulusnetworks.com, 
	jiri@...nulli.us, stephen@...workplumber.org, jimictw@...gle.com, 
	prohr@...gle.com, nicolas.dichtel@...nd.com, netdev@...r.kernel.org, 
	Maciej Żenczykowski <maze@...gle.com>, 
	Lorenzo Colitti <lorenzo@...gle.com>
Subject: Re: [PATCH iproute2-next] iproute2: add 'ip monitor mcaddr' support

Thanks for the suggestion! I will split the patch into two parts in v2.

Thanks,
Yuyang

Thanks,
Yuyang


On Tue, Nov 19, 2024 at 10:06 AM Andrew Lunn <andrew@...n.ch> wrote:
>
> On Mon, Nov 18, 2024 at 10:19:59PM +0900, Yuyang Huang wrote:
> > Thanks for the prompt review feedback.
> >
> > >No need changes for headers. Stephen will sync the headers.
> >
> > The patch will not compile without the header changes. I guess that
> > means I should put the patch on hold until the kernel change is merged
> > and the header changes get synced up to iproute2?
>
> What you can do is have two patches. The first updates the headers,
> the second adds the new functionality. What then happens is that the
> headers get syncd once, and then all patches queues up for merging get
> applied, dropping the first patch of each series.
>
>         Andrew