public inbox for linux-xfs@vger.kernel.org
 help / color / mirror / Atom feed
From: Dave Chinner <david@fromorbit.com>
To: Eric Sandeen <sandeen@sandeen.net>
Cc: Kasparek Tomas <kasparek@fit.vutbr.cz>,
	"Michael L. Semon" <mlsemon35@gmail.com>,
	xfs@oss.sgi.com
Subject: Re: How to use increased number of ACL entries?
Date: Tue, 5 Nov 2013 07:53:28 +1100	[thread overview]
Message-ID: <20131104205328.GP6188@dastard> (raw)
In-Reply-To: <5277AE19.0@sandeen.net>

On Mon, Nov 04, 2013 at 08:24:25AM -0600, Eric Sandeen wrote:
> On 11/3/13, 10:18 PM, Michael L. Semon wrote:
> > On 11/03/2013 09:37 PM, Eric Sandeen wrote:
> >> On 11/3/13, 6:39 PM, Dave Chinner wrote:
> >>> On Sun, Nov 03, 2013 at 09:17:04AM +0100, Kasparek Tomas wrote:
> >>>> Hello,
> >>>>
> >>>> I'm trying to get more then 25 ACLs entries to work according to
> >>>> http://oss.sgi.com/pipermail/xfs/2013-May/026544.html . I'm running 3.10.x
> >>>> kernel which seems to contain these changes. I understand, that this is
> >>>> on-disk format change, so I expect to need new xfsprogs too. I tried the
> >>>> version from CentOS 6.4 (3.1.1) and one from git repo (
> >>>> git://oss.sgi.com/xfs/cmds/xfsprogs), but still it fails to create more then
> >>>> 25 ACL entries (21 user defined). Is there something I'm still missing?
> >>>
> >>> You haven't told mkfs to change the on disk format to enable more
> >>> than 25 ACLs. Only the version from git will do it, and your CentOS
> >>> kernel will not support it.
> >>
> >> but the 3.10.x kernel you're running will IIRC; use "-m crc=1" on the mkfs.xfs
> >> commandline from a git mkfs.xfs.
> >>
> >> -Eric
> >>  
> >>> Cheers,
> >>>
> >>> Dave.
> > 
> > Y'know, Eric, your best suggestions are always made when I'm working on a 
> > non-test PC that I don't really want to touch ;-)  But anyway, (i686 Pentium 
> > 4, kernel 3.10.17)...
> > 
> > git xfsprogs will make the filesystem in question:
> > 
> > root@bpserver:/storage/devel/git-xfsprogs# mkfs/mkfs.xfs /dev/sdb3
> > mkfs.xfs: /dev/sdb3 appears to contain an existing filesystem (swap).
> > mkfs.xfs: Use the -f option to force overwrite.
> > root@bpserver:/storage/devel/git-xfsprogs# mkfs/mkfs.xfs -f -m crc=1 /dev/sdb3
> > meta-data=/dev/sdb3              isize=512    agcount=4, agsize=65536 blks
> >          =                       sectsz=512   attr=2, projid32bit=1
> >          =                       crc=1
> > data     =                       bsize=4096   blocks=262144, imaxpct=25
> >          =                       sunit=0      swidth=0 blks
> > naming   =version 2              bsize=4096   ascii-ci=0 ftype=1
> > log      =internal log           bsize=4096   blocks=12800, version=2
> >          =                       sectsz=512   sunit=0 blks, lazy-count=1
> > realtime =none                   extsz=4096   blocks=0, rtextents=0
> > 
> > However, it should be dirent (ftype=1 in the above output) that keeps a 
> > vanilla 3.10.17 kernel from mounting the resulting filesystem:
> 
> I'm sorry, you are right - it hit kernel v3.11:
> 
> 5c87d4bc1a86bd6e6754ac3d6e111d776ddcfe57 xfs: increase number of ACL entries for V5 superblocks
> 
> $ git describe --contains 5c87d4bc1a86bd6e6754ac3d6e111d776ddcfe57
> v3.11-rc1~18^2~41

And a current git mkfs for -m crc=1 sets ftype which didn't hit
until:

$ glo -n 1 0cb97766
0cb9776 xfs: Add read-only support for dirent filetype field
$ git describe --contains 0cb97766
xfs-for-linus-v3.12-rc1~19
$

So a 3.12 kernel will be required to use it with a mkfs from the git
tree.

Cheers,

Dave.
-- 
Dave Chinner
david@fromorbit.com

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

      parent reply	other threads:[~2013-11-04 20:53 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-03  8:17 How to use increased number of ACL entries? Kasparek Tomas
2013-11-04  0:39 ` Dave Chinner
2013-11-04  2:37   ` Eric Sandeen
2013-11-04  4:18     ` Michael L. Semon
2013-11-04  4:35       ` Jeff Liu
2013-11-04  5:52         ` Michael L. Semon
2013-11-04 15:05           ` Kasparek Tomas
2013-11-04 14:24       ` Eric Sandeen
2013-11-04 15:10         ` Kasparek Tomas
2013-11-04 20:46           ` Dave Chinner
2013-11-05 22:03             ` Michael L. Semon
2013-11-05 22:46               ` Ben Myers
2013-11-04 20:53         ` Dave Chinner [this message]

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=20131104205328.GP6188@dastard \
    --to=david@fromorbit.com \
    --cc=kasparek@fit.vutbr.cz \
    --cc=mlsemon35@gmail.com \
    --cc=sandeen@sandeen.net \
    --cc=xfs@oss.sgi.com \
    /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