From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from ipmail07.adl2.internode.on.net ([150.101.137.131]:5695 "EHLO ipmail07.adl2.internode.on.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751608AbeA0Dwq (ORCPT ); Fri, 26 Jan 2018 22:52:46 -0500 Date: Thu, 25 Jan 2018 11:02:15 +1100 From: Dave Chinner Subject: Re: [PATCH v2 4/8] metaperf: Include linux/param.h explicitly for HZ macro Message-ID: <20180125000215.qlbpu5i37luul6cl@destitution> References: <1516636639-9107-1-git-send-email-rostislav@tuxera.com> <1516636639-9107-5-git-send-email-rostislav@tuxera.com> <20180122202406.GD16776@infradead.org> <20180124035859.GA30514@eguan.usersys.redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20180124035859.GA30514@eguan.usersys.redhat.com> Sender: fstests-owner@vger.kernel.org To: Eryu Guan Cc: Christoph Hellwig , Rostislav Skudnov , fstests@vger.kernel.org List-ID: On Wed, Jan 24, 2018 at 11:58:59AM +0800, Eryu Guan wrote: > On Mon, Jan 22, 2018 at 12:24:06PM -0800, Christoph Hellwig wrote: > > On Mon, Jan 22, 2018 at 03:57:15PM +0000, Rostislav Skudnov wrote: > > > Glibc includes linux/param.h when we include sys/param.h, whereas musl > > > libc does not do that. HZ is a Linux-specific macro, therefore include > > > the header file that defines it explicitly. > > > > > > Signed-off-by: Rostislav Skudnov > > > > Using HZ in userspace doesn't make any sense at all. The kernel HZ > > (as in the granulairy for jiffies) can vary between architectures > > and even configurations. I guess it wants a hard coded 100 here, but > > someone will have to do a detailed analysis. > > No one is using metaperf now, all the old benchmark infrastructure was > removed by commit b020416d51ff ("xfstests: remove bench infrastructure") > > I think we can remove src/metaperf.c too. I use metaperf (and dirperf) quite regularly when testing my patches, so I'd really like them to remin available on all my test machines that have fstests installed.... Cheers, Dave. -- Dave Chinner david@fromorbit.com