Re: [PATCH] fstests: attr: add support for cephfs

[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]



[cc'ed fstests list]

On Thu, May 04, 2017 at 03:49:27PM +0800, Yan, Zheng wrote:
> On Thu, May 4, 2017 at 1:00 PM, Eryu Guan <eguan@xxxxxxxxxx> wrote:
> > On Thu, May 04, 2017 at 11:44:46AM +0800, Yan, Zheng wrote:
> >> On Thu, May 4, 2017 at 11:23 AM, Eryu Guan <eguan@xxxxxxxxxx> wrote:
> >> > On Wed, May 03, 2017 at 11:54:13AM +0100, Luis Henriques wrote:
> >> >> Block size for cephfs is 4M, which makes generic/020 test fail as the
> >> >> value for MAX_ATTRS and MAX_ATTRVAL_SIZE will be too high.  Restrict these
> >> >> two variables to sane values for this FSTYP.
> >> >>
> >> >> Signed-off-by: Luis Henriques <lhenriques@xxxxxxxx>
> >> >
> >> > Thanks for the patch! I'd like some reviews from ceph developers on the
> >> > ceph MAX_ATTRS and MAX_ATTRVAL_SIZE numbers.
> >> >
> >>
> >> Currently we don't have any limitation on single xattr size and number
> >> of xattrs. But single xattr size is limited to 65536 by kernel.
> >
> > Thanks! Can I take it as a Reviewed-by tag?
> 
> 
> yes

Thanks!

Eryu
--
To unsubscribe from this list: send the line "unsubscribe fstests" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Filesystems Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux