x86/mem-sharing: ensure consistent lock order in get_two_gfns()

While the comment validly says "Sort by domain, if same domain by gfn",
the implementation also included equal domain IDs in the first part of
the check, thus rending the second part entirely dead and leaving
deadlock potential when there's only a single domain involved.

Signed-off-by: Jan Beulich <jbeulich@suse.com>
Acked-by: Tamas K Lengyel <tamas@tklengyel.com>
master commit: 09af2d01a2
master date: 2021-07-07 12:35:12 +0200
This commit is contained in:
Jan Beulich committed 2021-07-15 09:23:10 +02:00
1 parent 16d2641ddf
commit e6d098e4cd
1 file changed
+1 -1
+1 -1
View File
@@ -576,7 +576,7 @@ do { \
dest ## _t = (source ## t) ?: &scratch_t; \
} while (0)
if ( (rd->domain_id <= ld->domain_id) ||
if ( (rd->domain_id < ld->domain_id) ||
((rd == ld) && (gfn_x(rgfn) <= gfn_x(lgfn))) )
{
assign_pointers(first, r);
Reference in new issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.