From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jan Kara Subject: Re: [PATCH 11/14] quota: Move quotaio_v[12].h from include/linux/ to fs/ Date: Wed, 29 Oct 2008 04:21:07 +0100 Message-ID: <20081029032106.GM24732@duck.suse.cz> References: <1225109512758-git-send-email-jack@suse.cz> <12251095122803-git-send-email-jack@suse.cz> <12251095124165-git-send-email-jack@suse.cz> <1225109513735-git-send-email-jack@suse.cz> <12251095131095-git-send-email-jack@suse.cz> <12251095133646-git-send-email-jack@suse.cz> <12251095133281-git-send-email-jack@suse.cz> <12251095131431-git-send-email-jack@suse.cz> <1225109513299-git-send-email-jack@suse.cz> <20081028153209.d66428fe.akpm@linux-foundation.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Jan Kara , linux-fsdevel@vger.kernel.org To: Andrew Morton Return-path: Received: from styx.suse.cz ([82.119.242.94]:47594 "EHLO mail.suse.cz" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751690AbYJ2DVI (ORCPT ); Tue, 28 Oct 2008 23:21:08 -0400 Content-Disposition: inline In-Reply-To: <20081028153209.d66428fe.akpm@linux-foundation.org> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Tue 28-10-08 15:32:09, Andrew Morton wrote: > On Mon, 27 Oct 2008 13:11:49 +0100 > Jan Kara wrote: > > > Since these include files are used only by implementation of quota formats, > > there's no need to have them in include/linux/. > > akpm2:/usr/src/25> make headers_check > CHK include/linux/version.h > make[1]: `scripts/unifdef' is up to date. > make[2]: *** No rule to make target `/usr/src/devel/include/linux/quotaio_v1.h', needed by `/usr/src/devel/usr/include/linux/.install'. Stop. > > > If this header really does need to be exported to userspace then should > it not remain in include/linux/? I admit I've forgotten about the fact that it won't be exported to userspace after moving. These files contain definitions of on disk quota format. Whether we should export this to userspace is a question - filesystems seem to make the choice "no" lately. Quota tools have their own definitions anyway so they don't care. I don't know about anybody else who would care but ... Anybody has any opinion? > Meanwhile I'll do this: > > --- a/include/linux/Kbuild~quota-move-quotaio_vh-from-include-linux-to-fs-fix > +++ a/include/linux/Kbuild > @@ -134,8 +134,6 @@ header-y += posix_types.h > header-y += ppdev.h > header-y += prctl.h > header-y += qnxtypes.h > -header-y += quotaio_v1.h > -header-y += quotaio_v2.h > header-y += radeonfb.h > header-y += raw.h > header-y += resource.h > > so I can compile stuff. > > btw, please work out how this build error escaped detection and arrange > for it to not happen in the future? Thanks. Hmm, I should read SubmitChecklist (and actually do it) more often ;). Honza -- Jan Kara SUSE Labs, CR