lkml.org 
[lkml]   [2000]   [Nov]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Date
SubjectRe: PATCH 2.4.0.11.3: sysctl.h fixes
On Sunday November 12, jgarzik@mandrakesoft.com wrote:
> Rasmus Andersen wrote:
> > I tried to include <linux/types.h> in md.c and had to include
> > <linux/blkdev.h> also. Otherwise I got the following:
>
> Here is the solution I prefer... md builds fine with this, core kernel builds fine with this, and
> I'm about 3/4 of the way through a "build everything" build with this.
>
> I tried to avoid including fs.h, but I do prefer updating sysctl.h, because it fixes potential
> breakage similar to md's as well.
>
> Jeff
>

The declaration of:

struct file;

in sysctl.h is a bit counter intuitive isn't it?

I avoided the problem with:

--- md.c 2000/11/12 23:00:49 1.1
+++ md.c 2000/11/12 23:00:53
@@ -30,8 +30,8 @@

#include <linux/module.h>
#include <linux/config.h>
-#include <linux/sysctl.h>
#include <linux/raid/md.h>
+#include <linux/sysctl.h>
#include <linux/raid/xor.h>
#include <linux/devfs_fs_kernel.h>

It is certainly arguable that a better fix is to add some extra
includes to sysctl.h, but the "struct file;" bothers me.

NeilBrown


>
>
>
> Index: include/linux/sysctl.h
> ===================================================================
> RCS file: /cvsroot/gkernel/linux_2_4/include/linux/sysctl.h,v
> retrieving revision 1.1.1.8
> diff -u -r1.1.1.8 sysctl.h
> --- include/linux/sysctl.h 2000/10/31 21:19:40 1.1.1.8
> +++ include/linux/sysctl.h 2000/11/12 14:28:04
> @@ -24,7 +24,11 @@
> #ifndef _LINUX_SYSCTL_H
> #define _LINUX_SYSCTL_H
>
> +#include <linux/kernel.h>
> +#include <linux/types.h>
> #include <linux/list.h>
> +
> +struct file;
>
> #define CTL_MAXNAME 10
>
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> Please read the FAQ at http://www.tux.org/lkml/
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 12:45    [W:0.170 / U:0.012 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site