| To: | Al Viro <viro@xxxxxxxxxxxxxxxxxx>, Oleg Nesterov <oleg@xxxxxxxxxx> |
|---|---|
| Subject: | Re: 3.14-rc2 XFS backtrace because irqs_disabled. |
| From: | Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx> |
| Date: | Wed, 12 Feb 2014 13:32:55 -0800 |
| Cc: | Dave Chinner <david@xxxxxxxxxxxxx>, Dave Jones <davej@xxxxxxxxxx>, Eric Sandeen <sandeen@xxxxxxxxxxx>, Linux Kernel <linux-kernel@xxxxxxxxxxxxxxx>, xfs@xxxxxxxxxxx |
| Delivered-to: | xfs@xxxxxxxxxxx |
| Dkim-signature: | v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:sender:in-reply-to:references:date:message-id:subject :from:to:cc:content-type; bh=2BWEZifKaySmKpj3guv694VQDRTiHzMRK4nPb/0JtKU=; b=PL1s2mksb+HzPdjltYQK//VHpyc4uPbMhwdkQ1N74fVpWkNmq+mCH2H64ouJGVHtBt JNi6txNlwTot3KZUJB9ZhM/aRt7tn7ywxnXuh/pw/s98AOBmUAHLolClm7ep9DKQ0RYp GX1SemZjbI4lTNfOB90GPwfhSCkFY6UcACyz7KO50PftVAFN8GDGOyDAd2C7HS9q52jm dv2z1d2TSVRwbdZ5R1mmRQyitkGu/TID/vUjB9XbsSrZ5n3OffwSQmGQLAgEaDElnh3K DTc9H+nR4ZxPslSIbSnBBtLNSuf/ZLfO8qGBqAHRIw4pYgLc1rG53LIef2075PKZvo4M n1yg== |
| Dkim-signature: | v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux-foundation.org; s=google; h=mime-version:sender:in-reply-to:references:date:message-id:subject :from:to:cc:content-type; bh=2BWEZifKaySmKpj3guv694VQDRTiHzMRK4nPb/0JtKU=; b=HYcETt7DTRiy824eFGK9cGEIdtVdZM/7+iLJi4jCTEe+7+/+oDzhxqz0cJbiMuG3M0 xz3iKABPlpn55G4779bhQVz5oN534GGrdmYXbiiSV70jyt7PlNfdkBqGII4jac37uQfZ 52KcgWdQ3Tr5N63i//yR4V69oGuySBU8QvGxo= |
| In-reply-to: | <20140212211421.GP18016@xxxxxxxxxxxxxxxxxx> |
| References: | <52FA9ADA.9040803@xxxxxxxxxxx> <20140212004403.GA17129@xxxxxxxxxx> <20140212010941.GM18016@xxxxxxxxxxxxxxxxxx> <CA+55aFwoWT-0A_KTkXMkNqOy8hc=YmouTMBgWUD_z+8qYPphjA@xxxxxxxxxxxxxx> <20140212040358.GA25327@xxxxxxxxxx> <20140212042215.GN18016@xxxxxxxxxxxxxxxxxx> <20140212054043.GB13997@dastard> <CA+55aFxy2t7bnCUc-DhhxYxsZ0+GwL9GuQXRYtE_VzqZusmB9A@xxxxxxxxxxxxxx> <20140212113928.GO18016@xxxxxxxxxxxxxxxxxx> <CA+55aFywwx0Q8xK2GJiRJ+FV7PQEKoBRxDUxW4052FVyd5XOpg@xxxxxxxxxxxxxx> <20140212211421.GP18016@xxxxxxxxxxxxxxxxxx> |
| Sender: | linus971@xxxxxxxxx |
On Wed, Feb 12, 2014 at 1:14 PM, Al Viro <viro@xxxxxxxxxxxxxxxxxx> wrote:
>
> Umm... What if we delay __sigqueue_free()? After all, that's where the
> fat sucker normally comes from. That way we might get away with much
> smaller structure on stack...
Sounds like the RightThing(tm) to do to me, and I don't see why it
wouldn't work.
We'd have to teach each user of "dequeue_signal()" to free the siginfo
thing. Which shouldn't be too bad - I think we've collected all of
that into generic code, and there isn't the mass or architecture code
that knows about these things any more. But there are a few odd
drivers etc and signalfd. I didn't look at what the lifetimes were.
Adding Oleg to the cc, since any time we touch any of that code, he
should be involved. Oleg - the issue is the biggish size of 'struct
ksignal' on stack, brought on by the silly "put a whole siginfo_t in
it".
Linus
|
| Previous by Date: | Re: 3.14-rc2 XFS backtrace because irqs_disabled., Dave Chinner |
|---|---|
| Next by Date: | Re: [PATCH v2] xfs_db: fix the setting of unaligned directory fields, Mark Tinguely |
| Previous by Thread: | Re: 3.14-rc2 XFS backtrace because irqs_disabled., Al Viro |
| Next by Thread: | Re: 3.14-rc2 XFS backtrace because irqs_disabled., Al Viro |
| Indexes: | [Date] [Thread] [Top] [All Lists] |