linux-fsdevel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Andrew Perepechko <Andrew.Perepechko@Sun.COM>
To: Jan Kara <jack@suse.cz>
Cc: linux-fsdevel@vger.kernel.org,
	Johann Lombardi <Johann.Lombardi@Sun.COM>,
	Zhiyong Landen tian <Zhiyong.Tian@Sun.COM>
Subject: Re: [PATCH] quota: additional range checks and mem_dqblk updates	to handle 64-bit limits
Date: Sat, 08 Mar 2008 03:56:17 +0300	[thread overview]
Message-ID: <200803080356.18583.andrew.perepechko@sun.com> (raw)
In-Reply-To: <20080307160043.GA16967@duck.suse.cz>

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).

Do you feel it might take considerably larger efforts to update quotatools
to 64-bit version compared to 32-bit version with variable quota unit size?

I'd like to start working on a kernel patch implementing 64-bit limits provided 
you approve the approach.

Of course, it seems to be a good idea to have different file names for each version,
not only for each format (where format and version have the same meaning
as in quota kernel modules). The drawback is that then journalling quota users 
need to know the exact quota file version, not only format...

Thank you for you kind response.
Andrew.


On Friday 07 March 2008 19:00:43 Jan Kara wrote:
>   Great, thanks. The patch is fine. Yesterday evening I got an idea, how to
> solve your problem with too low limits even easier. What we could do is to
> introduce a "block-limit-scale" and "inode-limit-scale" parameter to the
> quota info and we keep the rest of the file format the same. Now, the meaning
> of this parameter would simply be a unit in which space and inode limits
> are specified. When you have a filesystem where you'd like to set quotas
> over 4 TB, you probably don't want to specify limits with 1KB precision
> anyway... So you can just set scale to 1MB or even 16MB (giving you maximal
> limit of 64 PB) and 10000 files or so. This has two advantages - only a few
> trivial modifications to current kernel code, no change in quota file space
> usage. We could then provide a way to set this scale via setquota / edquota
> (which would have to convert the whole file but that should be no big deal).
>   What do you think about such solution? Would it fit your needs? Sorry,
> that I haven't through of this solution earlier...
> 								Honza

  parent reply	other threads:[~2008-03-08  0:54 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 [this message]
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
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=200803080356.18583.andrew.perepechko@sun.com \
    --to=andrew.perepechko@sun.com \
    --cc=Johann.Lombardi@Sun.COM \
    --cc=Zhiyong.Tian@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).