linux-kernel - Re: [PATCH] quota: Split out quota

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 4 Jun 2008 16:51:30 +0200
From:	Jan Kara <jack@...e.cz>
To:	Adrian Bunk <bunk@...nel.org>
Cc:	Andrew Morton <akpm@...ux-foundation.org>,
	LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] quota: Split out quota_struct.h out of quota.h

On Wed 04-06-08 16:56:24, Adrian Bunk wrote:
> On Mon, Jun 02, 2008 at 07:11:10PM +0200, Jan Kara wrote:
> > Split some structure definitions from quota.h to quota_struct.h so that they
> > can be separately included in fs.h. This makes it possible to declare some
> > useful inline functions in quota.h because it can now include fs.h.
> > 
> > Signed-off-by: Jan Kara <jack@...e.cz>
> > ---
> >  include/linux/fs.h       |    2 +-
> >  include/linux/quota.h    |  193 +---------------------------------------------
> >  include/linux/quotaops.h |    1 +
> >  include/linux/syscalls.h |    2 +-
> >  4 files changed, 5 insertions(+), 193 deletions(-)
> >...
> 
> Missing "git-add include/linux/quota_struct.h" ?
  Probably yes, I already noticed this and mailed Andrew a patch with this
file included.

> > --- a/include/linux/quota.h
> > +++ b/include/linux/quota.h
> > @@ -37,13 +37,11 @@
> >  
> >  #include <linux/errno.h>
> >  #include <linux/types.h>
> > +#include <linux/quota_struct.h>
> >...
> 
> This breaks our userspace headers.
> 
> Please always run "make headers_check" after touching #include's
> under include/
  I didn't know this existed :) I'll use it in future. Thanks. The question
is how to fix this. Probably the best would be if didn't have to export new
header file, wouldn't it? We can split the headers into quota.h containing
structures and all other userspace stuff and quota_func.h carrying other
kernel-internal stuff like inline functions etc. Oh well...

									Honza
-- 
Jan Kara <jack@...e.cz>
SUSE Labs, CR
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/