mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Indan Zupancic" <indan@nul.nu>
To: "mikie" <mikie.pl@gmail.com>
Cc: "Kay Sievers" <kay.sievers@vrfy.org>,
	"Duncan Sands" <duncan.sands@math.u-psud.fr>,
	linux-kernel@vger.kernel.org
Subject: Re: understanding firmware loader for speedtouch (kernel 2.6.21.5)
Date: Mon, 9 Jul 2007 17:03:10 +0200 (CEST)	[thread overview]
Message-ID: <33435.81.207.0.53.1183993390.squirrel@secure.samage.net> (raw)
In-Reply-To: <ca35ce500707090740n4de8f20av87fae2d3de5c3791@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 3130 bytes --]

On Mon, July 9, 2007 16:40, mikie wrote:
> 2007/7/9, Kay Sievers <kay.sievers@vrfy.org>:
>> On 7/9/07, mikie <mikie.pl@gmail.com> wrote:
>> > 2007/7/9, Indan Zupancic <indan@nul.nu>:
>> > > On Mon, July 9, 2007 10:49, mikie wrote:
>> > > > 2007/7/6, Indan Zupancic <indan@nul.nu>:
>> > > >> On Fri, July 6, 2007 16:20, Duncan Sands wrote:
>> > > >> > On Friday 6 July 2007 14:54:18 mikie wrote:
>> > > >> >> Hi,
>> > > >> >>
>> > > >> >> I experience some problems with the speedtch.c module, especially in
>> > > >> >> regards to its firmware loader.
>> > > >> >> I am not quite sure if this module is going to load the firmware
>> > > >> >> itself or does it use some external software to do that ?
>> > > >> >
>> > > >> > It loads it itself, using some external software!
>> > > >>
>> > > >> For information, it generates a hotplug event and the kernel calls the
>> > > >> program written at /proc/sys/kernel/hotplug with certain information.
>> > > >> That used to be /sbin/hotplug, became later udev, but today in general
>> > > >> udev reads the uevents generated by the kernel.
>> > > >
>> > > > On my system the /proc/sys/kernel/hotplug points to /sbin/hotplug.
>> > > > I copied your script to /sbin/hotplug and also added simple logging,
>> > > > so I can see whenever the script is being started. It turns out that
>> > > > the script is not started at all by the kernel...
>> > > >
>> > > > I am afraid that the kernel generates uevents only, could this be true?
>> > >
>> > > Not really, it would break backward compatibility, and if it were true,
>> > > they're remove the /proc/sys/kernel/hotplug setting too. As far as I know,
>> > > if it's set, that program will be executed, and if zero is written to it, only
>> > > uevents are generated.
>> > >
>> > > Make sure that the script is executable (chmod +x)
>> >
>> > Yes I have set it to be executable:
>> > root@srv:/sbin# ls -lah hotplug
>> > -rwxr-xr-x    1 root     root          934 Jul  9 09:13 hotplug
>> >
>> > > and has "#!/bin/sh"
>> > > at the top, or else it won't work. If that's already the case, I've no idea.
>> >
>> > root@srv:/sbin# head hotplug
>> > #!/bin/sh
>> > set -e
>> >
>> >
>> > Everything looks OK, but still it does not fire up the script...
>>
>> It should run, if the kernel creates events. Can you run "udevmonitor
>> --env" while loading the driver? That way we can see if events are
>> generated by the kernel (if you don't have it, no need to install
>> udev, just download the udev tarball, type "make" and run
>> "./udevmonitor --env").
>
> It seems that uevents are generated:

What if you run a very simple script like

#!/bin/sh
echo "test" >> /tmp/test

is it called at all?

If it is then it's probably something with the other script. Try removing
the set -e and/or run it manually with the right parameters set and see if
anything goes wrong. Or use a minimal udev and let it do the hotplugging,
having only rules for speedtouch and firmware loading (attached, but
make sure the paths and pppd call is correct for your system). /dev can stay
static, just change udev.conf to handle /udev or something.

Regards,

Indan

[-- Attachment #2: speedtch.rules --]
[-- Type: application/octet-stream, Size: 287 bytes --]

ACTION=="add", SUBSYSTEM=="firmware", ENV{FIRMWARE}=="?*", RUN+="/lib/udev/firmware.sh"

# add@/class/atm/speedtch0
ACTION=="add", SUBSYSTEM=="atm", KERNEL=="speedtch*" RUN="/usr/sbin/pppd call speedtch"

ACTION=="remove", SUBSYSTEM=="atm", KERNEL=="speedtch*" RUN="/usr/bin/pkill pppd"

  reply	other threads:[~2007-07-09 15:03 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-07-06 12:54 mikie
2007-07-06 14:20 ` Duncan Sands
2007-07-06 19:16   ` Indan Zupancic
2007-07-09  8:49     ` mikie
2007-07-09 10:42       ` Indan Zupancic
2007-07-09 11:08         ` mikie
2007-07-09 13:59           ` Kay Sievers
2007-07-09 14:40             ` mikie
2007-07-09 15:03               ` Indan Zupancic [this message]
2007-07-09 17:00                 ` mikie
     [not found]                   ` <200707091923.38577.duncan.sands@math.u-psud.fr>
2007-07-09 17:41                     ` mikie
2007-07-09 17:51                       ` Duncan Sands
2007-07-10  7:54                     ` mikie
2007-07-10  8:57                       ` Duncan Sands
2007-07-10  9:13                         ` mikie
2007-07-10  9:32                           ` Duncan Sands
2007-07-10 10:32                             ` mikie
2007-07-11 11:22                               ` mikie
2007-07-11 12:33                                 ` Duncan Sands
2007-07-11 12:47                                   ` mikie
2007-07-11 13:07                                     ` Duncan Sands
2007-07-11 16:55                                       ` mikie
2007-07-19 10:24                                         ` mikie
2007-07-24 23:03                                           ` Duncan Sands
2008-11-19 12:25                                             ` mikie
2008-11-19 12:49                                               ` Duncan Sands
2007-07-09 15:08               ` Kay Sievers
2007-07-09 10:50       ` Duncan Sands
2007-07-09 11:06         ` mikie
     [not found]   ` <ca35ce500707061006k74754bd6m8a9d2f1cb2910823@mail.gmail.com>
     [not found]     ` <200707061924.57099.duncan.sands@math.u-psud.fr>
2007-07-06 19:30       ` mikie

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=33435.81.207.0.53.1183993390.squirrel@secure.samage.net \
    --to=indan@nul.nu \
    --cc=duncan.sands@math.u-psud.fr \
    --cc=kay.sievers@vrfy.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mikie.pl@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®