On Thu, Feb 26, 2026 at 05:42:13PM -0800, Jay Vosburgh wrote: > >I added it here to help new readers and reviewers understand the logic > >quickly. If you think there’s no need to include it in the code, maybe > >we can move it to the commit description? > > Personally, I'd leave it out, and just put a reference. > > The churn machine isn't that critical, even the standards > committee didn't like it and removed it from the 2020 edition of > 802.1AX.
Oh, I didn't notice this. I posted a patch[1] to net-next to export the churn state via netlink recently. Should I revert this patch? > > Still, whatever we provide should work in accordance with the > 2014 standard we're nominally conforming to, so functionally the patch > looks fine to me. Got it, I will remove it. [1] http://lore.kernel.org/netdev/[email protected] Thanks Hangbin
