From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx1.redhat.com ([209.132.183.28]:42026 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932320AbdEDFAI (ORCPT ); Thu, 4 May 2017 01:00:08 -0400 Date: Thu, 4 May 2017 13:00:06 +0800 From: Eryu Guan Subject: Re: [PATCH] fstests: attr: add support for cephfs Message-ID: <20170504050006.GI7250@eguan.usersys.redhat.com> References: <20170503105413.8314-1-lhenriques@suse.com> <20170504032340.GH7250@eguan.usersys.redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: fstests-owner@vger.kernel.org To: "Yan, Zheng" Cc: Luis Henriques , fstests@vger.kernel.org, ceph-devel List-ID: On Thu, May 04, 2017 at 11:44:46AM +0800, Yan, Zheng wrote: > On Thu, May 4, 2017 at 11:23 AM, Eryu Guan 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 > > > > 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? Eryu