Signed-off-by: Jeff Layton <[email protected]>
Cc: Thomas Haynes <[email protected]>
---
fs/nfs/internal.h | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/fs/nfs/internal.h b/fs/nfs/internal.h
index efaa31c70fbe..ffef459f0b4f 100644
--- a/fs/nfs/internal.h
+++ b/fs/nfs/internal.h
@@ -1,6 +1,8 @@
/*
* NFS internal definitions
*/
+#ifndef __FS_NFS_INTERNAL_H
+#define __FS_NFS_INTERNAL_H
#include "nfs4_fs.h"
#include <linux/mount.h>
@@ -629,3 +631,5 @@ static inline u32 nfs_fhandle_hash(const struct nfs_fh *fh)
return 0;
}
#endif
+
+#endif /* __FS_NFS_INTERNAL_H */
--
2.1.0
On Fri, 12 Dec 2014 09:50:18 -0500
Jeff Layton <[email protected]> wrote:
> Signed-off-by: Jeff Layton <[email protected]>
> Cc: Thomas Haynes <[email protected]>
> ---
> fs/nfs/internal.h | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/fs/nfs/internal.h b/fs/nfs/internal.h
> index efaa31c70fbe..ffef459f0b4f 100644
> --- a/fs/nfs/internal.h
> +++ b/fs/nfs/internal.h
> @@ -1,6 +1,8 @@
> /*
> * NFS internal definitions
> */
> +#ifndef __FS_NFS_INTERNAL_H
Gyah...sorry for the whitespace typo above. Want me to resend with that
tab turned into a space?
> +#define __FS_NFS_INTERNAL_H
>
> #include "nfs4_fs.h"
> #include <linux/mount.h>
> @@ -629,3 +631,5 @@ static inline u32 nfs_fhandle_hash(const struct nfs_fh *fh)
> return 0;
> }
> #endif
> +
> +#endif /* __FS_NFS_INTERNAL_H */
--
Jeff Layton <[email protected]>
On Fri, Dec 12, 2014 at 9:58 AM, Jeff Layton <[email protected]> wrote:
> On Fri, 12 Dec 2014 09:50:18 -0500
> Jeff Layton <[email protected]> wrote:
>
>> Signed-off-by: Jeff Layton <[email protected]>
>> Cc: Thomas Haynes <[email protected]>
>> ---
>> fs/nfs/internal.h | 4 ++++
>> 1 file changed, 4 insertions(+)
>>
>> diff --git a/fs/nfs/internal.h b/fs/nfs/internal.h
>> index efaa31c70fbe..ffef459f0b4f 100644
>> --- a/fs/nfs/internal.h
>> +++ b/fs/nfs/internal.h
>> @@ -1,6 +1,8 @@
>> /*
>> * NFS internal definitions
>> */
>> +#ifndef __FS_NFS_INTERNAL_H
>
> Gyah...sorry for the whitespace typo above. Want me to resend with that
> tab turned into a space?
That's OK. I'll manage somehow :-)
>> +#define __FS_NFS_INTERNAL_H
>>
>> #include "nfs4_fs.h"
>> #include <linux/mount.h>
>> @@ -629,3 +631,5 @@ static inline u32 nfs_fhandle_hash(const struct nfs_fh *fh)
>> return 0;
>> }
>> #endif
>> +
>> +#endif /* __FS_NFS_INTERNAL_H */
>
>
> --
> Jeff Layton <[email protected]>
--
Trond Myklebust
Linux NFS client maintainer, PrimaryData
[email protected]
On Fri, Dec 12, 2014 at 10:50 AM, Trond Myklebust
<[email protected]> wrote:
> On Fri, Dec 12, 2014 at 9:58 AM, Jeff Layton <[email protected]> wrote:
>> On Fri, 12 Dec 2014 09:50:18 -0500
>> Jeff Layton <[email protected]> wrote:
>>
>>> Signed-off-by: Jeff Layton <[email protected]>
>>> Cc: Thomas Haynes <[email protected]>
>>> ---
>>> fs/nfs/internal.h | 4 ++++
>>> 1 file changed, 4 insertions(+)
>>>
>>> diff --git a/fs/nfs/internal.h b/fs/nfs/internal.h
>>> index efaa31c70fbe..ffef459f0b4f 100644
>>> --- a/fs/nfs/internal.h
>>> +++ b/fs/nfs/internal.h
>>> @@ -1,6 +1,8 @@
>>> /*
>>> * NFS internal definitions
>>> */
>>> +#ifndef __FS_NFS_INTERNAL_H
>>
>> Gyah...sorry for the whitespace typo above. Want me to resend with that
>> tab turned into a space?
>
> That's OK. I'll manage somehow :-)
>
>>> +#define __FS_NFS_INTERNAL_H
>>>
>>> #include "nfs4_fs.h"
>>> #include <linux/mount.h>
>>> @@ -629,3 +631,5 @@ static inline u32 nfs_fhandle_hash(const struct nfs_fh *fh)
>>> return 0;
>>> }
>>> #endif
>>> +
>>> +#endif /* __FS_NFS_INTERNAL_H */
>>
>>
>> --
>> Jeff Layton <[email protected]>
>
So, having rethought this a little, I think I'd prefer to keep the
current status. Right now, we catch all instances of someone
incorrectly including this file in another header by virtue of the
fact that the duplicate include will cause a compile error.
--
Trond Myklebust
Linux NFS client maintainer, PrimaryData
[email protected]
On Fri, 12 Dec 2014 13:58:10 -0500
Trond Myklebust <[email protected]> wrote:
> On Fri, Dec 12, 2014 at 10:50 AM, Trond Myklebust
> <[email protected]> wrote:
> > On Fri, Dec 12, 2014 at 9:58 AM, Jeff Layton <[email protected]> wrote:
> >> On Fri, 12 Dec 2014 09:50:18 -0500
> >> Jeff Layton <[email protected]> wrote:
> >>
> >>> Signed-off-by: Jeff Layton <[email protected]>
> >>> Cc: Thomas Haynes <[email protected]>
> >>> ---
> >>> fs/nfs/internal.h | 4 ++++
> >>> 1 file changed, 4 insertions(+)
> >>>
> >>> diff --git a/fs/nfs/internal.h b/fs/nfs/internal.h
> >>> index efaa31c70fbe..ffef459f0b4f 100644
> >>> --- a/fs/nfs/internal.h
> >>> +++ b/fs/nfs/internal.h
> >>> @@ -1,6 +1,8 @@
> >>> /*
> >>> * NFS internal definitions
> >>> */
> >>> +#ifndef __FS_NFS_INTERNAL_H
> >>
> >> Gyah...sorry for the whitespace typo above. Want me to resend with that
> >> tab turned into a space?
> >
> > That's OK. I'll manage somehow :-)
> >
> >>> +#define __FS_NFS_INTERNAL_H
> >>>
> >>> #include "nfs4_fs.h"
> >>> #include <linux/mount.h>
> >>> @@ -629,3 +631,5 @@ static inline u32 nfs_fhandle_hash(const struct nfs_fh *fh)
> >>> return 0;
> >>> }
> >>> #endif
> >>> +
> >>> +#endif /* __FS_NFS_INTERNAL_H */
> >>
> >>
> >> --
> >> Jeff Layton <[email protected]>
> >
>
> So, having rethought this a little, I think I'd prefer to keep the
> current status. Right now, we catch all instances of someone
> incorrectly including this file in another header by virtue of the
> fact that the duplicate include will cause a compile error.
>
Fair enough. The problem that we saw with it turns out to be a bug in a
different patch under development so it's not really required.
Thanks,
--
Jeff Layton <[email protected]>