Re: [PATCH] [PATCH] linux-user/aarch64: Reset target data on MADV_DONTNEED

[Richard Henderson]

On 8/10/22 13:32, Vitaly Buka wrote:
> Sorry, I only noticed today that it's not submitted.
> Version is not critical for us, as we build from masters anyway.
> Richard, do you know a reason to consider this critical?
>
> On Wed, 10 Aug 2022 at 13:04, Peter Maydell <peter.maydell@linaro.org 
> <mailto:peter.maydell@linaro.org>> wrote:
>
>     On Wed, 10 Aug 2022 at 21:00, Vitaly Buka <vitalybuka@google.com
>     <mailto:vitalybuka@google.com>> wrote:
>      >
>      > How can we land this one?
>
>     Pinging it a week ago rather than now would have been a good start :-(
>     I think it got missed because you didn't cc the linux-user maintainer.
>
>     Is this a critical fix for 7.1 or can we let it slip to 7.2 ?

It's unfortunate that it got missed.  It's not critical, but it would be nice, because 
support for MADV_DONTNEED is new in 7.1 (previously, we ignored all madvise).

I'll note there are missing braces for coding style on an IF.

Laurent, do you have an objection to merging this for rc3?


r~