Bug 113038 - Failure to load slamr module
Summary: Failure to load slamr module
Status: RESOLVED FIXED
: 113165 (view as bug list)
Alias: None
Product: SUSE LINUX 10.0
Classification: openSUSE
Component: Kernel (show other bugs)
Version: Beta 3
Hardware: PC All
: P5 - None : Major
Target Milestone: ---
Assignee: Marian Jancar
QA Contact: E-mail List
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-08-25 15:22 UTC by Takashi Iwai
Modified: 2005-09-09 15:51 UTC (History)
2 users (show)

See Also:
Found By: Other
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Takashi Iwai 2005-08-25 15:22:37 UTC
The module slamr can't be loaded:

slamr: Unknown symbol class_destroy
slamr: Unknown symbol class_create
slamr: Unknown symbol class_device_create
slamr: Unknown symbol class_device_destroy
Comment 1 Olaf Kirch 2005-08-25 15:37:47 UTC
The module's license is 
 
MODULE_LICENSE("Smart Link Ltd."); 
 
The symbols listed above are exported to GPL modules only. 
So we won't be able to support the driver in 10.0 unless SmartLink changes 
their license. 
Comment 2 Takashi Iwai 2005-08-25 15:52:08 UTC
Well, but this module is chosen by YaST.  If this can't work, we should disable
the yast functionality.
Comment 3 Marian Jancar 2005-08-25 16:37:34 UTC
Greg, can this be worked-around somehow in the module?
Comment 4 Takashi Iwai 2005-08-25 17:04:50 UTC
You can simply remove class_* stuff from the code.  But this will remove the
entries in sysfs, too, of course, so udev won't work together...

BTW, I'll be possibly able to try snd-intel8x0m instead of slamr.
Comment 5 Olaf Kirch 2005-08-26 08:08:19 UTC
*** Bug 113165 has been marked as a duplicate of this bug. ***
Comment 6 Olaf Kirch 2005-08-26 08:11:09 UTC
Takashi, can you elaborate - is the snd-intel8x0m driver a fully functional 
replacement for slamr? 
Comment 7 Takashi Iwai 2005-08-26 09:47:37 UTC
snd-intel8x0m, snd-via82xx-modem and snd-atiixp-modem drivers provide the ALSA
interface to access to the AC97 modem.  Then user-space slmodemd talks with the
device (when started with a specific option).
Since the binary-only part belongs only to slmodemd, the kernel modules are all
clean GPL.

I'm not sure whether all devices can be covered by ALSA modem drivers.
Need to check PCI table.

Nevertheless, we don't provide a way to configure these modules on YaST yet...
Comment 8 Gerald Pfeifer 2005-08-26 10:08:45 UTC
This is a regression from 9.1 and 9.2 at least, and means that Novell
employees (like me) with a corporate Thinkpad notebook won't be able to
upgrade.  Raising severity.
Comment 9 Takashi Iwai 2005-08-26 10:15:56 UTC
I don't like a pingpong game, but this isn't a bug I should solve.

I guess it's too late to build up the yast configurator for ALSA modem drivers.
Thus we need to fix slarmr somehow.  So I'll pass this to Marian :)
Comment 10 Marian Jancar 2005-08-26 15:31:09 UTC
Kay, what do you think about a /etc/udev/static-devices.d/, much like the
/etc/modprobe.d/? That would allow sane management of the entries from packages
and other tools.
Comment 11 Kay Sievers 2005-08-26 15:47:28 UTC
How is that related to this bug?

No, I don't want to spread this hack all over the packages. For now I want all
the static crap in the udev package itself. Most cases are just "bugs" or lazy
subsystem maintainers. I will continuously bug the people to fix it instead of
playing old tricks here.
Comment 12 Takashi Iwai 2005-08-26 15:50:26 UTC
The problem is that this driver can't use kernel sysfs API because of GPL
restriction.  Thus the device wouldn't be created if the driver is patched to
remove the access to class_* functions.
Comment 13 Marian Jancar 2005-08-26 15:59:11 UTC
With all the static nodes in udev everybody will get them unconditionaly.
Comment 14 Takashi Iwai 2005-08-26 16:11:25 UTC
I think simply adding to static_devices.txt is safer at this stage.  Also, I
agree with Kay, that the static devices should be as restricted as possible.

Comment 15 Marian Jancar 2005-08-26 16:16:24 UTC
I think there should be as few static devices under /dev as possible. For
virtual devices and now also for devices with non-GPL drivers there is no other
way. I maintain four such modem drivers and I don't feel comfortable poluting
the /dev with the static nodes unless neccessary, ie when the customer installs
the rpm.
Comment 16 Kay Sievers 2005-08-26 16:40:06 UTC
If the module is crippled that way, why not add the stuff to the slmodemd init
script then? That sounds like the best option to me.

I still try to avoid providing a generic infrastructure create static nodes. It
should be a uncomfortable and well reasoned exception, otherwise it will end in
the same mess we are currently try to get rid of.
Comment 17 Marian Jancar 2005-08-26 16:53:28 UTC
Ok, I will do that for the slmodem. The others are pure kernel driver without
initscripts, the nodes will have to be in udev then.
Comment 18 Gerald Pfeifer 2005-08-26 16:58:52 UTC
I'll be able to do basic tests (w/o actual dialing in) at any time, and 
could do functionality tests next Friday and Saturday if you want.
Comment 21 Marian Jancar 2005-08-29 15:14:38 UTC
fixed, could you please test, Gerald?
Comment 22 Gerald Pfeifer 2005-08-30 21:08:45 UTC
Thanks, Marian.  I installed a the current autobuild kernel and mbuilt
your latest smartlink-softmodem submission, but unfortunately I still get
the following, even after a reboot:
  
  # rpm -qa | egrep 'kernel|smart'
  kernel-default-nongpl-2.6.13-3
  kernel-default-2.6.13-3
  smartlink-softmodem-2.9.10-13
  kernel-default-debuginfo-2.6.13-3
  km_smartlink-softmodem-2.9.10-13

  # /etc/init.d/slmodemd status
  Status of SmartLink Modem driver:                                    unused

  # /etc/init.d/slmodemd start
  Starting SmartLink Modem driver: FATAL: Module slamr not found.
  FATAL: Module slusb not found.

  startproc:  exit status of parent of /usr/sbin/slmodemd: 255
                                                                     failed

Comment 23 Marian Jancar 2005-08-31 10:08:31 UTC
The fix has not been checked in yet, you can grab the source from
/work/src/done/STABLE/smartlink-softmodem
Comment 24 Gerald Pfeifer 2005-08-31 10:16:13 UTC
That's what I used for last night's testing (cf. the comment on me mbuilding
your latest smartlink-softmodem submission) in comment #22.
Comment 25 Gerald Pfeifer 2005-08-31 23:28:36 UTC
I confirmed this today by installing Beta 4 and then replacing the
smartlink-softmodem package by /work/built/mbuild/lifschitz-gp-3/.
Comment 26 Marian Jancar 2005-09-07 09:38:08 UTC
has been caused by a typo in the patch and the devices created staring from 1
instead of 0, fixed
Comment 27 Marian Jancar 2005-09-09 15:51:52 UTC
there was yet another error in the initscript, the device has to be with full
path, ie /dev/...