public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* Show module version sysfs file for built-in modules?
@ 2009-07-24 19:08 Tomas Carnecky
  2009-07-29  5:23 ` Greg KH
  0 siblings, 1 reply; 5+ messages in thread
From: Tomas Carnecky @ 2009-07-24 19:08 UTC (permalink / raw)
  To: linux-kernel

Some userspace tools check /sys/modules/XYZ/version to determine  
whether the required module is available and the version compatible  
with the userspace tool. But that sysfs file is not shown when the  
module is built into the kernel (as opposed being a loadable module).  
Is there a reason for that behavior? Or is there another way for  
userspace to check the presence and version of a module?
The userspace tool in question here is the open-iscsi daemon (iscsid)  
which checks /sys/modules/scsi_transport_iscsi/version and refuses to  
start if that file doesn't exist.

thanks
tom


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

* Re: Show module version sysfs file for built-in modules?
  2009-07-24 19:08 Tomas Carnecky
@ 2009-07-29  5:23 ` Greg KH
  0 siblings, 0 replies; 5+ messages in thread
From: Greg KH @ 2009-07-29  5:23 UTC (permalink / raw)
  To: Tomas Carnecky; +Cc: linux-kernel

On Fri, Jul 24, 2009 at 09:08:51PM +0200, Tomas Carnecky wrote:
> Some userspace tools check /sys/modules/XYZ/version to determine  
> whether the required module is available and the version compatible  
> with the userspace tool. But that sysfs file is not shown when the  
> module is built into the kernel (as opposed being a loadable module).  
> Is there a reason for that behavior? Or is there another way for  
> userspace to check the presence and version of a module?
> The userspace tool in question here is the open-iscsi daemon (iscsid)  
> which checks /sys/modules/scsi_transport_iscsi/version and refuses to  
> start if that file doesn't exist.

A patch to fix this would probably be gladly accepted.

thanks,

greg k-h

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

* Re: Show module version sysfs file for built-in modules?
@ 2009-07-29 18:40 devzero
  2009-07-29 19:08 ` Alan Jenkins
  0 siblings, 1 reply; 5+ messages in thread
From: devzero @ 2009-07-29 18:40 UTC (permalink / raw)
  To: Tomas Carnecky; +Cc: linux-kernel

what about this ?

neoware:/sys/module/8139too # cat /sys/module/8139too/version
0.9.28

neoware:/sys/module/8139too # modinfo 8139too |grep "^version:"
version:        0.9.28

the latter should work completely without consulting the kernel for that information. 

regards
roland

>List:       linux-kernel
>Subject:    Show module version sysfs file for built-in modules?
>From:       Tomas Carnecky <tom () dbservice ! com>
>Date:       2009-07-24 19:08:51
>Message-ID: 5A77D882-B628-4105-85A3-ADE423BBD9F8 () dbservice ! com
>[Download message RAW]
>
>Some userspace tools check /sys/modules/XYZ/version to determine  
>whether the required module is available and the version compatible  
>with the userspace tool. But that sysfs file is not shown when the  
>module is built into the kernel (as opposed being a loadable module).  
>Is there a reason for that behavior? Or is there another way for  
>userspace to check the presence and version of a module?
>The userspace tool in question here is the open-iscsi daemon (iscsid)  
vwhich checks /sys/modules/scsi_transport_iscsi/version and refuses to  
>start if that file doesn't exist.
>
>thanks
>tom

________________________________________________________________
Neu: WEB.DE Doppel-FLAT mit Internet-Flatrate + Telefon-Flatrate
für nur 19,99 Euro/mtl.!* http://produkte.web.de/go/02/


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

* Re: Show module version sysfs file for built-in modules?
  2009-07-29 18:40 Show module version sysfs file for built-in modules? devzero
@ 2009-07-29 19:08 ` Alan Jenkins
  0 siblings, 0 replies; 5+ messages in thread
From: Alan Jenkins @ 2009-07-29 19:08 UTC (permalink / raw)
  To: devzero; +Cc: Tomas Carnecky, linux-kernel

On 7/29/09, devzero@web.de <devzero@web.de> wrote:
> what about this ?
>
> neoware:/sys/module/8139too # cat /sys/module/8139too/version
> 0.9.28
>
> neoware:/sys/module/8139too # modinfo 8139too |grep "^version:"
> version:        0.9.28
>
> the latter should work completely without consulting the kernel for that
> information.
>
> regards
> roland
>
>>List:       linux-kernel
>>Subject:    Show module version sysfs file for built-in modules?

modinfo clearly won't work for *built in* drivers.  Apparently
/sys/module/$MODULE/version doesn't work for built in modules either.
Hence this proposal.

Alan

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

* Re: Show module version sysfs file for built-in modules?
@ 2009-07-29 21:48 devzero
  0 siblings, 0 replies; 5+ messages in thread
From: devzero @ 2009-07-29 21:48 UTC (permalink / raw)
  To: Alan Jenkins; +Cc: linux-kernel, Tomas Carnecky

pardon - that was dumb.
must have read over that built-in.

indeed, this is a missing feature. 
there are even modules/drivers which donŽt spit any line in dmesg when they are initialized - builtin or not.
there is a LOT of different behaviour in module load/init and i`d welcome if module initialization messages including version information would follow a more standard way. many modules don`t even have a version.

regards
roland


> -----Ursprüngliche Nachricht-----
> Von: "Alan Jenkins" <sourcejedi.lkml@googlemail.com>
> Gesendet: 29.07.09 21:10:24
> An: devzero@web.de
> CC: Tomas Carnecky <tom@dbservice.com>, linux-kernel@vger.kernel.org
> Betreff: Re: Show module version sysfs file for built-in modules?


> On 7/29/09, devzero@web.de <devzero@web.de> wrote:
> > what about this ?
> >
> > neoware:/sys/module/8139too # cat /sys/module/8139too/version
> > 0.9.28
> >
> > neoware:/sys/module/8139too # modinfo 8139too |grep "^version:"
> > version:        0.9.28
> >
> > the latter should work completely without consulting the kernel for that
> > information.
> >
> > regards
> > roland
> >
> >>List:       linux-kernel
> >>Subject:    Show module version sysfs file for built-in modules?
> 
> modinfo clearly won't work for *built in* drivers.  Apparently
> /sys/module/$MODULE/version doesn't work for built in modules either.
> Hence this proposal.
> 
> Alan
> 


______________________________________________________
GRATIS für alle WEB.DE-Nutzer: Die maxdome Movie-FLAT!
Jetzt freischalten unter http://movieflat.web.de


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

end of thread, other threads:[~2009-07-29 21:48 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-07-29 18:40 Show module version sysfs file for built-in modules? devzero
2009-07-29 19:08 ` Alan Jenkins
  -- strict thread matches above, loose matches on Subject: below --
2009-07-29 21:48 devzero
2009-07-24 19:08 Tomas Carnecky
2009-07-29  5:23 ` Greg KH

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