[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v2 9/9] x86/shadow: adjust TLB flushing in sh_unshadow_for_p2m_change()

To: Jan Beulich <jbeulich@xxxxxxxx>
From: Tim Deegan <tim@xxxxxxx>
Date: Sat, 7 Nov 2020 08:48:04 +0000
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, George Dunlap <George.Dunlap@xxxxxxxxxxxxx>
Delivery-date: Sat, 07 Nov 2020 08:48:16 +0000
List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

At 10:39 +0100 on 06 Nov (1604659168), Jan Beulich wrote:
> Accumulating transient state of d->dirty_cpumask in a local variable is
> unnecessary here: The flush is fine to make with the dirty set at the
> time of the call. With this, move the invocation to a central place at
> the end of the function.
> 
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>

Acked-by: Tim Deegan <tim@xxxxxxx>

References:
- [PATCH v2 0/9] x86/p2m: hook adjustments
  - From: Jan Beulich
- [PATCH v2 9/9] x86/shadow: adjust TLB flushing in sh_unshadow_for_p2m_change()
  - From: Jan Beulich

Prev by Date: Re: [PATCH v2 8/9] x86/shadow: cosmetics to sh_unshadow_for_p2m_change()
Next by Date: [xen-4.14-testing test] 156525: tolerable FAIL - PUSHED
Previous by thread: [PATCH v2 9/9] x86/shadow: adjust TLB flushing in sh_unshadow_for_p2m_change()
Next by thread: [xen-unstable-smoke test] 156523: tolerable all pass - PUSHED
Index(es):
- Date
- Thread

Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.