From: Nikita Danilov <Nikita@Namesys.COM>
To: erik@hensema.net
Cc: linux-kernel@vger.kernel.org
Subject: Re: incorrect inode count on reiserfs
Date: Mon, 8 Dec 2003 18:33:54 +0300 [thread overview]
Message-ID: <16340.39394.588507.612973@laputa.namesys.com> (raw)
In-Reply-To: <slrnbt9322.27h.erik@bender.home.hensema.net>
Erik Hensema writes:
> Nikita Danilov (Nikita@Namesys.COM) wrote:
> > Petr Sebor writes:
> > > I have noticed this behavior when moving the inn2 news server to
> > > 2.6.0-test11 kernel
> > > from 2.4.23
> > > (inn2 refuses to start because if free inode shortage)
>
> [...]
>
> > reiserfs has no fixed predefined number of inodes on the file
> > system. Hence, field f_files of struct statfs (see man 2 statfs) is not
> > applicable to this file system. Man page explicitly says:
> >
> > Fields that are undefined for a particular file system are
> > set to 0.
> >
> > Previous man page stated that file system should put -1 (4294967295)
> > into undefined fields. Reiserfs has been changed to conform to the
> > changed specification.
>
> [...]
>
> > Fix would really be simple: ignore test results if ->f_files is 0 or
> > 0xffffffff.
>
> But innwatch checks for a out-of-inodes condition. How can it differentiate
> between a undefined number of inodes (field set to 0) and a system that ran
> out of inodes (field dropped to 0)?
>
> A '4294967295 inodes should be enough for anyone'-situation is preferable I
> think.
This is messy, because we have both statfs and statfs64 and this would
lead to the overflow detection problems.
I don't know what is the best solution here. statfs(2) is just not very
good interface. It was obviously designed to serve ffs/ufs/ext2 type
file systems only.
Looking at the magic in f_type field, is ugly, but should work.
>
> --
> Erik Hensema <erik@hensema.net>
Nikita.
next prev parent reply other threads:[~2003-12-08 15:34 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-12-08 13:26 Petr Sebor
2003-12-08 13:51 ` Nikita Danilov
2003-12-08 14:06 ` Petr Sebor
2003-12-08 14:11 ` Andreas Schwab
2003-12-08 14:35 ` Erik Hensema
2003-12-08 15:33 ` Nikita Danilov [this message]
2003-12-08 16:09 ` Matthias Urlichs
2003-12-08 17:24 ` Erik Hensema
2003-12-09 3:01 ` Herbert Poetzl
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=16340.39394.588507.612973@laputa.namesys.com \
--to=nikita@namesys.com \
--cc=erik@hensema.net \
--cc=linux-kernel@vger.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®