Re: [merged] mm-fix-potential-data-race-in-sys_swapon.patch removed from -mm tree

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

 



On Mon, 24 Aug 2015, Andrew Morton wrote:
> On Mon, 24 Aug 2015 12:46:54 -0700 (PDT) Hugh Dickins <hughd@xxxxxxxxxx> wrote:
> 
> > Adding Mel to Cc.
> > 
> > On Mon, 24 Aug 2015, akpm@xxxxxxxxxxxxxxxxxxxx wrote:
> > > 
> > > The patch titled
> > >      Subject: mm: fix potential data race in SyS_swapon
> > > has been removed from the -mm tree.  Its filename was
> > >      mm-fix-potential-data-race-in-sys_swapon.patch
> > > 
> > > This patch was dropped because it was merged into mainline or a subsystem tree
> > 
> > Administrative error?  I don't see this merged into mainline yet,
> > and didn't see your usual mail when you send in a batch to Linus.
> 
> Al Viro grabbed it and put it into linux-next.
> 
> He didn't include the cc:stable.  I stared at that for a while and
> decided to let it all stand - as you say, the -stable backport is
> marginal.  swapon isn't exactly a high-frequency operation.

Ah, thanks for the explanation, that makes sense: yes,
I Cc'ed Al on the original, as it is as much in his area as in ours.

Hugh

> 
> > And I wouldn't want it rushed too quickly to Linus: that stable
> > tag is barely justified, this is a very narrow race window that
> > has gone unnoticed for years, and swapon requires CAP_SYS_ADMIN.
> > 
> > But also I spotted Mel proposing a swap-over-NFS patch in this area
> > on LKML last Thursday: he appeared to be relying on the loop that I
> > remove here, so he might want to veto this one (though can always
> > reinstate what he needs later, if that's how it plays out).
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]