Hi Jan,
On 28/05/2021 14:29, Jan Beulich wrote:
On 26.05.2021 17:21, Julien Grall wrote:
From: Julien Grall
Since XSA-288 (commit 02cbeeb62075 "gnttab: split maptrack lock to make
XSA-228 I suppose?
Yes, I will update in the next version.
it fulfill its purpose again"), v->maptrack_head
On 26.05.2021 17:21, Julien Grall wrote:
> From: Julien Grall
>
> Since XSA-288 (commit 02cbeeb62075 "gnttab: split maptrack lock to make
XSA-228 I suppose?
> it fulfill its purpose again"), v->maptrack_head and v->maptrack_tail
> are with the lock v->maptrack_freelist_lock held.
Nit: missing
From: Julien Grall
Since XSA-288 (commit 02cbeeb62075 "gnttab: split maptrack lock to make
it fulfill its purpose again"), v->maptrack_head and v->maptrack_tail
are with the lock v->maptrack_freelist_lock held.
Therefore it is not necessary to update the fields using cmpxchg()
and also read them
