Re: [PATCH v7 00/30] NT synchronization primitive driver

[Elizabeth Figura]

On Tuesday, 7 January 2025 13:42:41 CST Peter Zijlstra wrote:
> On Tue, Jan 07, 2025 at 06:06:03PM +0100, Greg Kroah-Hartman wrote:
> > Given a lack of complaints, I've now applied this to my testing tree.
> > Thanks for sticking with it!
>
> Right, so I acked v6, which wasn't preserved. The v7 changes are minor
> and seem fine (IIRc I even suggested them some very very long time ago).

I wasn't sure it was fine to preserve the ack across changes (even if they were the ones you suggested) and decided to err on the safe side.

Thank you for the review!

[PATCH v7 00/30] NT synchronization primitive driver

[David Kahurani]

On Wed, Jan 8, 2025 at 12:16 AM Elizabeth Figura <zfigura(a)codeweavers.com>
wrote:
> On Tuesday, 7 January 2025 13:42:41 CST Peter Zijlstra wrote:
> > On Tue, Jan 07, 2025 at 06:06:03PM +0100, Greg Kroah-Hartman wrote:
> > > Given a lack of complaints, I've now applied this to my testing tree.
> > > Thanks for sticking with it!
> >
> > Right, so I acked v6, which wasn't preserved. The v7 changes are minor
> > and seem fine (IIRc I even suggested them some very very long time ago).
>
> I wasn't sure it was fine to preserve the ack across changes (even if they
> were the ones you suggested) and decided to err on the safe side.
>
> Thank you for the review!

If you work from ideas to code, it might help with the review problem.