All of lore.kernel.org
 help / color / mirror / Atom feed
From: Zhiyong Landen tian <Zhiyong.Tian@Sun.COM>
To: Jan Kara <jack@suse.cz>
Cc: Andrew Perepechko <Andrew.Perepechko@Sun.COM>,
	linux-fsdevel@vger.kernel.org,
	Johann Lombardi <Johann.Lombardi@Sun.COM>
Subject: Re: [PATCH] quota: additional range checks and mem_dqblk	updates?to handle 64-bit limits
Date: Tue, 11 Mar 2008 11:16:02 +0800	[thread overview]
Message-ID: <47D5F972.10504@sun.com> (raw)
In-Reply-To: <20080310152059.GJ24873@duck.suse.cz>

Jan Kara wrote:
> On Sat 08-03-08 03:56:17, Andrew Perepechko wrote:
>   
>> Jan, this idea would work for us for now. As Andreas pointed out, we deal 
>> with MB-aligned quota limits, so this would give us the largest possible block 
>> quota limit value of 4 PB.
>>
>> However, I feel that it is worth to implement a clean 64-bit format, so that
>> we avoid additional semantics (like quota unit size) and do make quota scale 
>> better (4 PB clusters already exist).
>>     
>   Wow. I wonder when 64-bits won't be enough :). Anyway, my point was that
> when you get to 4 PB limits, you can just run: "setquota --set-scale 1GB"
> (hopefully at that time you won't care about rounding limits to 1GB) and
> you are at 4 HB limits (or what is the right suffix). No quota format
> change needed.
>   But what I fear more is that we may run out of 2^32 limit on the number
> of files one user can have (I don't have experience with that large systems
> but I guess you are comming near to 2^32 files on the filesystem, aren't
> you?). And for that we would have to do basically the changes you've
> suggested
The core of the "scale" way is: it does harm to the accuracy of quota so 
that we can set a larger
quota limitation. If the scale is 1G, 500M quota the user sets is equal 
to 0. Some customers may
like it; others may confuse. Anyway, we can't regulate the way of users 
using quota.

For lustre, I have a plan to set a minimum unit to 1K instead of current 
1M. So 64bit quota is
a must. I just wonder why not we implement these two points in different 
patches:
1. 64bit quota limitation
2. give a "scale" to the users who are glad to adjust it.

  parent reply	other threads:[~2008-03-11  3:22 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-03-07  0:29 [PATCH] quota: additional range checks and mem_dqblk updates to handle 64-bit limits Andrew Perepechko
2008-03-07 16:00 ` Jan Kara
2008-03-07 21:10   ` Andreas Dilger
2008-03-10 14:54     ` Jan Kara
2008-03-08  0:56   ` Andrew Perepechko
2008-03-10 15:20     ` [PATCH] quota: additional range checks and mem_dqblk updates?to " Jan Kara
2008-03-10 22:46       ` Andrew Perepechko
2008-03-11  3:16       ` Zhiyong Landen tian [this message]
2008-03-10 16:28 ` [PATCH] quota: additional range checks and mem_dqblk updates to " Jan Kara
2008-03-10 21:25   ` Andrew Perepechko
2008-03-12 17:21     ` [PATCH] quota: additional range checks and mem_dqblk updates?to " Jan Kara
2008-03-12 22:35       ` Andrew Perepechko

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=47D5F972.10504@sun.com \
    --to=zhiyong.tian@sun.com \
    --cc=Andrew.Perepechko@Sun.COM \
    --cc=Johann.Lombardi@Sun.COM \
    --cc=jack@suse.cz \
    --cc=linux-fsdevel@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 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.