mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Jan Kara <jack@suse.cz>
To: Neil Brown <neilb@cse.unsw.edu.au>
Cc: Jan Kara <jack@suse.cz>,
	linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: RFC - tree quotas for Linux (2.4.12, ext2)
Date: Tue, 30 Oct 2001 13:33:09 +0100	[thread overview]
Message-ID: <20011030133309.F6302@atrey.karlin.mff.cuni.cz> (raw)
In-Reply-To: <15310.25406.789271.793284@notabene.cse.unsw.edu.au> <20011024171658.B10075@atrey.karlin.mff.cuni.cz> <15319.12709.29314.342313@notabene.cse.unsw.edu.au> <20011025174815.C4644@atrey.karlin.mff.cuni.cz> <15320.59456.565780.111066@notabene.cse.unsw.edu.au> <20011029150602.G11994@atrey.karlin.mff.cuni.cz> <15325.58603.350619.609850@notabene.cse.unsw.edu.au>
In-Reply-To: <15325.58603.350619.609850@notabene.cse.unsw.edu.au>

> On Monday October 29, jack@suse.cz wrote:
> > > 
> > > I accept that it does look like a bit of a hack.
> > > But I think it is simple, understandable, and predictable.
> > > And I think that (for me) the value of tree quotas is more than enough
> > > to offset that cost.
> >   I just don't like the idea that when you do lookup you can suddenly get
> > Disk quota exceeded... I'd concern this behaviour a bit nonintuitive. I agree
> > that if root makes lookup of every file after moving directories then this
> > doesn't happen but still I don't like the design :).
> > 
> 
> You cannot get "Disk quota exceeded" on a lookup. If treequota_check
> finds a discrepancy it fixes it with "notify_change" with
> ia_valid set to ATTR_FORCE | ATTR_TID.
> I changed quota_transfer to take ATTR_FORCE to mean "just do it, even
> if it exceeds quota, and don't give an error".   Given that ATTR_FORCE
> is not actually used at all in the current kernel, I felt fairly free
> to interpret it how I wanted.
  Hmm.. I should have read your patch more carefuly.. Sorry. 

> So the only non-intuitive thing that can happen is that you find your
> usage mysteriously changes.  However this can only happen after
> administrator intervention, and with uid quotas administrator
> intervention (e.g. chown -R) can equally cause mysterious changes of
> usage.
> 
> However I'm not particularly trying to convince anyone to use or
> approve of tree-quotas.  I was after comments to make sure that I
> hadn't missed something in thinking through the issues.  I thank you
> and others for your comments.  The fact that I am comfortable with my
> answers (though you may not be) encourages me that I haven't missed
> anything.
> 
> I will be using treequotas locally next year and will keep the
> patches on my web-page up-to-date.  I have heard from at least one
> person who thinks they might be useful, so there are probably a few
> dozen who might find it useful.
  :) I also think tree quotas are useful I'd just like to think of some
nicer solution...:)

								Honza
--
Jan Kara <jack@suse.cz>
SuSE CR Labs

  reply	other threads:[~2001-10-30 12:32 UTC|newest]

Thread overview: 76+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-10-18  5:06 Neil Brown
2001-10-18  5:53 ` Ben Greear
2001-10-18  8:38   ` James Sutherland
2001-10-18 20:20     ` Mike Fedyk
2001-10-18 20:47       ` Tim Walberg
2001-10-19  1:07         ` Neil Brown
2001-10-19  3:03           ` Rik van Riel
2001-10-19 11:50             ` Horst von Brand
2001-10-19 17:00               ` Mike Fedyk
2001-10-18 21:17       ` Andreas Dilger
2001-10-18 22:56         ` Mike Fedyk
2001-10-19  0:14           ` Horst von Brand
2001-10-19  0:51             ` Mike Fedyk
2001-10-19  1:13         ` Neil Brown
2001-10-19  0:53       ` Neil Brown
2001-10-24 15:16 ` Jan Kara
2001-10-24 15:34   ` James Sutherland
2001-10-24 15:39     ` Jan Kara
2001-10-24 15:50       ` James Sutherland
2001-10-24 17:41         ` Rik van Riel
2001-10-24 18:08           ` James Sutherland
2001-10-26 11:25     ` Pavel Machek
2001-10-24 21:24   ` Neil Brown
2001-10-25 15:48     ` Jan Kara
2001-10-26  4:36       ` Neil Brown
2001-10-29 14:06         ` Jan Kara
2001-10-29 23:23           ` Neil Brown
2001-10-30 12:33             ` Jan Kara [this message]
2001-10-19  3:00 Toivo Pedaste
2001-10-19  4:39 ` Neil Brown
2001-10-24 18:44 Jesse Pollard
2001-10-25  6:15 ` Albert D. Cahalan
     [not found] <fa.inl6g6v.1mmbp4@ifi.uio.no>
     [not found] ` <fa.heevhav.sjs8an@ifi.uio.no>
2001-11-18 22:15   ` Swap Dan Maas
2001-11-18 22:43     ` Swap François Cami
2001-11-19  9:18       ` Swap James A Sutherland
2001-11-19 10:51         ` Swap Remco Post
2001-11-19 13:33           ` Swap James A Sutherland
2001-11-19 13:46             ` Swap Remco Post
2001-11-19 16:58               ` Swap Rik van Riel
     [not found]                 ` <Pine.LNX.4.33L.0111191458150.1491-100000@duckman.distro.conecti va>
2001-11-19 21:13                   ` Swap Alex Bligh - linux-kernel
2001-11-19 21:17                     ` Swap Rik van Riel
     [not found]                       ` <Pine.LNX.4.33L.0111191917000.1491-100000@duckman.distro.conecti va>
2001-11-19 21:52                         ` Swap Alex Bligh - linux-kernel
2001-11-19 16:36             ` Swap Jesse Pollard
2001-11-20 14:51             ` Swap J.A. Magallon
2001-11-20 16:01               ` Swap Wolfgang Rohdewald
2001-11-20 16:06                 ` Swap Remco Post
2001-11-20 16:12                 ` Swap Nick LeRoy
2001-11-20 16:20                 ` Swap Richard B. Johnson
2001-11-20 17:14                   ` Swap Christopher Friesen
2001-11-20 17:40                     ` Swap Richard B. Johnson
2001-11-20 18:14                       ` Swap Nick LeRoy
2001-11-21 10:17                         ` Swap Helge Hafting
2001-11-21 11:17                           ` Swap Alan Cox
2001-11-20 23:20                       ` Swap Luigi Genoni
2001-11-21 16:44                       ` Swap Remco Post
2001-11-20 17:58                     ` Swap Wolfgang Rohdewald
2001-11-26 21:51                       ` [Linux-abi-devel] Swap Christoph Hellwig
2001-11-20 21:05                     ` Swap Steffen Persvold
2001-11-20 21:18                       ` Swap Mike Fedyk
2001-11-20 21:33                         ` Swap Nick LeRoy
2001-11-20 21:44                           ` Swap Mike Fedyk
2001-11-20 22:00                             ` Swap Nick LeRoy
2001-11-21 16:53                             ` Swap Remco Post
2001-11-20 21:43                         ` Swap Richard B. Johnson
2001-11-20 21:50                           ` NFS, Paging & Installing [was: Re: Swap] Mike Fedyk
2001-11-21  1:22                             ` Horst von Brand
2001-11-21  1:46                               ` Mike Fedyk
2001-11-21 10:55                                 ` Trond Myklebust
2001-11-22  5:16                                   ` Bernd Eckenfels
2001-11-22 12:19                                     ` Trond Myklebust
2001-11-23 19:33                                   ` Mike Fedyk
2001-11-20 21:19                       ` Swap Nick LeRoy
2001-11-21 16:48                       ` Swap Remco Post
2001-11-20 20:58               ` Swap Mike Fedyk
2001-11-19 10:03       ` Swap Tim Connors
2001-11-19 10:16         ` Swap Dan Maas

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=20011030133309.F6302@atrey.karlin.mff.cuni.cz \
    --to=jack@suse.cz \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=neilb@cse.unsw.edu.au \
    /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®