linux-fsdevel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] fs/Makefile: Always inspect exofs/
@ 2011-10-24 23:55 Boaz Harrosh
       [not found] ` <4EA5FAD7.4020806-C4P08NqkoRlBDgjK7y7TUQ@public.gmane.org>
  0 siblings, 1 reply; 3+ messages in thread
From: Boaz Harrosh @ 2011-10-24 23:55 UTC (permalink / raw)
  To: Michal Marek, Al Viro, linux-fsdevel, NFS list, open-osd


fs/exofs directory has multiple targets now, of which the
ore.ko will be needed by the pnfs-objects-layout-driver
(fs/nfs/objlayout).

As suggested by: Michal Marek <mmarek@suse.cz>  convert
inclusion of exofs/ from obj-$(CONFIG_EXOFS_FS) => obj-$(y).
So ORE can be selected also from fs/nfs/Kconfig

CC: Michal Marek <mmarek@suse.cz>
CC: Al Viro <viro@ZenIV.linux.org.uk>
Signed-off-by: Boaz Harrosh <bharrosh@panasas.com>
---
 fs/Makefile |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/fs/Makefile b/fs/Makefile
index afc1096..5c30a13 100644
--- a/fs/Makefile
+++ b/fs/Makefile
@@ -120,6 +120,6 @@ obj-$(CONFIG_DEBUG_FS)		+= debugfs/
 obj-$(CONFIG_OCFS2_FS)		+= ocfs2/
 obj-$(CONFIG_BTRFS_FS)		+= btrfs/
 obj-$(CONFIG_GFS2_FS)           += gfs2/
-obj-$(CONFIG_EXOFS_FS)          += exofs/
+obj-$(y)          		+= exofs/ # Multiple mods, used by nfs/objlayout
 obj-$(CONFIG_CEPH_FS)		+= ceph/
 obj-$(CONFIG_PSTORE)		+= pstore/
-- 
1.7.6.4


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] fs/Makefile: Always inspect exofs/
       [not found] ` <4EA5FAD7.4020806-C4P08NqkoRlBDgjK7y7TUQ@public.gmane.org>
@ 2011-10-27 10:54   ` Michal Marek
       [not found]     ` <4EA93868.8070404-AlSwsSmVLrQ@public.gmane.org>
  0 siblings, 1 reply; 3+ messages in thread
From: Michal Marek @ 2011-10-27 10:54 UTC (permalink / raw)
  To: Boaz Harrosh; +Cc: Al Viro, linux-fsdevel, NFS list, open-osd

On 25.10.2011 01:55, Boaz Harrosh wrote:
> 
> fs/exofs directory has multiple targets now, of which the
> ore.ko will be needed by the pnfs-objects-layout-driver
> (fs/nfs/objlayout).
> 
> As suggested by: Michal Marek <mmarek-AlSwsSmVLrQ@public.gmane.org>  convert
> inclusion of exofs/ from obj-$(CONFIG_EXOFS_FS) => obj-$(y).

I suggested obj-y verbatim, at least this is what I meant. obj-$(y)
expands to "obj-" and will do nothing.

Michal
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] fs/Makefile: Always inspect exofs/
       [not found]     ` <4EA93868.8070404-AlSwsSmVLrQ@public.gmane.org>
@ 2011-10-27 23:25       ` Boaz Harrosh
  0 siblings, 0 replies; 3+ messages in thread
From: Boaz Harrosh @ 2011-10-27 23:25 UTC (permalink / raw)
  To: Michal Marek; +Cc: Al Viro, linux-fsdevel, NFS list, open-osd

On 10/27/2011 03:54 AM, Michal Marek wrote:
> On 25.10.2011 01:55, Boaz Harrosh wrote:
>>
>> fs/exofs directory has multiple targets now, of which the
>> ore.ko will be needed by the pnfs-objects-layout-driver
>> (fs/nfs/objlayout).
>>
>> As suggested by: Michal Marek <mmarek-AlSwsSmVLrQ@public.gmane.org>  convert
>> inclusion of exofs/ from obj-$(CONFIG_EXOFS_FS) => obj-$(y).
> 
> I suggested obj-y verbatim, at least this is what I meant. obj-$(y)
> expands to "obj-" and will do nothing.
> 
> Michal

Thanks Michal.

Yes that was a stupid, stupid TYPO bug I did and I'm so embarrassed
I already sent a fix to Linus, he pulled it.

Dear, you have eyes of an hawk, how did you see it? I stared at it for
15 minutes and did not get it. 

Boaz
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2011-10-27 23:25 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-10-24 23:55 [PATCH] fs/Makefile: Always inspect exofs/ Boaz Harrosh
     [not found] ` <4EA5FAD7.4020806-C4P08NqkoRlBDgjK7y7TUQ@public.gmane.org>
2011-10-27 10:54   ` Michal Marek
     [not found]     ` <4EA93868.8070404-AlSwsSmVLrQ@public.gmane.org>
2011-10-27 23:25       ` Boaz Harrosh

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).