| Message ID | 49DCA88E.6060400@tuffmail.co.uk |
|---|---|
| State | Not Applicable, archived |
| Delegated to: | David Miller |
| Headers | show |
On Wed, 2009-04-08 at 14:37 +0100, Alan Jenkins wrote: > This unused feature was removed in 4dec9b807be757780ca3611a959ac22c28d292a7 > > Signed-off-by: Alan Jenkins <alan-jenkins@tuffmail.co.uk> > --- > > p.s. Is netdev really the right list for rfkill (as listed in MAINTAINERS)? > I have a feeling most development happens on linux-wireless. You're right, and I'm rewriting rfkill and part of that is changing the maintainer to myself and the list to linux-wireless. I have no problems with this patch, but note that I'm rewriting that entire file as well so it's not really necessary in the long run -- now that -rc1 is out I can hopefully convert all in-tree rfkill stuff and get the patch in. johannes
Johannes Berg wrote: > On Wed, 2009-04-08 at 14:37 +0100, Alan Jenkins wrote: > >> This unused feature was removed in 4dec9b807be757780ca3611a959ac22c28d292a7 >> >> Signed-off-by: Alan Jenkins <alan-jenkins@tuffmail.co.uk> >> --- >> >> p.s. Is netdev really the right list for rfkill (as listed in MAINTAINERS)? >> I have a feeling most development happens on linux-wireless. >> > > You're right, and I'm rewriting rfkill and part of that is changing the > maintainer to myself and the list to linux-wireless. > Good luck! I'll try to have a look at your patch, to see how (if) it would affect the benighted eeepc-laptop. > I have no problems with this patch, but note that I'm rewriting that > entire file as well so it's not really necessary in the long run -- now > that -rc1 is out I can hopefully convert all in-tree rfkill stuff and > get the patch in. > Heh, ok. Alan -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Mon, 2009-04-13 at 20:00 +0100, Alan Jenkins wrote: > Alan Jenkins wrote: > > Johannes Berg wrote: > > > >> On Wed, 2009-04-08 at 14:37 +0100, Alan Jenkins wrote: > >> > >> You're right, and I'm rewriting rfkill and part of that is changing the > >> maintainer to myself and the list to linux-wireless. > >> > >> > > > > Good luck! I'll try to have a look at your patch, to see how (if) it > > would affect the benighted eeepc-laptop. > > > > I tried the latest patch (#19) from your website, it didn't work so well > :-). :) Thanks. I need to find some spare cycles to go through that patch again, probably sometime this week. johannes
Alan Jenkins wrote: > Johannes Berg wrote: > >> On Wed, 2009-04-08 at 14:37 +0100, Alan Jenkins wrote: >> >> You're right, and I'm rewriting rfkill and part of that is changing the >> maintainer to myself and the list to linux-wireless. >> >> > > Good luck! I'll try to have a look at your patch, to see how (if) it > would affect the benighted eeepc-laptop. > I tried the latest patch (#19) from your website, it didn't work so well :-). [ 78.301309] BUG: unable to handle kernel NULL pointer dereference at (null) [ 78.301326] IP: [<c0279468>] led_trigger_event+0x31/0x40 [ 78.301349] *pdpt = 0000000012375001 *pde = 0000000000000000 [ 78.301362] Oops: 0000 [#1] [ 78.301369] last sysfs file: /sys/module/backlight/initstate [ 78.301378] Modules linked in: eeepc_laptop(+) rfkill aes_i586 aes_generic af_packet i915 drm i2c_algo_bit cfbcopyarea cfbimgblt cfbfillrect ipv6 joydev arc4 ecb usual_tables(P) snd_hda_codec_realtek snd_hda_intel snd_hda_codec snd_hwdep ath5k snd_pcm_oss snd_mixer_oss mac80211 psmouse i2c_i801 uhci_hcd led_class serio_raw snd_pcm i2c_core pcspkr intel_agp ehci_hcd atl2 cfg80211 usbcore snd_timer snd_page_alloc agpgart video backlight output battery ac button processor evdev thermal fan ata_generic [ 78.301492] [ 78.301503] Pid: 2904, comm: modprobe Tainted: P (2.6.30-rc1-wleeepc #2) 701 [ 78.301513] EIP: 0060:[<c0279468>] EFLAGS: 00210286 CPU: 0 [ 78.301523] EIP is at led_trigger_event+0x31/0x40 [ 78.301532] EAX: df34e618 EBX: 00000000 ECX: df34e618 EDX: 000000ff [ 78.301541] ESI: 000000ff EDI: df34e624 EBP: d2373e44 ESP: d2373e38 [ 78.301551] DS: 007b ES: 007b FS: 0000 GS: 0033 SS: 0068 [ 78.301561] Process modprobe (pid: 2904, ti=d2372000 task=d2312400 task.ti=d2372000) [ 78.301569] Stack: [ 78.301574] df34e600 e0017f58 df268800 d2373e4c e01692af d2373e5c e01695f8 00000001 [ 78.301592] d21f6800 d2373e80 e00178bc 00000000 00000200 00000010 d22b1280 df268800 [ 78.301611] ffffffed e0019478 d2373e9c c020eff3 df268a04 ffffffda df268a04 ffffffed [ 78.301631] Call Trace: [ 78.301638] [<e01692af>] ? rfkill_led_trigger_event+0x1a/0x1c [rfkill] [ 78.301663] [<e01695f8>] ? rfkill_set_sw_state+0x53/0x69 [rfkill] [ 78.301685] [<e00178bc>] ? eeepc_hotk_add+0x2ca/0x3ef [eeepc_laptop] [ 78.301706] [<c020eff3>] ? acpi_device_probe+0x44/0x127 [ 78.301723] [<c02470a9>] ? driver_probe_device+0x7f/0xf2 [ 78.301740] [<c024715f>] ? __driver_attach+0x43/0x5f [ 78.301753] [<c024675b>] ? bus_for_each_dev+0x39/0x5a [ 78.301766] [<c0246f7c>] ? driver_attach+0x14/0x16 [ 78.301778] [<c024711c>] ? __driver_attach+0x0/0x5f [ 78.301791] [<c0246c5f>] ? bus_add_driver+0xd7/0x1e7 [ 78.301804] [<c0247365>] ? driver_register+0x7b/0xd7 [ 78.301822] [<c02106c9>] ? acpi_bus_register_driver+0x3a/0x3c [ 78.301835] [<e0011022>] ? eeepc_laptop_init+0x22/0x1b9 [eeepc_laptop] [ 78.301852] [<c0101131>] ? _stext+0x49/0x10b [ 78.301865] [<e0011000>] ? eeepc_laptop_init+0x0/0x1b9 [eeepc_laptop] [ 78.301882] [<c012f3c2>] ? __blocking_notifier_call_chain+0x40/0x4c [ 78.301899] [<c013a680>] ? sys_init_module+0x87/0x18b [ 78.301915] [<c0102804>] ? sysenter_do_call+0x12/0x22 [ 78.301928] Code: 56 89 d6 53 74 2f 8b 58 0c 8d 78 0c eb 1d 8d 4b c4 8b 41 08 39 c6 89 c2 0f 46 d6 f6 41 0c 01 89 51 04 75 05 89 c8 ff 51 10 8b 1b <8b> 03 0f 18 00 90 39 fb 75 d9 5b 5e 5f 5d c3 55 89 e5 56 8d 70 [ 78.302018] EIP: [<c0279468>] led_trigger_event+0x31/0x40 SS:ESP 0068:d2373e38 [ 78.302032] CR2: 0000000000000000 [ 78.302041] ---[ end trace 3e0b201a788d0ed8 ]--- Regards Alan -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Mon, 2009-04-13 at 20:00 +0100, Alan Jenkins wrote: > I tried the latest patch (#19) from your website, it didn't work so well > :-). I think I fixed that now, I allowed using set_sw_state before register and that didn't go too well with the LED system w/o any checks. johannes
Hi! > Ok, I retried with v6. It seemed to work well enough; my wireless > toggle key worked and either state was preserved over suspend and > hibernation. Alright, cool. > When I looked at the code earlier, I saw no obvious replacement for > rfkill_set_default(). So I tried disabling the wireless and rebooting > to see what happened. It didn't like that :-). Oops. I'll take a look tonight, thanks for testing! johannes > [ 3.258061] ------------[ cut here ]------------ > [ 3.258160] kernel BUG at kernel/workqueue.c:192! > [ 3.258256] invalid opcode: 0000 [#1] > [ 3.258391] last sysfs file: /sys/module/backlight/initstate > [ 3.258489] Modules linked in: video(+) output eeepc_laptop(+) > backlight ac battery rfkill but > ton processor evdev thermal fan ata_generic > [ 3.259261] > [ 3.259346] Pid: 1571, comm: modprobe Not tainted (2.6.30-rc2-wleeepc > #13) 701 > [ 3.259482] EIP: 0060:[<c0129e0a>] EFLAGS: 00010203 CPU: 0 > [ 3.259590] EIP is at queue_work_on+0x1a/0x31 > [ 3.259687] EAX: df089730 EBX: df00a060 ECX: df08972c EDX: 00000000 > [ 3.259790] ESI: e0045f54 EDI: df284800 EBP: df361e3c ESP: df361e38 > [ 3.259893] DS: 007b ES: 007b FS: 0000 GS: 0033 SS: 0068 > [ 3.259996] Process modprobe (pid: 1571, ti=df360000 task=de853c00 > task.ti=df360000) > [ 3.260131] Stack: > [ 3.260160] df089600 df361e44 c0129e41 df361e4c c0129e52 df361e5c > e00125f7 00010000 > [ 3.260160] df09c400 df361e80 e00458b9 00000000 00000200 00000010 > de59ae90 df284800 > [ 3.260160] ffffffed e0047474 df361e9c c02116c3 df2849fc ffffffda > df2849fc f > fffffed > [ 3.260160] Call Trace: > [ 3.260160] [<c0129e41>] ? queue_work+0xe/0x10 > [ 3.260160] [<c0129e52>] ? schedule_work+0xf/0x11 > [ 3.260160] [<e00125f7>] ? rfkill_set_sw_state+0x4c/0x69 [rfkill] > [ 3.260160] [<e00458b9>] ? eeepc_hotk_add+0x2c7/0x3ec [eeepc_laptop] > [ 3.260160] [<c02116c3>] ? acpi_device_probe+0x44/0x127 > [ 3.260160] [<c0249719>] ? driver_probe_device+0x7f/0xf2 > [ 3.260160] [<c02497cf>] ? __driver_attach+0x43/0x5f > [ 3.260160] [<c0248dcb>] ? bus_for_each_dev+0x39/0x5a > [ 3.260160] [<c02495ec>] ? driver_attach+0x14/0x16 > [ 3.260160] [<c024978c>] ? __driver_attach+0x0/0x5f > [ 3.260160] [<c02492cf>] ? bus_add_driver+0xd7/0x1e7 > [ 3.260160] [<c02499d5>] ? driver_register+0x7b/0xd7 > [ 3.260160] [<c0212d44>] ? acpi_bus_register_driver+0x3a/0x3c > [ 3.260160] [<e004a022>] ? eeepc_laptop_init+0x22/0x1b9 [eeepc_laptop] > [ 3.260160] [<c0101131>] ? _stext+0x49/0x10b > [ 3.260160] [<e004a000>] ? eeepc_laptop_init+0x0/0x1b9 [eeepc_laptop] > [ 3.260160] [<c012f476>] ? __blocking_notifier_call_chain+0x40/0x4c > [ 3.260160] [<c013a738>] ? sys_init_module+0x87/0x18f > [ 3.260160] [<c0102804>] ? sysenter_do_call+0x12/0x22 > [ 3.260160] Code: 83 e2 fc 8b 4a 14 89 c2 8b 01 e8 d9 ff ff ff 5d c3 > 55 89 e5 53 89 d3 0f ba 29 00 19 c0 31 d2 85 c0 75 1a 8d 41 04 39 41 04 > 74 04 <0f> 0b eb fe 8b 03 89 ca e8 b0 ff ff ff ba 01 00 00 00 5b 89 d0 > [ 3.260160] EIP: [<c0129e0a>] queue_work_on+0x1a/0x31 SS:ESP > 0068:df361e38 > [ 3.269670] ---[ end trace f8b6941350b5e97c ]--- > > Thanks > Alan >
Johannes Berg wrote: > I think I fixed that now, I allowed using set_sw_state before register > and that didn't go too well with the LED system w/o any checks. > > johannes > Ok, I retried with v6. It seemed to work well enough; my wireless toggle key worked and either state was preserved over suspend and hibernation. When I looked at the code earlier, I saw no obvious replacement for rfkill_set_default(). So I tried disabling the wireless and rebooting to see what happened. It didn't like that :-). [ 3.258061] ------------[ cut here ]------------ [ 3.258160] kernel BUG at kernel/workqueue.c:192! [ 3.258256] invalid opcode: 0000 [#1] [ 3.258391] last sysfs file: /sys/module/backlight/initstate [ 3.258489] Modules linked in: video(+) output eeepc_laptop(+) backlight ac battery rfkill but ton processor evdev thermal fan ata_generic [ 3.259261] [ 3.259346] Pid: 1571, comm: modprobe Not tainted (2.6.30-rc2-wleeepc #13) 701 [ 3.259482] EIP: 0060:[<c0129e0a>] EFLAGS: 00010203 CPU: 0 [ 3.259590] EIP is at queue_work_on+0x1a/0x31 [ 3.259687] EAX: df089730 EBX: df00a060 ECX: df08972c EDX: 00000000 [ 3.259790] ESI: e0045f54 EDI: df284800 EBP: df361e3c ESP: df361e38 [ 3.259893] DS: 007b ES: 007b FS: 0000 GS: 0033 SS: 0068 [ 3.259996] Process modprobe (pid: 1571, ti=df360000 task=de853c00 task.ti=df360000) [ 3.260131] Stack: [ 3.260160] df089600 df361e44 c0129e41 df361e4c c0129e52 df361e5c e00125f7 00010000 [ 3.260160] df09c400 df361e80 e00458b9 00000000 00000200 00000010 de59ae90 df284800 [ 3.260160] ffffffed e0047474 df361e9c c02116c3 df2849fc ffffffda df2849fc f fffffed [ 3.260160] Call Trace: [ 3.260160] [<c0129e41>] ? queue_work+0xe/0x10 [ 3.260160] [<c0129e52>] ? schedule_work+0xf/0x11 [ 3.260160] [<e00125f7>] ? rfkill_set_sw_state+0x4c/0x69 [rfkill] [ 3.260160] [<e00458b9>] ? eeepc_hotk_add+0x2c7/0x3ec [eeepc_laptop] [ 3.260160] [<c02116c3>] ? acpi_device_probe+0x44/0x127 [ 3.260160] [<c0249719>] ? driver_probe_device+0x7f/0xf2 [ 3.260160] [<c02497cf>] ? __driver_attach+0x43/0x5f [ 3.260160] [<c0248dcb>] ? bus_for_each_dev+0x39/0x5a [ 3.260160] [<c02495ec>] ? driver_attach+0x14/0x16 [ 3.260160] [<c024978c>] ? __driver_attach+0x0/0x5f [ 3.260160] [<c02492cf>] ? bus_add_driver+0xd7/0x1e7 [ 3.260160] [<c02499d5>] ? driver_register+0x7b/0xd7 [ 3.260160] [<c0212d44>] ? acpi_bus_register_driver+0x3a/0x3c [ 3.260160] [<e004a022>] ? eeepc_laptop_init+0x22/0x1b9 [eeepc_laptop] [ 3.260160] [<c0101131>] ? _stext+0x49/0x10b [ 3.260160] [<e004a000>] ? eeepc_laptop_init+0x0/0x1b9 [eeepc_laptop] [ 3.260160] [<c012f476>] ? __blocking_notifier_call_chain+0x40/0x4c [ 3.260160] [<c013a738>] ? sys_init_module+0x87/0x18f [ 3.260160] [<c0102804>] ? sysenter_do_call+0x12/0x22 [ 3.260160] Code: 83 e2 fc 8b 4a 14 89 c2 8b 01 e8 d9 ff ff ff 5d c3 55 89 e5 53 89 d3 0f ba 29 00 19 c0 31 d2 85 c0 75 1a 8d 41 04 39 41 04 74 04 <0f> 0b eb fe 8b 03 89 ca e8 b0 ff ff ff ba 01 00 00 00 5b 89 d0 [ 3.260160] EIP: [<c0129e0a>] queue_work_on+0x1a/0x31 SS:ESP 0068:df361e38 [ 3.269670] ---[ end trace f8b6941350b5e97c ]--- Thanks Alan -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/Documentation/rfkill.txt b/Documentation/rfkill.txt index 4d3ee31..357ef01 100644 --- a/Documentation/rfkill.txt +++ b/Documentation/rfkill.txt @@ -47,10 +47,8 @@ know when they should enable or disable a wireless network device transmitter. This is enabled by the CONFIG_RFKILL Kconfig option. The rfkill class support makes sure userspace will be notified of all state -changes on rfkill devices through uevents. It provides a notification chain -for interested parties in the kernel to also get notified of rfkill state -changes in other drivers. It creates several sysfs entries which can be used -by userspace. See section "Userspace support". +changes on rfkill devices through uevents. It creates several sysfs entries +which can be used by userspace. See section "Userspace support". The rfkill-input module provides the kernel with the ability to implement a basic response when the user presses a key or button (or toggles a switch) @@ -156,9 +154,8 @@ rfkill class: transmitter state; * Keeps track of the wireless transmitter state (with help from the driver); - * Generates userspace notifications (uevents) and a call to a - notification chain (kernel) when there is a wireless transmitter - state change; + * Generates userspace notifications (uevents) when there is a wireless + transmitter state change; * Connects a wireless communications driver with the common rfkill control system, which, for example, allows actions such as "switch all bluetooth devices offline" to be carried out by @@ -206,18 +203,15 @@ Userspace input handlers (uevents) or kernel input handlers (rfkill-input): restore the transmitters to their state before the EPO, or unblock them all. -Userspace uevent handler or kernel platform-specific drivers hooked to the -rfkill notifier chain: +Userspace uevent handler or kernel platform-specific drivers: - * Taps into the rfkill notifier chain or to KOBJ_CHANGE uevents, - in order to know when a device that is registered with the rfkill - class changes state; + * Listens to KOBJ_CHANGE uevents or the platform in order to know when + a device that is registered with the rfkill class changes state; * Issues feedback notifications to the user; * In the rare platforms where this is required, synthesizes an input event to command all *OTHER* rfkill devices to also change their statues when a specific rfkill device changes state. - =============================================================================== 3: Kernel driver guidelines @@ -269,8 +263,7 @@ SW_RFKILL_ALL, etc) when ALL of the folowing conditions are met: When in doubt, do not issue input events. For drivers that should generate input events in some platforms, but not in others (e.g. b43), the best solution is to NEVER generate input events in the first place. That work should be -deferred to a platform-specific kernel module (which will know when to generate -events through the rfkill notifier chain) or to userspace. This avoids the +deferred to a platform-specific kernel module or to userspace. This avoids the usual maintenance problems with DMI whitelisting.
This unused feature was removed in 4dec9b807be757780ca3611a959ac22c28d292a7 Signed-off-by: Alan Jenkins <alan-jenkins@tuffmail.co.uk> --- p.s. Is netdev really the right list for rfkill (as listed in MAINTAINERS)? I have a feeling most development happens on linux-wireless. Documentation/rfkill.txt | 23 ++++++++--------------- 1 files changed, 8 insertions(+), 15 deletions(-)