public inbox for linux-xfs@vger.kernel.org
 help / color / mirror / Atom feed
* xfs module insert fails with "xfs: Unknown symbol exportfs_decode_fh (err 0)"
@ 2013-04-26 17:24 Chandra Seetharaman
  2013-04-26 18:02 ` Eric Sandeen
  2013-04-26 18:16 ` Ben Myers
  0 siblings, 2 replies; 5+ messages in thread
From: Chandra Seetharaman @ 2013-04-26 17:24 UTC (permalink / raw)
  To: XFS mailing list

Hello,

When I tried to insert xfs module in my test machine it failed with 

	xfs: Unknown symbol exportfs_decode_fh (err 0).

Found out that I did not have CONFIG_EXPORTFS enabled in my kernel
config. 

All .*by_handle code in XFS is open coded. Wondering if they should be
under #ifdef CONFIG_EXPORTFS ?

I can create a patch, but wanted to find out if it is indeed needed :)

Regards,

Chandra




_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

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

* Re: xfs module insert fails with "xfs: Unknown symbol exportfs_decode_fh (err 0)"
  2013-04-26 17:24 xfs module insert fails with "xfs: Unknown symbol exportfs_decode_fh (err 0)" Chandra Seetharaman
@ 2013-04-26 18:02 ` Eric Sandeen
  2013-04-26 18:29   ` Chandra Seetharaman
  2013-04-26 18:16 ` Ben Myers
  1 sibling, 1 reply; 5+ messages in thread
From: Eric Sandeen @ 2013-04-26 18:02 UTC (permalink / raw)
  To: sekharan; +Cc: XFS mailing list

On 4/26/13 12:24 PM, Chandra Seetharaman wrote:
> Hello,
> 
> When I tried to insert xfs module in my test machine it failed with 
> 
> 	xfs: Unknown symbol exportfs_decode_fh (err 0).

so don't use insmod, use modprobe :)

> Found out that I did not have CONFIG_EXPORTFS enabled in my kernel
> config. 

Oh, hum. But fs/xfs/Kconfig:

config XFS_FS
        tristate "XFS filesystem support"
        depends on BLOCK
        select EXPORTFS

it should be turned on automatically.  That seems to be the problem,
why wasn't it for your case?

> All .*by_handle code in XFS is open coded. Wondering if they should be
> under #ifdef CONFIG_EXPORTFS ?

CONFIG_EXPORTFS is supposed to be on if xfs is selected, unless something
changed recently.

-Eric

> I can create a patch, but wanted to find out if it is indeed needed :)
> 
> Regards,
> 
> Chandra
> 
> 
> 
> 
> _______________________________________________
> xfs mailing list
> xfs@oss.sgi.com
> http://oss.sgi.com/mailman/listinfo/xfs
> 

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

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

* Re: xfs module insert fails with "xfs: Unknown symbol exportfs_decode_fh (err 0)"
  2013-04-26 17:24 xfs module insert fails with "xfs: Unknown symbol exportfs_decode_fh (err 0)" Chandra Seetharaman
  2013-04-26 18:02 ` Eric Sandeen
@ 2013-04-26 18:16 ` Ben Myers
  2013-04-26 21:46   ` Dave Chinner
  1 sibling, 1 reply; 5+ messages in thread
From: Ben Myers @ 2013-04-26 18:16 UTC (permalink / raw)
  To: Chandra Seetharaman; +Cc: XFS mailing list

Hey Chandra,

On Fri, Apr 26, 2013 at 12:24:09PM -0500, Chandra Seetharaman wrote:
> When I tried to insert xfs module in my test machine it failed with 
> 
> 	xfs: Unknown symbol exportfs_decode_fh (err 0).
> 
> Found out that I did not have CONFIG_EXPORTFS enabled in my kernel
> config. 

Most people are running with NFS enabled in their kernel config, I think.

> All .*by_handle code in XFS is open coded. Wondering if they should be
> under #ifdef CONFIG_EXPORTFS ?
>
> I can create a patch, but wanted to find out if it is indeed needed :)

foo_by_handle
  xfs_handlereq_to_dentry
    xfs_handle_to_dentry
      exportfs_decode_fh

xfsdump does open by handle, so making that conditional on CONFIG_EXPORTFS will
break things.  I think would be better to just make CONFIG_XFS depend upon
CONFIG_EXPORTFS in fs/xfs/Kconfig.  Looks like we'll have the same problem with
libcrc32c.

Regards,
	Ben

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

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

* Re: xfs module insert fails with "xfs: Unknown symbol exportfs_decode_fh (err 0)"
  2013-04-26 18:02 ` Eric Sandeen
@ 2013-04-26 18:29   ` Chandra Seetharaman
  0 siblings, 0 replies; 5+ messages in thread
From: Chandra Seetharaman @ 2013-04-26 18:29 UTC (permalink / raw)
  To: Eric Sandeen; +Cc: XFS mailing list

On Fri, 2013-04-26 at 13:02 -0500, Eric Sandeen wrote:
> On 4/26/13 12:24 PM, Chandra Seetharaman wrote:
> > Hello,
> > 
> > When I tried to insert xfs module in my test machine it failed with 
> > 
> > 	xfs: Unknown symbol exportfs_decode_fh (err 0).
> 
> so don't use insmod, use modprobe :)
> 

yup. It is the problem. Config seems to be fine.

false alarm.
> > Found out that I did not have CONFIG_EXPORTFS enabled in my kernel
> > config. 
> 
> Oh, hum. But fs/xfs/Kconfig:
> 
> config XFS_FS
>         tristate "XFS filesystem support"
>         depends on BLOCK
>         select EXPORTFS
> 
> it should be turned on automatically.  That seems to be the problem,
> why wasn't it for your case?
> 
> > All .*by_handle code in XFS is open coded. Wondering if they should be
> > under #ifdef CONFIG_EXPORTFS ?
> 
> CONFIG_EXPORTFS is supposed to be on if xfs is selected, unless something
> changed recently.
> 
> -Eric
> 
> > I can create a patch, but wanted to find out if it is indeed needed :)
> > 
> > Regards,
> > 
> > Chandra
> > 
> > 
> > 
> > 
> > _______________________________________________
> > xfs mailing list
> > xfs@oss.sgi.com
> > http://oss.sgi.com/mailman/listinfo/xfs
> > 
> 
> _______________________________________________
> xfs mailing list
> xfs@oss.sgi.com
> http://oss.sgi.com/mailman/listinfo/xfs
> 


_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

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

* Re: xfs module insert fails with "xfs: Unknown symbol exportfs_decode_fh (err 0)"
  2013-04-26 18:16 ` Ben Myers
@ 2013-04-26 21:46   ` Dave Chinner
  0 siblings, 0 replies; 5+ messages in thread
From: Dave Chinner @ 2013-04-26 21:46 UTC (permalink / raw)
  To: Ben Myers; +Cc: Chandra Seetharaman, XFS mailing list

On Fri, Apr 26, 2013 at 01:16:24PM -0500, Ben Myers wrote:
> Hey Chandra,
> 
> On Fri, Apr 26, 2013 at 12:24:09PM -0500, Chandra Seetharaman wrote:
> > When I tried to insert xfs module in my test machine it failed with 
> > 
> > 	xfs: Unknown symbol exportfs_decode_fh (err 0).
> > 
> > Found out that I did not have CONFIG_EXPORTFS enabled in my kernel
> > config. 
> 
> Most people are running with NFS enabled in their kernel config, I think.
> 
> > All .*by_handle code in XFS is open coded. Wondering if they should be
> > under #ifdef CONFIG_EXPORTFS ?
> >
> > I can create a patch, but wanted to find out if it is indeed needed :)
> 
> foo_by_handle
>   xfs_handlereq_to_dentry
>     xfs_handle_to_dentry
>       exportfs_decode_fh
> 
> xfsdump does open by handle, so making that conditional on CONFIG_EXPORTFS will
> break things.  I think would be better to just make CONFIG_XFS depend upon
> CONFIG_EXPORTFS in fs/xfs/Kconfig.  Looks like we'll have the same problem with
> libcrc32c.

AFAIK, "select" isthe right keyword for these dependencies. They are
like saying "depends on", but instead of having the XFS config options
only appear in the menus if CONFIG_EXPORTFS is selected, it will
select CONFIG_EXPORTFS=y when CONFIG_XFS_FS=y....

So I don't think there are any problems here...

Cheers,

Dave.
-- 
Dave Chinner
david@fromorbit.com

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

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

end of thread, other threads:[~2013-04-26 21:46 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-04-26 17:24 xfs module insert fails with "xfs: Unknown symbol exportfs_decode_fh (err 0)" Chandra Seetharaman
2013-04-26 18:02 ` Eric Sandeen
2013-04-26 18:29   ` Chandra Seetharaman
2013-04-26 18:16 ` Ben Myers
2013-04-26 21:46   ` Dave Chinner

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox