From: Vlastimil Babka <vbabka@suse.cz>
To: Dave Chinner <david@fromorbit.com>, Michal Hocko <mhocko@kernel.org>
Cc: Johannes Weiner <hannes@cmpxchg.org>,
lsf-pc@lists.linux-foundation.org, linux-mm@kvack.org,
linux-fsdevel@vger.kernel.org
Subject: Re: [LSF/MM TOPIC] proposals for topics
Date: Mon, 1 Feb 2016 13:24:12 +0100 [thread overview]
Message-ID: <56AF4E6C.7010408@suse.cz> (raw)
In-Reply-To: <20160131232901.GO20456@dastard>
On 02/01/2016 12:29 AM, Dave Chinner wrote:
> On Thu, Jan 28, 2016 at 11:04:23PM +0100, Michal Hocko wrote:
>> On Fri 29-01-16 07:55:25, Dave Chinner wrote:
>>> On Tue, Jan 26, 2016 at 10:50:23AM +0100, Michal Hocko wrote:
>> [...]
>>>> There have been patches posted during the year to fortify those places
>>>> which cannot cope with allocation failures for ext[34] and testing
>>>> has shown that ext* resp. xfs are quite ready to see NOFS allocation
>>>> failures.
>>>
>>> The XFS situation is compeletely unchanged from last year, and the
>>> fact that you say it handles NOFS allocation failures just fine
>>> makes me seriously question your testing methodology.
>>
>> I am quite confused now. I remember you were the one who complained
>> about the silent nofail behavior of the allocator because that means
>> you cannot implement an appropriate fallback strategy.
>
> I complained about the fact the allocator did not behave as
> documented (or expected) in that it didn't fail allocations we
> expected it to fail.
Yes, I believe this is exactly what Michal was talking about in the
original e-mail:
> - GFP_NOFS is another one which would be good to discuss. Its primary
> use is to prevent from reclaim recursion back into FS. This makes
> such an allocation context weaker and historically we haven't
> triggered OOM killer and rather hopelessly retry the request and
> rely on somebody else to make a progress for us. There are two issues
> here.
> First we shouldn't retry endlessly and rather fail the allocation and
> allow the FS to handle the error. As per my experiments most FS cope
> with that quite reasonably. Btrfs unfortunately handles many of those
> failures by BUG_ON which is really unfortunate.
So this should address your complain above.
>> That being said, I do understand that allowing GFP_NOFS allocation to
>> fail is not an easy task and nothing to be done tomorrow or in few
>> months, but I believe that a discussion with FS people about what
>> can/should be done in order to make this happen is valuable.
>
> The discussion - from my perspective - is likely to be no different
> to previous years. None of the proposals that FS people have come up
> to address the "need memory allocation guarantees" issue have got
> any traction on the mm side. Unless there's something fundamentally
> new from the MM side that provides filesystems with a replacement
> for __GFP_NOFAIL type behaviour, I don't think further discussion is
> going to change the status quo.
Yeah, the guaranteed reserves as discussed last year didn't happen so
far. But that's a separate issue than GPF_NOFS *without* __GFP_NOFAIL.
It just got mixed up in this thread.
> Cheers,
>
> Dave.
>
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
next prev parent reply other threads:[~2016-02-01 12:24 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-01-25 13:33 [LSF/MM TOPIC] proposals for topics Michal Hocko
2016-01-25 14:21 ` [Lsf-pc] " Jan Kara
2016-01-25 14:40 ` Michal Hocko
2016-01-25 15:08 ` Tetsuo Handa
2016-01-26 9:43 ` Michal Hocko
2016-01-27 13:44 ` Tetsuo Handa
2016-01-27 14:33 ` [Lsf-pc] " Jan Kara
2016-01-25 18:45 ` Johannes Weiner
2016-01-26 9:50 ` Michal Hocko
2016-01-26 17:17 ` Vlastimil Babka
2016-01-26 17:20 ` [Lsf-pc] " Jan Kara
2016-01-27 9:08 ` Michal Hocko
2016-01-28 20:55 ` Dave Chinner
2016-01-28 22:04 ` Michal Hocko
2016-01-31 23:29 ` Dave Chinner
2016-02-01 12:24 ` Vlastimil Babka [this message]
2016-01-26 17:07 ` Vlastimil Babka
2016-01-26 18:09 ` Johannes Weiner
2016-01-30 18:18 ` Greg Thelen
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=56AF4E6C.7010408@suse.cz \
--to=vbabka@suse.cz \
--cc=david@fromorbit.com \
--cc=hannes@cmpxchg.org \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=lsf-pc@lists.linux-foundation.org \
--cc=mhocko@kernel.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.