linux-fsdevel.vger.kernel.org archive mirror
 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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).