mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
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.

  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®