[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 2/2] x86: Add Support for Paging-Write Feature
- To: Jan Beulich <jbeulich@xxxxxxxx>
- From: Petr Beneš <w1benny@xxxxxxxxx>
- Date: Fri, 20 Dec 2024 17:34:22 +0100
- Cc: Tamas K Lengyel <tamas@xxxxxxxxxxxxx>, Alexandru Isaila <aisaila@xxxxxxxxxxxxxxx>, Petre Pircalabu <ppircalabu@xxxxxxxxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Bertrand Marquis <bertrand.marquis@xxxxxxx>, Michal Orzel <michal.orzel@xxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Anthony PERARD <anthony.perard@xxxxxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx
- Delivery-date: Fri, 20 Dec 2024 16:35:01 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On Fri, Dec 20, 2024 at 2:09 PM Jan Beulich <jbeulich@xxxxxxxx> wrote:
>
> To avoid the flag becoming "sticky", I think you need to clear it ahead of
> this switch() (or right at the start of the function).
>
> Jan
When thinking about it now, wouldn't it make sense to only reset it in
the p2m_ram_rw case above? I mean, where else would be guest
page-tables located.
P.
|