[Top] [All Lists]

Re: XFS causing stack overflow

To: Dave Chinner <david@xxxxxxxxxxxxx>
Subject: Re: XFS causing stack overflow
From: "Ryan C. England" <ryan.england@xxxxxxxxxxxxx>
Date: Mon, 12 Dec 2011 08:43:57 -0500
Cc: Andi Kleen <andi@xxxxxxxxxxxxxx>, Christoph Hellwig <hch@xxxxxxxxxxxxx>, linux-mm@xxxxxxxxx, xfs@xxxxxxxxxxx
In-reply-to: <20111212090033.GQ14273@dastard>
References: <CAAnfqPAm559m-Bv8LkHARm7iBW5Kfs7NmjTFidmg-idhcOq4sQ@xxxxxxxxxxxxxx> <20111209115513.GA19994@xxxxxxxxxxxxx> <20111209221956.GE14273__25752.826271537$1323469420$gmane$org@dastard> <m262hop5kc.fsf@xxxxxxxxxxxxxx> <20111210221345.GG14273@dastard> <20111211000036.GH24062@xxxxxxxxxxxxxxxxxx> <20111211230511.GH14273@dastard> <20111212023130.GI24062@xxxxxxxxxxxxxxxxxx> <20111212043657.GO14273@dastard> <20111212051311.GJ24062@xxxxxxxxxxxxxxxxxx> <20111212090033.GQ14273@dastard>
Is it possible to apply this patch to my current installation?  We use this box in production and the reboots that we're experiencing are an inconvenience.

Is there is a walkthrough on how to apply this patch?  If not, could your provide the steps necessary to apply successfully?  I would greatly appreciate it.

Thank you

On Mon, Dec 12, 2011 at 4:00 AM, Dave Chinner <david@xxxxxxxxxxxxx> wrote:
On Mon, Dec 12, 2011 at 06:13:11AM +0100, Andi Kleen wrote:
> > It's ~180 bytes, so it's not really that small.
> Quite small compared to what real code uses. And also fixed
> size.
> >
> > > is on the new stack. ISTs are not used for interrupts, only for
> > > some special exceptions.
> >
> > IST = ???
> That's a hardware mechanism on x86-64 to switch stacks
> (Interrupt Stack Table or somesuch)
> With ISTs it would have been possible to move the the pt_regs too,
> but the software mechanism is somewhat simpler.
> > at the top of the stack frame? Is the stack unwinder walking back
> > across the interrupt stack to the previous task stack?
> Yes, the unwinder knows about all the extra stacks (interrupt
> and exception stacks) and crosses them as needed.
> BTW I suppose it wouldn't be all that hard to add more stacks and
> switch to them too, similar to what the 32bit do_IRQ does.
> Perhaps XFS could just allocate its own stack per thread
> (or maybe only if it detects some specific configuration that
> is known to need much stack)

That's possible, but rather complex, I think.
> It would need to be per thread if you could sleep inside them.

Yes, we'd need to sleep, do IO, possibly operate within a
transaction context, etc, and a workqueue handles all these cases
without having to do anything special. Splitting the stack at a
logical point is probably better, such as this patch:



Dave Chinner

Ryan C. England
office: 704-799-6944 x158
cell:    980-521-2297
<Prev in Thread] Current Thread [Next in Thread>