2024-01-25 18:33:56

by Dave Hansen

Subject: Re: [PATCH] x86/mm/cpa-test: Correct length selection

On 1/25/24 10:24, Jason Andryuk wrote:
> The length of pages to modify corresponding to addr[i] is given in
> len[i]. Remove the hard coded 1.
>
> Signed-off-by: Jason Andryuk <[email protected]>
> ---
> This is a typo, right? The other cases use len[i].

Sure looks like it to me.

Any idea how this could have gone unnoticed for 5 years?

2024-01-25 18:55:46

by Jason Andryuk

Subject: Re: [PATCH] x86/mm/cpa-test: Correct length selection

On Thu, Jan 25, 2024 at 1:30 PM Dave Hansen <[email protected]> wrote:
>
> On 1/25/24 10:24, Jason Andryuk wrote:
> > The length of pages to modify corresponding to addr[i] is given in
> > len[i]. Remove the hard coded 1.
> >
> > Signed-off-by: Jason Andryuk <[email protected]>
> > ---
> > This is a typo, right? The other cases use len[i].
>
> Sure looks like it to me.
>
> Any idea how this could have gone unnoticed for 5 years?

Not really, which is why I wondered if I was missing something.

Regards,
Jason