lkml.org 
[lkml]   [2008]   [Aug]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
SubjectRe: next-20080808 fs/nfsctl.c build error
Date
James Morris <jmorris@namei.org> wrote:

> In any case, it seems that linux/cred.h should include linux/sched.h to
> provide a full definition of 'struct task_struct', so that 'current' can
> be dereferenced (e.g. in current_cred()).

Not good. That makes sched.h and cred.h recursively dependent. sched.h
really needs splitting up, and the include dependencies need sorting.

However, for the moment I recommend the attached patch.

David
---
From: David Howells <dhowells@redhat.com>

CRED: Fix inclusion problems

Under certain configurations, some .c files obtain linux/sched.h and/or
linux/cred.h through other include files, but not under all configurations.

A couple of places (one in Coda, one in nfsd) need extra includes.

Signed-off-by: David Howells <dhowells@redhat.com>
---

fs/coda/file.c | 1 +
fs/nfsctl.c | 1 +
2 files changed, 2 insertions(+), 0 deletions(-)


diff --git a/fs/coda/file.c b/fs/coda/file.c
index 5a87699..9d0d703 100644
--- a/fs/coda/file.c
+++ b/fs/coda/file.c
@@ -16,6 +16,7 @@
#include <linux/errno.h>
#include <linux/smp_lock.h>
#include <linux/string.h>
+#include <linux/cred.h>
#include <asm/uaccess.h>

#include <linux/coda.h>
diff --git a/fs/nfsctl.c b/fs/nfsctl.c
index cc4ef26..358e469 100644
--- a/fs/nfsctl.c
+++ b/fs/nfsctl.c
@@ -14,6 +14,7 @@
#include <linux/namei.h>
#include <linux/mount.h>
#include <linux/syscalls.h>
+#include <linux/sched.h>
#include <asm/uaccess.h>

/*


\
 
 \ /
  Last update: 2008-08-09 01:01    [W:0.034 / U:1.232 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site