From: Ian Kent <raven@themaw.net>
To: Heiko Carstens <heiko.carstens@de.ibm.com>
Cc: David Rientjes <rientjes@google.com>,
Andrew Morton <akpm@linux-foundation.org>,
Hendrik Brueckner <brueckner@linux.vnet.ibm.com>,
Thorsten Diehl <thorsten.diehl@de.ibm.com>,
Andrea Righi <andrea@betterlinux.com>,
Christoph Hellwig <hch@infradead.org>,
Al Viro <viro@zeniv.linux.org.uk>,
Stefan Bader <stefan.bader@canonical.com>,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/2] /proc/stat vs. failed order-4 allocation
Date: Thu, 26 Jun 2014 10:20:01 +0800 [thread overview]
Message-ID: <1403749201.2969.18.camel@perseus.fritz.box> (raw)
In-Reply-To: <20140625061551.GA4260@osiris>
On Wed, 2014-06-25 at 08:15 +0200, Heiko Carstens wrote:
> On Tue, Jun 24, 2014 at 04:52:22PM -0700, David Rientjes wrote:
> > On Mon, 23 Jun 2014, Andrew Morton wrote:
> > > On Sat, 21 Jun 2014 11:10:58 +0200 Heiko Carstens <heiko.carstens@de.ibm.com> wrote:
> > > > On Wed, Jun 18, 2014 at 02:29:31PM -0700, Andrew Morton wrote:
> > > > > I'm unclear on how urgent these fixes are. I semi-randomly tagged them
> > > > > for 3.16 with a -stable backport, but that could be changed?
> > > >
> > > > I assume tagged for 3.16 means you intend to get it merged before 3.16
> > > > gets released?
> > > > If so, then that would be fine with me.
> > >
> > > um, actually, no, sorry, I meant merge for 3.17-rc1 with a -stable backport.
> > >
> > > We can do 3.16 of course, but for what reasons?
> >
> > Heiko's patches should also fix
> > https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1319244 as reported
> > by Stefan Bader. I've pinged them to determine if there is any other
> > issues with -mm.
>
> The other distribution where this has been seen was RHEL7 (on s390).
>
> FWIW, I thought there was a 'rule' that patches with a -stable tag shouldn't
> stay in -next for a couple of rc releases. But then again that probably was
> just a discussion somewhere.
> Anyway, if neither Stefan or Ian speak up I'm fine with post 3.16 as well.
>
It's very much up to Andrew, of course.
I would like to see this go in sooner rather than later, if possible,
due to the potential RHEL-7 disruption it can cause. More so now the
above shows we can see this on other architectures and higher speced
machines.
Looking at the patches it appears there's no change from the original
semantics.
So the potential for regression is fairly low now since Heiko's patches
are much simpler than the original proposal and are much more straight
forward to review, so much so that they make sense even to me!
Ian
prev parent reply other threads:[~2014-06-26 2:20 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-06-16 9:04 [PATCH 0/2] /proc/stat vs. failed order-4 allocation Heiko Carstens
2014-06-16 9:04 ` [PATCH 1/2] proc/stat: convert to single_open_size() Heiko Carstens
2014-06-18 23:28 ` David Rientjes
2014-06-16 9:04 ` [PATCH 2/2] fs/seq_file: fallback to vmalloc allocation Heiko Carstens
2014-06-18 23:31 ` David Rientjes
2014-06-18 21:29 ` [PATCH 0/2] /proc/stat vs. failed order-4 allocation Andrew Morton
2014-06-21 9:10 ` Heiko Carstens
2014-06-23 23:04 ` Andrew Morton
2014-06-24 23:52 ` David Rientjes
2014-06-25 6:15 ` Heiko Carstens
2014-06-26 2:20 ` Ian Kent [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1403749201.2969.18.camel@perseus.fritz.box \
--to=raven@themaw.net \
--cc=akpm@linux-foundation.org \
--cc=andrea@betterlinux.com \
--cc=brueckner@linux.vnet.ibm.com \
--cc=hch@infradead.org \
--cc=heiko.carstens@de.ibm.com \
--cc=linux-kernel@vger.kernel.org \
--cc=rientjes@google.com \
--cc=stefan.bader@canonical.com \
--cc=thorsten.diehl@de.ibm.com \
--cc=viro@zeniv.linux.org.uk \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox