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

Re: [Xen-devel] [PATCH] x86/HAP: avoid using bogus/misleading locking

To: Jan Beulich <JBeulich@xxxxxxxx>, xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>
From: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
Date: Mon, 29 May 2017 14:55:17 +0100
Cc: George Dunlap <George.Dunlap@xxxxxxxxxxxxx>, Julien Grall <julien.grall@xxxxxxx>, Tim Deegan <tim@xxxxxxx>
Delivery-date: Mon, 29 May 2017 13:55:34 +0000
List-id: Xen developer discussion <xen-devel.lists.xen.org>

On 29/05/2017 13:37, Jan Beulich wrote:
> hap_teardown() unconditionally releases the paging lock and is always
> being called without the lock held: Lock acquire should then be
> unconditional too.
>
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>

Reviewed-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel

Follow-Ups:
- Re: [Xen-devel] [PATCH] x86/HAP: avoid using bogus/misleading locking
  - From: Julien Grall

References:
- [Xen-devel] [PATCH] x86/HAP: avoid using bogus/misleading locking
  - From: Jan Beulich

Prev by Date: Re: [Xen-devel] [PATCH v3 0/9] vpci: PCI config space emulation
Next by Date: Re: [Xen-devel] [PATCH v3 0/9] vpci: PCI config space emulation
Previous by thread: [Xen-devel] [xen-unstable-smoke test] 109887: regressions - trouble: blocked/broken/fail/pass
Next by thread: Re: [Xen-devel] AMD Ryzen Support
Index(es):
- Date
- Thread

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