Re: [Xen-devel] race condition in xen-gntdev

[Marek Marczykowski-GÃrecki <marmarek@xxxxxxxxxxxxxxxxxxxxxx>]

On Tue, Jun 30, 2015 at 04:26:45PM +0100, David Vrabel wrote:
> On 26/06/15 02:28, Marek Marczykowski-GÃrecki wrote:
> > 
> > From b876e14888bdafa112c3265e6420543fa74aa709 Mon Sep 17 00:00:00 2001
> > From: =?UTF-8?q?Marek=20Marczykowski-G=C3=B3recki?=
> >  <marmarek@xxxxxxxxxxxxxxxxxxxxxx>
> > Date: Fri, 26 Jun 2015 02:16:49 +0200
> > Subject: [PATCH] xen/grant: fix race condition in gntdev_release
> > 
> > While gntdev_release is called, MMU notifier is still registered and
> > can traverse priv->maps list even if no pages are mapped (which is the
> > case - gntdev_release is called after all). But gntdev_release will
> > clear that list, so make sure that only one of those things happens at
> > the same time.
> 
> Applied to for-linus-4.2 and tagged for stable, thanks.

Regarding stable backports - remember that before 4.0, there was a spin
lock not a mutex. I've stumbled on this while testing on 3.18...

-- 
Best Regards,
Marek Marczykowski-GÃrecki
Invisible Things Lab
A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?

Attachment: pgpoV9522BX85.pgp
Description: PGP signature

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