[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] [PATCH] linux-user/aarch64: Reset target data on MADV_DONTNE
From: |
Peter Maydell |
Subject: |
Re: [PATCH] [PATCH] linux-user/aarch64: Reset target data on MADV_DONTNEED |
Date: |
Thu, 11 Aug 2022 12:54:49 +0100 |
On Thu, 11 Aug 2022 at 09:29, Laurent Vivier <laurent@vivier.eu> wrote:
>
> Le 10/08/2022 à 22:47, Richard Henderson a écrit :
> > 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?
> >
>
> No objection.
>
> Do you want it goes via the arm branch or via the linux-user branch?
>
> If it goes via linux-user I can run the LTP testsuite but it takes 1 day.
I think we should definitely run the LTP testsuite on it, so
taking it via linux-user probably makes more sense.
thanks
-- PMM
- Re: [PATCH] [PATCH] linux-user/aarch64: Reset target data on MADV_DONTNEED, Vitaly Buka, 2022/08/10
- Re: [PATCH] [PATCH] linux-user/aarch64: Reset target data on MADV_DONTNEED, Peter Maydell, 2022/08/10
- Re: [PATCH] [PATCH] linux-user/aarch64: Reset target data on MADV_DONTNEED, Vitaly Buka, 2022/08/10
- Re: [PATCH] [PATCH] linux-user/aarch64: Reset target data on MADV_DONTNEED, Richard Henderson, 2022/08/10
- Re: [PATCH] [PATCH] linux-user/aarch64: Reset target data on MADV_DONTNEED, Laurent Vivier, 2022/08/11
- Re: [PATCH] [PATCH] linux-user/aarch64: Reset target data on MADV_DONTNEED,
Peter Maydell <=
- Re: [PATCH] [PATCH] linux-user/aarch64: Reset target data on MADV_DONTNEED, Laurent Vivier, 2022/08/11
- Re: [PATCH] [PATCH] linux-user/aarch64: Reset target data on MADV_DONTNEED, Alex Bennée, 2022/08/11
- Re: [PATCH] [PATCH] linux-user/aarch64: Reset target data on MADV_DONTNEED, Laurent Vivier, 2022/08/13
- Re: [PATCH] [PATCH] linux-user/aarch64: Reset target data on MADV_DONTNEED, Alex Bennée, 2022/08/16
- Re: [PATCH] [PATCH] linux-user/aarch64: Reset target data on MADV_DONTNEED, Laurent Vivier, 2022/08/16