public inbox for linux-xfs@vger.kernel.org
 help / color / mirror / Atom feed
From: "Bhagi rathi" <jahnu77@gmail.com>
To: Eric Sandeen <sandeen@sandeen.net>
Cc: Lachlan McIlroy <lachlan@sgi.com>,
	sgi.bugs.xfs@engr.sgi.com, xfs@oss.sgi.com
Subject: Re: TAKE 981498 - use KM_MAYFAIL in xfs_mountfs
Date: Thu, 7 Aug 2008 22:48:46 +0530	[thread overview]
Message-ID: <cc7060690808071018n3a264ca3ld1368d27b5b2f59b@mail.gmail.com> (raw)
In-Reply-To: <489A01B0.5050606@sandeen.net>

On Thu, Aug 7, 2008 at 1:25 AM, Eric Sandeen <sandeen@sandeen.net> wrote:

> Bhagi rathi wrote:
> > Why are we going to block for ever? Mounting a file-system
> > requires in-core log space buffers, reading of other buffers
> > which needs allocation of memory greater than per ag
> > structures.
> >
> > I am trying to understand why xfs_perag_t? Mount/Unmount
> > are not frequent activities, it is better for them to succeed
> > if operating system can allocate memory and take them
> > forward.
>
> But that's the big if, right?
>
> If the system is so starved that you can't get this memory to even start
> the mount process, I'm sure it's better to fail the mount with -ENOMEM
> than to add to the current system memory stress.


 Not really. It is going to fail many automated scripts.  We are designing
 for a problem that system  is starved with memory. It points to a bug
 in memory & system dirty state cleaning, they are the ideal problems
 to be solved this instead of this.  As long as system recovers, it is  good
 not to disturb automated scripts by introducing these kind of  unnecessary
 failures of the mount command.



>
>
> In general KM_MAYFAIL sounds like a good plan when you can handle the
> failure gracefully, I think.


 Not really. It fails mount gracefully, however, it needs administrative
action.
 It is expected that operating system will recover and functional without
 admin intervention.

Cheers,
Bhagi.

>
>
> -Eric
>


[[HTML alternate version deleted]]

      parent reply	other threads:[~2008-08-07 17:17 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-08-06  5:41 TAKE 981498 - use KM_MAYFAIL in xfs_mountfs Lachlan McIlroy
2008-08-06 17:22 ` Bhagi rathi
2008-08-06 19:55   ` Eric Sandeen
2008-08-06 20:22     ` Dave Chinner
2008-08-07 17:23       ` Bhagi rathi
2008-08-08  0:31         ` Dave Chinner
2008-08-08  5:22           ` Bhagi rathi
2008-08-07 17:18     ` Bhagi rathi [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=cc7060690808071018n3a264ca3ld1368d27b5b2f59b@mail.gmail.com \
    --to=jahnu77@gmail.com \
    --cc=lachlan@sgi.com \
    --cc=sandeen@sandeen.net \
    --cc=sgi.bugs.xfs@engr.sgi.com \
    --cc=xfs@oss.sgi.com \
    /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