* [patch] ACPI toshiba: only register rfkill if bt is enabled
2008-10-28 19:10 2.6.28-rc2 - kernel BUG at /home/bor/src/linux-git/net/rfkill/rfkill.c:347! Andrey Borzenkov
@ 2008-10-28 23:51 ` Frederik Deweerdt
0 siblings, 0 replies; 12+ messages in thread
From: Frederik Deweerdt @ 2008-10-28 23:51 UTC (permalink / raw)
To: Andrey Borzenkov; +Cc: netdev, Linux Kernel Mailing List, philipl, akpm
Hello Andrey,
It looks like a part of the rfkill initialization was done whenever
BT was on or not. The following patch checks for BT presence before
registering the rfkill to the input layer. Does it solve the problem for
you?
Regards,
Frederik
Register toshiba RFKill Switch only if bluetooth is enabled
Part of the rfkill initialization was done whenever BT was on or not. The
following patch checks for BT presence before registering the rfkill to
the input layer. Some minor cleanups (> 80 char lines) were also added
in the process.
On Tue, Oct 28, 2008 at 10:10:37PM +0300, Andrey Borzenkov wrote:
[...]
> [ 66.633036] toshiba_acpi: Toshiba Laptop ACPI Extras version 0.19
> [ 66.633054] toshiba_acpi: HCI method: \_SB_.VALD.GHCI
> [ 66.637764] input: Toshiba RFKill Switch as /devices/virtual/input/input3
[...]
> [ 113.920753] ------------[ cut here ]------------
> [ 113.920828] kernel BUG at /home/bor/src/linux-git/net/rfkill/rfkill.c:347!
> [ 113.920845] invalid opcode: 0000 [#1]
> [ 113.920877] last sysfs file: /sys/devices/pci0000:00/0000:00:04.0/host0/target0:0:0/0:0:0:0/block/sda/size
> [ 113.920900] Dumping ftrace buffer:
> [ 113.920919] (ftrace buffer empty)
> [ 113.920933] Modules linked in: af_packet irnet ppp_generic slhc ircomm_tty ircomm binfmt_misc loop dm_mirror dm_region_hash dm_log dm_round_robin dm_multipath dm_mod alim15x3 ide_core nvram toshiba cryptomgr aead crypto_blkcipher michael_mic crypto_algapi orinoco_cs orinoco hermes_dld hermes pcmcia firmware_class snd_ali5451 snd_ac97_codec ac97_bus snd_seq_dummy snd_seq_oss snd_seq_midi_event snd_seq snd_seq_device smsc_ircc2 snd_pcm_oss snd_pcm rtc_cmos irda snd_timer snd_mixer_oss rtc_core snd crc_ccitt yenta_socket rtc_lib rsrc_nonstatic i2c_ali1535 pcmcia_core pcspkr psmouse soundcore i2c_core evdev sr_mod snd_page_alloc alim1535_wdt cdrom fan sg video output toshiba_acpi rfkill thermal backlight ali_agp processor ac button input_polldev battery agpgart ohci_hcd usbcore reiserfs pata_ali libata sd_mod scsi_mod [last unloaded: scsi_wait_scan]
> [ 113.921765]
> [ 113.921785] Pid: 3272, comm: ipolldevd Not tainted (2.6.28-rc2-1avb #3) PORTEGE 4000
> [ 113.921801] EIP: 0060:[<dfaa4683>] EFLAGS: 00010246 CPU: 0
> [ 113.921854] EIP is at rfkill_force_state+0x53/0x90 [rfkill]
> [ 113.921870] EAX: 00000000 EBX: 00000000 ECX: 00000003 EDX: 00000000
> [ 113.921885] ESI: 00000000 EDI: ddd50300 EBP: d8d7af40 ESP: d8d7af24
> [ 113.921900] DS: 007b ES: 007b FS: 0000 GS: 0000 SS: 0068
> [ 113.921918] Process ipolldevd (pid: 3272, ti=d8d7a000 task=d8d93c90 task.ti=d8d7a000)
> [ 113.921933] Stack:
> [ 113.921945] d8d7af38 00000246 dfb029d8 dfb029c0 dfb029d8 dfb029c0 ddd50300 d8d7af5c
> [ 113.922014] dfb018e2 01000246 01000000 ddd50300 ddd50314 ddabb8a0 d8d7af68 dfb381c1
> [ 113.922098] 00000000 d8d7afa4 c012ec0a 00000000 00000002 00000000 c012eba8 ddabb8c0
> [ 113.922240] Call Trace:
> [ 113.922240] [<dfb018e2>] ? bt_poll_rfkill+0x5c/0x82 [toshiba_acpi]
> [ 113.922240] [<dfb381c1>] ? input_polled_device_work+0x11/0x40 [input_polldev]
> [ 113.922240] [<c012ec0a>] ? run_workqueue+0xea/0x1f0
> [ 113.922240] [<c012eba8>] ? run_workqueue+0x88/0x1f0
> [ 113.922240] [<dfb381b0>] ? input_polled_device_work+0x0/0x40 [input_polldev]
> [ 113.922240] [<c012f047>] ? worker_thread+0x87/0xf0
> [ 113.922240] [<c0132b00>] ? autoremove_wake_function+0x0/0x50
> [ 113.922240] [<c012efc0>] ? worker_thread+0x0/0xf0
> [ 113.922240] [<c013280f>] ? kthread+0x3f/0x80
> [ 113.922240] [<c01327d0>] ? kthread+0x0/0x80
> [ 113.922240] [<c01040d7>] ? kernel_thread_helper+0x7/0x10
> [ 113.922240] Code: 43 54 89 73 54 39 c6 74 11 89 d9 ba 01 00 00 00 b8 40 68 aa df e8 3e 35 69 e0 89 f8 e8 77 fd 85 e0 31 c0 83 c4 10 5b 5e 5f 5d c3 <0f> 0b eb fe 89 f6 8d bc 27 00 00 00 00 be f4 4d aa df bb 5f 01
> [ 113.922240] EIP: [<dfaa4683>] rfkill_force_state+0x53/0x90 [rfkill] SS:ESP 0068:d8d7af24
> [ 113.924700] ---[ end trace 0e404eb40cadd5f0 ]---
Signed-off-by: Frederik Deweerdt <frederik.deweerdt@gmail.com>
diff --git a/drivers/acpi/toshiba_acpi.c b/drivers/acpi/toshiba_acpi.c
index 66aac06..c1c9d87 100644
--- a/drivers/acpi/toshiba_acpi.c
+++ b/drivers/acpi/toshiba_acpi.c
@@ -824,34 +824,38 @@ static int __init toshiba_acpi_init(void)
toshiba_acpi_exit();
return -ENOMEM;
}
- }
- /* Register input device for kill switch */
- toshiba_acpi.poll_dev = input_allocate_polled_device();
- if (!toshiba_acpi.poll_dev) {
- printk(MY_ERR "unable to allocate kill-switch input device\n");
- toshiba_acpi_exit();
- return -ENOMEM;
- }
- toshiba_acpi.poll_dev->private = &toshiba_acpi;
- toshiba_acpi.poll_dev->poll = bt_poll_rfkill;
- toshiba_acpi.poll_dev->poll_interval = 1000; /* msecs */
-
- toshiba_acpi.poll_dev->input->name = toshiba_acpi.rfk_name;
- toshiba_acpi.poll_dev->input->id.bustype = BUS_HOST;
- toshiba_acpi.poll_dev->input->id.vendor = 0x0930; /* Toshiba USB ID */
- set_bit(EV_SW, toshiba_acpi.poll_dev->input->evbit);
- set_bit(SW_RFKILL_ALL, toshiba_acpi.poll_dev->input->swbit);
- input_report_switch(toshiba_acpi.poll_dev->input, SW_RFKILL_ALL, TRUE);
- input_sync(toshiba_acpi.poll_dev->input);
-
- ret = input_register_polled_device(toshiba_acpi.poll_dev);
- if (ret) {
- printk(MY_ERR "unable to register kill-switch input device\n");
- rfkill_free(toshiba_acpi.rfk_dev);
- toshiba_acpi.rfk_dev = NULL;
- toshiba_acpi_exit();
- return ret;
+ /* Register input device for kill switch */
+ toshiba_acpi.poll_dev = input_allocate_polled_device();
+ if (!toshiba_acpi.poll_dev) {
+ printk(MY_ERR
+ "unable to allocate kill-switch input device\n");
+ toshiba_acpi_exit();
+ return -ENOMEM;
+ }
+ toshiba_acpi.poll_dev->private = &toshiba_acpi;
+ toshiba_acpi.poll_dev->poll = bt_poll_rfkill;
+ toshiba_acpi.poll_dev->poll_interval = 1000; /* msecs */
+
+ toshiba_acpi.poll_dev->input->name = toshiba_acpi.rfk_name;
+ toshiba_acpi.poll_dev->input->id.bustype = BUS_HOST;
+ /* Toshiba USB ID */
+ toshiba_acpi.poll_dev->input->id.vendor = 0x0930;
+ set_bit(EV_SW, toshiba_acpi.poll_dev->input->evbit);
+ set_bit(SW_RFKILL_ALL, toshiba_acpi.poll_dev->input->swbit);
+ input_report_switch(toshiba_acpi.poll_dev->input,
+ SW_RFKILL_ALL, TRUE);
+ input_sync(toshiba_acpi.poll_dev->input);
+
+ ret = input_register_polled_device(toshiba_acpi.poll_dev);
+ if (ret) {
+ printk(MY_ERR
+ "unable to register kill-switch input device\n");
+ rfkill_free(toshiba_acpi.rfk_dev);
+ toshiba_acpi.rfk_dev = NULL;
+ toshiba_acpi_exit();
+ return ret;
+ }
}
return 0;
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [patch] ACPI toshiba: only register rfkill if bt is enabled
@ 2008-10-29 8:19 Frederik Deweerdt
2008-10-29 13:10 ` Frans Pop
` (3 more replies)
0 siblings, 4 replies; 12+ messages in thread
From: Frederik Deweerdt @ 2008-10-29 8:19 UTC (permalink / raw)
To: arvidjaar; +Cc: linux-kernel, akpm, philipl
[resending as it didn't make it to the list, removing netdev from cc
also]
Hello Andrey,
It looks like a part of the rfkill initialization was done whenever
BT was on or not. The following patch checks for BT presence before
registering the rfkill to the input layer. Does it solve the problem for
you?
Regards,
Frederik
Register toshiba RFKill Switch only if bluetooth is enabled
Part of the rfkill initialization was done whenever BT was on or not. The
following patch checks for BT presence before registering the rfkill to
the input layer. Some minor cleanups (> 80 char lines) were also added
in the process.
On Tue, Oct 28, 2008 at 10:10:37PM +0300, Andrey Borzenkov wrote:
[...]
> [ 66.633036] toshiba_acpi: Toshiba Laptop ACPI Extras version 0.19
> [ 66.633054] toshiba_acpi: HCI method: \_SB_.VALD.GHCI
> [ 66.637764] input: Toshiba RFKill Switch as /devices/virtual/input/input3
[...]
> [ 113.920753] ------------[ cut here ]------------
> [ 113.920828] kernel BUG at /home/bor/src/linux-git/net/rfkill/rfkill.c:347!
> [ 113.920845] invalid opcode: 0000 [#1]
> [ 113.920877] last sysfs file: /sys/devices/pci0000:00/0000:00:04.0/host0/target0:0:0/0:0:0:0/block/sda/size
> [ 113.920900] Dumping ftrace buffer:
> [ 113.920919] (ftrace buffer empty)
> [ 113.920933] Modules linked in: af_packet irnet ppp_generic slhc ircomm_tty ircomm binfmt_misc loop dm_mirror dm_region_hash dm_log dm_round_robin dm_multipath dm_mod alim15x3 ide_core nvram toshiba cryptomgr aead crypto_blkcipher michael_mic crypto_algapi orinoco_cs orinoco hermes_dld hermes pcmcia firmware_class snd_ali5451 snd_ac97_codec ac97_bus snd_seq_dummy snd_seq_oss snd_seq_midi_event snd_seq snd_seq_device smsc_ircc2 snd_pcm_oss snd_pcm rtc_cmos irda snd_timer snd_mixer_oss rtc_core snd crc_ccitt yenta_socket rtc_lib rsrc_nonstatic i2c_ali1535 pcmcia_core pcspkr psmouse soundcore i2c_core evdev sr_mod snd_page_alloc alim1535_wdt cdrom fan sg video output toshiba_acpi rfkill thermal backlight ali_agp processor ac button input_polldev battery agpgart ohci_hcd usbcore reiserfs pata_ali libata sd_mod scsi_mod [last unloaded: scsi_wait_scan]
> [ 113.921765]
> [ 113.921785] Pid: 3272, comm: ipolldevd Not tainted (2.6.28-rc2-1avb #3) PORTEGE 4000
> [ 113.921801] EIP: 0060:[<dfaa4683>] EFLAGS: 00010246 CPU: 0
> [ 113.921854] EIP is at rfkill_force_state+0x53/0x90 [rfkill]
> [ 113.921870] EAX: 00000000 EBX: 00000000 ECX: 00000003 EDX: 00000000
> [ 113.921885] ESI: 00000000 EDI: ddd50300 EBP: d8d7af40 ESP: d8d7af24
> [ 113.921900] DS: 007b ES: 007b FS: 0000 GS: 0000 SS: 0068
> [ 113.921918] Process ipolldevd (pid: 3272, ti=d8d7a000 task=d8d93c90 task.ti=d8d7a000)
> [ 113.921933] Stack:
> [ 113.921945] d8d7af38 00000246 dfb029d8 dfb029c0 dfb029d8 dfb029c0 ddd50300 d8d7af5c
> [ 113.922014] dfb018e2 01000246 01000000 ddd50300 ddd50314 ddabb8a0 d8d7af68 dfb381c1
> [ 113.922098] 00000000 d8d7afa4 c012ec0a 00000000 00000002 00000000 c012eba8 ddabb8c0
> [ 113.922240] Call Trace:
> [ 113.922240] [<dfb018e2>] ? bt_poll_rfkill+0x5c/0x82 [toshiba_acpi]
> [ 113.922240] [<dfb381c1>] ? input_polled_device_work+0x11/0x40 [input_polldev]
> [ 113.922240] [<c012ec0a>] ? run_workqueue+0xea/0x1f0
> [ 113.922240] [<c012eba8>] ? run_workqueue+0x88/0x1f0
> [ 113.922240] [<dfb381b0>] ? input_polled_device_work+0x0/0x40 [input_polldev]
> [ 113.922240] [<c012f047>] ? worker_thread+0x87/0xf0
> [ 113.922240] [<c0132b00>] ? autoremove_wake_function+0x0/0x50
> [ 113.922240] [<c012efc0>] ? worker_thread+0x0/0xf0
> [ 113.922240] [<c013280f>] ? kthread+0x3f/0x80
> [ 113.922240] [<c01327d0>] ? kthread+0x0/0x80
> [ 113.922240] [<c01040d7>] ? kernel_thread_helper+0x7/0x10
> [ 113.922240] Code: 43 54 89 73 54 39 c6 74 11 89 d9 ba 01 00 00 00 b8 40 68 aa df e8 3e 35 69 e0 89 f8 e8 77 fd 85 e0 31 c0 83 c4 10 5b 5e 5f 5d c3 <0f> 0b eb fe 89 f6 8d bc 27 00 00 00 00 be f4 4d aa df bb 5f 01
> [ 113.922240] EIP: [<dfaa4683>] rfkill_force_state+0x53/0x90 [rfkill] SS:ESP 0068:d8d7af24
> [ 113.924700] ---[ end trace 0e404eb40cadd5f0 ]---
Signed-off-by: Frederik Deweerdt <frederik.deweerdt@gmail.com>
diff --git a/drivers/acpi/toshiba_acpi.c b/drivers/acpi/toshiba_acpi.c
index 66aac06..c1c9d87 100644
--- a/drivers/acpi/toshiba_acpi.c
+++ b/drivers/acpi/toshiba_acpi.c
@@ -824,34 +824,38 @@ static int __init toshiba_acpi_init(void)
toshiba_acpi_exit();
return -ENOMEM;
}
- }
- /* Register input device for kill switch */
- toshiba_acpi.poll_dev = input_allocate_polled_device();
- if (!toshiba_acpi.poll_dev) {
- printk(MY_ERR "unable to allocate kill-switch input device\n");
- toshiba_acpi_exit();
- return -ENOMEM;
- }
- toshiba_acpi.poll_dev->private = &toshiba_acpi;
- toshiba_acpi.poll_dev->poll = bt_poll_rfkill;
- toshiba_acpi.poll_dev->poll_interval = 1000; /* msecs */
-
- toshiba_acpi.poll_dev->input->name = toshiba_acpi.rfk_name;
- toshiba_acpi.poll_dev->input->id.bustype = BUS_HOST;
- toshiba_acpi.poll_dev->input->id.vendor = 0x0930; /* Toshiba USB ID */
- set_bit(EV_SW, toshiba_acpi.poll_dev->input->evbit);
- set_bit(SW_RFKILL_ALL, toshiba_acpi.poll_dev->input->swbit);
- input_report_switch(toshiba_acpi.poll_dev->input, SW_RFKILL_ALL, TRUE);
- input_sync(toshiba_acpi.poll_dev->input);
-
- ret = input_register_polled_device(toshiba_acpi.poll_dev);
- if (ret) {
- printk(MY_ERR "unable to register kill-switch input device\n");
- rfkill_free(toshiba_acpi.rfk_dev);
- toshiba_acpi.rfk_dev = NULL;
- toshiba_acpi_exit();
- return ret;
+ /* Register input device for kill switch */
+ toshiba_acpi.poll_dev = input_allocate_polled_device();
+ if (!toshiba_acpi.poll_dev) {
+ printk(MY_ERR
+ "unable to allocate kill-switch input device\n");
+ toshiba_acpi_exit();
+ return -ENOMEM;
+ }
+ toshiba_acpi.poll_dev->private = &toshiba_acpi;
+ toshiba_acpi.poll_dev->poll = bt_poll_rfkill;
+ toshiba_acpi.poll_dev->poll_interval = 1000; /* msecs */
+
+ toshiba_acpi.poll_dev->input->name = toshiba_acpi.rfk_name;
+ toshiba_acpi.poll_dev->input->id.bustype = BUS_HOST;
+ /* Toshiba USB ID */
+ toshiba_acpi.poll_dev->input->id.vendor = 0x0930;
+ set_bit(EV_SW, toshiba_acpi.poll_dev->input->evbit);
+ set_bit(SW_RFKILL_ALL, toshiba_acpi.poll_dev->input->swbit);
+ input_report_switch(toshiba_acpi.poll_dev->input,
+ SW_RFKILL_ALL, TRUE);
+ input_sync(toshiba_acpi.poll_dev->input);
+
+ ret = input_register_polled_device(toshiba_acpi.poll_dev);
+ if (ret) {
+ printk(MY_ERR
+ "unable to register kill-switch input device\n");
+ rfkill_free(toshiba_acpi.rfk_dev);
+ toshiba_acpi.rfk_dev = NULL;
+ toshiba_acpi_exit();
+ return ret;
+ }
}
return 0;
----- End forwarded message -----
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [patch] ACPI toshiba: only register rfkill if bt is enabled
2008-10-29 8:19 [patch] ACPI toshiba: only register rfkill if bt is enabled Frederik Deweerdt
@ 2008-10-29 13:10 ` Frans Pop
2008-10-29 15:30 ` Philip Langdale
` (2 subsequent siblings)
3 siblings, 0 replies; 12+ messages in thread
From: Frans Pop @ 2008-10-29 13:10 UTC (permalink / raw)
To: Frederik Deweerdt; +Cc: arvidjaar, linux-kernel, akpm, philipl
Frederik Deweerdt wrote:
> It looks like a part of the rfkill initialization was done whenever
> BT was on or not. The following patch checks for BT presence before
> registering the rfkill to the input layer. Does it solve the problem for
> you?
>
> Regards,
> Frederik
>
> Register toshiba RFKill Switch only if bluetooth is enabled
>
> Part of the rfkill initialization was done whenever BT was on or not.
> The following patch checks for BT presence before registering the rfkill
> to the input layer. Some minor cleanups (> 80 char lines) were also
> added in the process.
I just saw the same issue on my Toshiba Satellite A40 with Atheros pcmcia
card. The patch solves the kernel BUG for me when applied on top of
current git.
Cheers,
FJP
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [patch] ACPI toshiba: only register rfkill if bt is enabled
2008-10-29 8:19 [patch] ACPI toshiba: only register rfkill if bt is enabled Frederik Deweerdt
2008-10-29 13:10 ` Frans Pop
@ 2008-10-29 15:30 ` Philip Langdale
2008-10-29 17:05 ` Andrey Borzenkov
2008-11-16 8:28 ` Andrey Borzenkov
2008-12-01 19:30 ` Andrew Morton
3 siblings, 1 reply; 12+ messages in thread
From: Philip Langdale @ 2008-10-29 15:30 UTC (permalink / raw)
To: Frederik Deweerdt; +Cc: arvidjaar, linux-kernel, akpm
Acked-by: Philip Langdale <philipl@overt.org>
--phil
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [patch] ACPI toshiba: only register rfkill if bt is enabled
2008-10-29 15:30 ` Philip Langdale
@ 2008-10-29 17:05 ` Andrey Borzenkov
0 siblings, 0 replies; 12+ messages in thread
From: Andrey Borzenkov @ 2008-10-29 17:05 UTC (permalink / raw)
To: Philip Langdale; +Cc: Frederik Deweerdt, linux-kernel, akpm
[-- Attachment #1: Type: text/plain, Size: 158 bytes --]
On Wednesday 29 October 2008, Philip Langdale wrote:
> Acked-by: Philip Langdale <philipl@overt.org>
>
Tested-by: Andrey Borzenkov <arvidjaar@mail.ru>
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 197 bytes --]
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [patch] ACPI toshiba: only register rfkill if bt is enabled
2008-10-29 8:19 [patch] ACPI toshiba: only register rfkill if bt is enabled Frederik Deweerdt
2008-10-29 13:10 ` Frans Pop
2008-10-29 15:30 ` Philip Langdale
@ 2008-11-16 8:28 ` Andrey Borzenkov
2008-11-16 12:40 ` Rafael J. Wysocki
2008-12-01 19:30 ` Andrew Morton
3 siblings, 1 reply; 12+ messages in thread
From: Andrey Borzenkov @ 2008-11-16 8:28 UTC (permalink / raw)
To: Frederik Deweerdt, linux-acpi, Rafael J. Wysocki
Cc: linux-kernel, akpm, philipl
[-- Attachment #1: Type: text/plain, Size: 7336 bytes --]
On Wednesday 29 October 2008, Frederik Deweerdt wrote:
> [resending as it didn't make it to the list, removing netdev from cc
> also]
> Hello Andrey,
>
> It looks like a part of the rfkill initialization was done whenever
> BT was on or not. The following patch checks for BT presence before
> registering the rfkill to the input layer. Does it solve the problem for
> you?
>
This patch fixes regression in 2.6.28. Patch has been tested and
acnowledged. Rafael, could you add it to list of open regressions?
Thank you!
> Regards,
> Frederik
>
> Register toshiba RFKill Switch only if bluetooth is enabled
>
> Part of the rfkill initialization was done whenever BT was on or not. The
> following patch checks for BT presence before registering the rfkill to
> the input layer. Some minor cleanups (> 80 char lines) were also added
> in the process.
>
> On Tue, Oct 28, 2008 at 10:10:37PM +0300, Andrey Borzenkov wrote:
> [...]
> > [ 66.633036] toshiba_acpi: Toshiba Laptop ACPI Extras version 0.19
> > [ 66.633054] toshiba_acpi: HCI method: \_SB_.VALD.GHCI
> > [ 66.637764] input: Toshiba RFKill Switch as /devices/virtual/input/input3
> [...]
> > [ 113.920753] ------------[ cut here ]------------
> > [ 113.920828] kernel BUG at /home/bor/src/linux-git/net/rfkill/rfkill.c:347!
> > [ 113.920845] invalid opcode: 0000 [#1]
> > [ 113.920877] last sysfs file: /sys/devices/pci0000:00/0000:00:04.0/host0/target0:0:0/0:0:0:0/block/sda/size
> > [ 113.920900] Dumping ftrace buffer:
> > [ 113.920919] (ftrace buffer empty)
> > [ 113.920933] Modules linked in: af_packet irnet ppp_generic slhc ircomm_tty ircomm binfmt_misc loop dm_mirror dm_region_hash dm_log dm_round_robin dm_multipath dm_mod alim15x3 ide_core nvram toshiba cryptomgr aead crypto_blkcipher michael_mic crypto_algapi orinoco_cs orinoco hermes_dld hermes pcmcia firmware_class snd_ali5451 snd_ac97_codec ac97_bus snd_seq_dummy snd_seq_oss snd_seq_midi_event snd_seq snd_seq_device smsc_ircc2 snd_pcm_oss snd_pcm rtc_cmos irda snd_timer snd_mixer_oss rtc_core snd crc_ccitt yenta_socket rtc_lib rsrc_nonstatic i2c_ali1535 pcmcia_core pcspkr psmouse soundcore i2c_core evdev sr_mod snd_page_alloc alim1535_wdt cdrom fan sg video output toshiba_acpi rfkill thermal backlight ali_agp processor ac button input_polldev battery agpgart ohci_hcd usbcore reiserfs pata_ali libata sd_mod scsi_mod [last unloaded: scsi_wait_scan]
> > [ 113.921765]
> > [ 113.921785] Pid: 3272, comm: ipolldevd Not tainted (2.6.28-rc2-1avb #3) PORTEGE 4000
> > [ 113.921801] EIP: 0060:[<dfaa4683>] EFLAGS: 00010246 CPU: 0
> > [ 113.921854] EIP is at rfkill_force_state+0x53/0x90 [rfkill]
> > [ 113.921870] EAX: 00000000 EBX: 00000000 ECX: 00000003 EDX: 00000000
> > [ 113.921885] ESI: 00000000 EDI: ddd50300 EBP: d8d7af40 ESP: d8d7af24
> > [ 113.921900] DS: 007b ES: 007b FS: 0000 GS: 0000 SS: 0068
> > [ 113.921918] Process ipolldevd (pid: 3272, ti=d8d7a000 task=d8d93c90 task.ti=d8d7a000)
> > [ 113.921933] Stack:
> > [ 113.921945] d8d7af38 00000246 dfb029d8 dfb029c0 dfb029d8 dfb029c0 ddd50300 d8d7af5c
> > [ 113.922014] dfb018e2 01000246 01000000 ddd50300 ddd50314 ddabb8a0 d8d7af68 dfb381c1
> > [ 113.922098] 00000000 d8d7afa4 c012ec0a 00000000 00000002 00000000 c012eba8 ddabb8c0
> > [ 113.922240] Call Trace:
> > [ 113.922240] [<dfb018e2>] ? bt_poll_rfkill+0x5c/0x82 [toshiba_acpi]
> > [ 113.922240] [<dfb381c1>] ? input_polled_device_work+0x11/0x40 [input_polldev]
> > [ 113.922240] [<c012ec0a>] ? run_workqueue+0xea/0x1f0
> > [ 113.922240] [<c012eba8>] ? run_workqueue+0x88/0x1f0
> > [ 113.922240] [<dfb381b0>] ? input_polled_device_work+0x0/0x40 [input_polldev]
> > [ 113.922240] [<c012f047>] ? worker_thread+0x87/0xf0
> > [ 113.922240] [<c0132b00>] ? autoremove_wake_function+0x0/0x50
> > [ 113.922240] [<c012efc0>] ? worker_thread+0x0/0xf0
> > [ 113.922240] [<c013280f>] ? kthread+0x3f/0x80
> > [ 113.922240] [<c01327d0>] ? kthread+0x0/0x80
> > [ 113.922240] [<c01040d7>] ? kernel_thread_helper+0x7/0x10
> > [ 113.922240] Code: 43 54 89 73 54 39 c6 74 11 89 d9 ba 01 00 00 00 b8 40 68 aa df e8 3e 35 69 e0 89 f8 e8 77 fd 85 e0 31 c0 83 c4 10 5b 5e 5f 5d c3 <0f> 0b eb fe 89 f6 8d bc 27 00 00 00 00 be f4 4d aa df bb 5f 01
> > [ 113.922240] EIP: [<dfaa4683>] rfkill_force_state+0x53/0x90 [rfkill] SS:ESP 0068:d8d7af24
> > [ 113.924700] ---[ end trace 0e404eb40cadd5f0 ]---
>
> Signed-off-by: Frederik Deweerdt <frederik.deweerdt@gmail.com>
>
> diff --git a/drivers/acpi/toshiba_acpi.c b/drivers/acpi/toshiba_acpi.c
> index 66aac06..c1c9d87 100644
> --- a/drivers/acpi/toshiba_acpi.c
> +++ b/drivers/acpi/toshiba_acpi.c
> @@ -824,34 +824,38 @@ static int __init toshiba_acpi_init(void)
> toshiba_acpi_exit();
> return -ENOMEM;
> }
> - }
>
> - /* Register input device for kill switch */
> - toshiba_acpi.poll_dev = input_allocate_polled_device();
> - if (!toshiba_acpi.poll_dev) {
> - printk(MY_ERR "unable to allocate kill-switch input device\n");
> - toshiba_acpi_exit();
> - return -ENOMEM;
> - }
> - toshiba_acpi.poll_dev->private = &toshiba_acpi;
> - toshiba_acpi.poll_dev->poll = bt_poll_rfkill;
> - toshiba_acpi.poll_dev->poll_interval = 1000; /* msecs */
> -
> - toshiba_acpi.poll_dev->input->name = toshiba_acpi.rfk_name;
> - toshiba_acpi.poll_dev->input->id.bustype = BUS_HOST;
> - toshiba_acpi.poll_dev->input->id.vendor = 0x0930; /* Toshiba USB ID */
> - set_bit(EV_SW, toshiba_acpi.poll_dev->input->evbit);
> - set_bit(SW_RFKILL_ALL, toshiba_acpi.poll_dev->input->swbit);
> - input_report_switch(toshiba_acpi.poll_dev->input, SW_RFKILL_ALL, TRUE);
> - input_sync(toshiba_acpi.poll_dev->input);
> -
> - ret = input_register_polled_device(toshiba_acpi.poll_dev);
> - if (ret) {
> - printk(MY_ERR "unable to register kill-switch input device\n");
> - rfkill_free(toshiba_acpi.rfk_dev);
> - toshiba_acpi.rfk_dev = NULL;
> - toshiba_acpi_exit();
> - return ret;
> + /* Register input device for kill switch */
> + toshiba_acpi.poll_dev = input_allocate_polled_device();
> + if (!toshiba_acpi.poll_dev) {
> + printk(MY_ERR
> + "unable to allocate kill-switch input device\n");
> + toshiba_acpi_exit();
> + return -ENOMEM;
> + }
> + toshiba_acpi.poll_dev->private = &toshiba_acpi;
> + toshiba_acpi.poll_dev->poll = bt_poll_rfkill;
> + toshiba_acpi.poll_dev->poll_interval = 1000; /* msecs */
> +
> + toshiba_acpi.poll_dev->input->name = toshiba_acpi.rfk_name;
> + toshiba_acpi.poll_dev->input->id.bustype = BUS_HOST;
> + /* Toshiba USB ID */
> + toshiba_acpi.poll_dev->input->id.vendor = 0x0930;
> + set_bit(EV_SW, toshiba_acpi.poll_dev->input->evbit);
> + set_bit(SW_RFKILL_ALL, toshiba_acpi.poll_dev->input->swbit);
> + input_report_switch(toshiba_acpi.poll_dev->input,
> + SW_RFKILL_ALL, TRUE);
> + input_sync(toshiba_acpi.poll_dev->input);
> +
> + ret = input_register_polled_device(toshiba_acpi.poll_dev);
> + if (ret) {
> + printk(MY_ERR
> + "unable to register kill-switch input device\n");
> + rfkill_free(toshiba_acpi.rfk_dev);
> + toshiba_acpi.rfk_dev = NULL;
> + toshiba_acpi_exit();
> + return ret;
> + }
> }
>
> return 0;
>
> ----- End forwarded message -----
>
>
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 197 bytes --]
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [patch] ACPI toshiba: only register rfkill if bt is enabled
2008-11-16 8:28 ` Andrey Borzenkov
@ 2008-11-16 12:40 ` Rafael J. Wysocki
2008-11-16 12:50 ` Andrey Borzenkov
0 siblings, 1 reply; 12+ messages in thread
From: Rafael J. Wysocki @ 2008-11-16 12:40 UTC (permalink / raw)
To: Andrey Borzenkov
Cc: Frederik Deweerdt, linux-acpi, linux-kernel, akpm, philipl
On Sunday, 16 of November 2008, Andrey Borzenkov wrote:
> On Wednesday 29 October 2008, Frederik Deweerdt wrote:
> > [resending as it didn't make it to the list, removing netdev from cc
> > also]
> > Hello Andrey,
> >
> > It looks like a part of the rfkill initialization was done whenever
> > BT was on or not. The following patch checks for BT presence before
> > registering the rfkill to the input layer. Does it solve the problem for
> > you?
> >
>
> This patch fixes regression in 2.6.28. Patch has been tested and
> acnowledged. Rafael, could you add it to list of open regressions?
> Thank you!
I will, if you provide me with a link to the patch and (optionally) to the
original bug report.
Thanks,
Rafael
> > Register toshiba RFKill Switch only if bluetooth is enabled
> >
> > Part of the rfkill initialization was done whenever BT was on or not. The
> > following patch checks for BT presence before registering the rfkill to
> > the input layer. Some minor cleanups (> 80 char lines) were also added
> > in the process.
> >
> > On Tue, Oct 28, 2008 at 10:10:37PM +0300, Andrey Borzenkov wrote:
> > [...]
> > > [ 66.633036] toshiba_acpi: Toshiba Laptop ACPI Extras version 0.19
> > > [ 66.633054] toshiba_acpi: HCI method: \_SB_.VALD.GHCI
> > > [ 66.637764] input: Toshiba RFKill Switch as /devices/virtual/input/input3
> > [...]
> > > [ 113.920753] ------------[ cut here ]------------
> > > [ 113.920828] kernel BUG at /home/bor/src/linux-git/net/rfkill/rfkill.c:347!
> > > [ 113.920845] invalid opcode: 0000 [#1]
> > > [ 113.920877] last sysfs file: /sys/devices/pci0000:00/0000:00:04.0/host0/target0:0:0/0:0:0:0/block/sda/size
> > > [ 113.920900] Dumping ftrace buffer:
> > > [ 113.920919] (ftrace buffer empty)
> > > [ 113.920933] Modules linked in: af_packet irnet ppp_generic slhc ircomm_tty ircomm binfmt_misc loop dm_mirror dm_region_hash dm_log dm_round_robin dm_multipath dm_mod alim15x3 ide_core nvram toshiba cryptomgr aead crypto_blkcipher michael_mic crypto_algapi orinoco_cs orinoco hermes_dld hermes pcmcia firmware_class snd_ali5451 snd_ac97_codec ac97_bus snd_seq_dummy snd_seq_oss snd_seq_midi_event snd_seq snd_seq_device smsc_ircc2 snd_pcm_oss snd_pcm rtc_cmos irda snd_timer snd_mixer_oss rtc_core snd crc_ccitt yenta_socket rtc_lib rsrc_nonstatic i2c_ali1535 pcmcia_core pcspkr psmouse soundcore i2c_core evdev sr_mod snd_page_alloc alim1535_wdt cdrom fan sg video output toshiba_acpi rfkill thermal backlight ali_agp processor ac button input_polldev battery agpgart ohci_hcd usbcore reiserfs pata_ali libata sd_mod scsi_mod [last unloaded: scsi_wait_scan]
> > > [ 113.921765]
> > > [ 113.921785] Pid: 3272, comm: ipolldevd Not tainted (2.6.28-rc2-1avb #3) PORTEGE 4000
> > > [ 113.921801] EIP: 0060:[<dfaa4683>] EFLAGS: 00010246 CPU: 0
> > > [ 113.921854] EIP is at rfkill_force_state+0x53/0x90 [rfkill]
> > > [ 113.921870] EAX: 00000000 EBX: 00000000 ECX: 00000003 EDX: 00000000
> > > [ 113.921885] ESI: 00000000 EDI: ddd50300 EBP: d8d7af40 ESP: d8d7af24
> > > [ 113.921900] DS: 007b ES: 007b FS: 0000 GS: 0000 SS: 0068
> > > [ 113.921918] Process ipolldevd (pid: 3272, ti=d8d7a000 task=d8d93c90 task.ti=d8d7a000)
> > > [ 113.921933] Stack:
> > > [ 113.921945] d8d7af38 00000246 dfb029d8 dfb029c0 dfb029d8 dfb029c0 ddd50300 d8d7af5c
> > > [ 113.922014] dfb018e2 01000246 01000000 ddd50300 ddd50314 ddabb8a0 d8d7af68 dfb381c1
> > > [ 113.922098] 00000000 d8d7afa4 c012ec0a 00000000 00000002 00000000 c012eba8 ddabb8c0
> > > [ 113.922240] Call Trace:
> > > [ 113.922240] [<dfb018e2>] ? bt_poll_rfkill+0x5c/0x82 [toshiba_acpi]
> > > [ 113.922240] [<dfb381c1>] ? input_polled_device_work+0x11/0x40 [input_polldev]
> > > [ 113.922240] [<c012ec0a>] ? run_workqueue+0xea/0x1f0
> > > [ 113.922240] [<c012eba8>] ? run_workqueue+0x88/0x1f0
> > > [ 113.922240] [<dfb381b0>] ? input_polled_device_work+0x0/0x40 [input_polldev]
> > > [ 113.922240] [<c012f047>] ? worker_thread+0x87/0xf0
> > > [ 113.922240] [<c0132b00>] ? autoremove_wake_function+0x0/0x50
> > > [ 113.922240] [<c012efc0>] ? worker_thread+0x0/0xf0
> > > [ 113.922240] [<c013280f>] ? kthread+0x3f/0x80
> > > [ 113.922240] [<c01327d0>] ? kthread+0x0/0x80
> > > [ 113.922240] [<c01040d7>] ? kernel_thread_helper+0x7/0x10
> > > [ 113.922240] Code: 43 54 89 73 54 39 c6 74 11 89 d9 ba 01 00 00 00 b8 40 68 aa df e8 3e 35 69 e0 89 f8 e8 77 fd 85 e0 31 c0 83 c4 10 5b 5e 5f 5d c3 <0f> 0b eb fe 89 f6 8d bc 27 00 00 00 00 be f4 4d aa df bb 5f 01
> > > [ 113.922240] EIP: [<dfaa4683>] rfkill_force_state+0x53/0x90 [rfkill] SS:ESP 0068:d8d7af24
> > > [ 113.924700] ---[ end trace 0e404eb40cadd5f0 ]---
> >
> > Signed-off-by: Frederik Deweerdt <frederik.deweerdt@gmail.com>
> >
> > diff --git a/drivers/acpi/toshiba_acpi.c b/drivers/acpi/toshiba_acpi.c
> > index 66aac06..c1c9d87 100644
> > --- a/drivers/acpi/toshiba_acpi.c
> > +++ b/drivers/acpi/toshiba_acpi.c
> > @@ -824,34 +824,38 @@ static int __init toshiba_acpi_init(void)
> > toshiba_acpi_exit();
> > return -ENOMEM;
> > }
> > - }
> >
> > - /* Register input device for kill switch */
> > - toshiba_acpi.poll_dev = input_allocate_polled_device();
> > - if (!toshiba_acpi.poll_dev) {
> > - printk(MY_ERR "unable to allocate kill-switch input device\n");
> > - toshiba_acpi_exit();
> > - return -ENOMEM;
> > - }
> > - toshiba_acpi.poll_dev->private = &toshiba_acpi;
> > - toshiba_acpi.poll_dev->poll = bt_poll_rfkill;
> > - toshiba_acpi.poll_dev->poll_interval = 1000; /* msecs */
> > -
> > - toshiba_acpi.poll_dev->input->name = toshiba_acpi.rfk_name;
> > - toshiba_acpi.poll_dev->input->id.bustype = BUS_HOST;
> > - toshiba_acpi.poll_dev->input->id.vendor = 0x0930; /* Toshiba USB ID */
> > - set_bit(EV_SW, toshiba_acpi.poll_dev->input->evbit);
> > - set_bit(SW_RFKILL_ALL, toshiba_acpi.poll_dev->input->swbit);
> > - input_report_switch(toshiba_acpi.poll_dev->input, SW_RFKILL_ALL, TRUE);
> > - input_sync(toshiba_acpi.poll_dev->input);
> > -
> > - ret = input_register_polled_device(toshiba_acpi.poll_dev);
> > - if (ret) {
> > - printk(MY_ERR "unable to register kill-switch input device\n");
> > - rfkill_free(toshiba_acpi.rfk_dev);
> > - toshiba_acpi.rfk_dev = NULL;
> > - toshiba_acpi_exit();
> > - return ret;
> > + /* Register input device for kill switch */
> > + toshiba_acpi.poll_dev = input_allocate_polled_device();
> > + if (!toshiba_acpi.poll_dev) {
> > + printk(MY_ERR
> > + "unable to allocate kill-switch input device\n");
> > + toshiba_acpi_exit();
> > + return -ENOMEM;
> > + }
> > + toshiba_acpi.poll_dev->private = &toshiba_acpi;
> > + toshiba_acpi.poll_dev->poll = bt_poll_rfkill;
> > + toshiba_acpi.poll_dev->poll_interval = 1000; /* msecs */
> > +
> > + toshiba_acpi.poll_dev->input->name = toshiba_acpi.rfk_name;
> > + toshiba_acpi.poll_dev->input->id.bustype = BUS_HOST;
> > + /* Toshiba USB ID */
> > + toshiba_acpi.poll_dev->input->id.vendor = 0x0930;
> > + set_bit(EV_SW, toshiba_acpi.poll_dev->input->evbit);
> > + set_bit(SW_RFKILL_ALL, toshiba_acpi.poll_dev->input->swbit);
> > + input_report_switch(toshiba_acpi.poll_dev->input,
> > + SW_RFKILL_ALL, TRUE);
> > + input_sync(toshiba_acpi.poll_dev->input);
> > +
> > + ret = input_register_polled_device(toshiba_acpi.poll_dev);
> > + if (ret) {
> > + printk(MY_ERR
> > + "unable to register kill-switch input device\n");
> > + rfkill_free(toshiba_acpi.rfk_dev);
> > + toshiba_acpi.rfk_dev = NULL;
> > + toshiba_acpi_exit();
> > + return ret;
> > + }
> > }
> >
> > return 0;
> >
> > ----- End forwarded message -----
> >
> >
>
>
>
--
Everyone knows that debugging is twice as hard as writing a program
in the first place. So if you're as clever as you can be when you write it,
how will you ever debug it? --- Brian Kernighan
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [patch] ACPI toshiba: only register rfkill if bt is enabled
2008-11-16 12:40 ` Rafael J. Wysocki
@ 2008-11-16 12:50 ` Andrey Borzenkov
2008-11-16 16:44 ` Rafael J. Wysocki
0 siblings, 1 reply; 12+ messages in thread
From: Andrey Borzenkov @ 2008-11-16 12:50 UTC (permalink / raw)
To: Rafael J. Wysocki
Cc: Frederik Deweerdt, linux-acpi, linux-kernel, akpm, philipl
[-- Attachment #1: Type: text/plain, Size: 976 bytes --]
On Sunday 16 November 2008, Rafael J. Wysocki wrote:
> On Sunday, 16 of November 2008, Andrey Borzenkov wrote:
> > On Wednesday 29 October 2008, Frederik Deweerdt wrote:
> > > [resending as it didn't make it to the list, removing netdev from cc
> > > also]
> > > Hello Andrey,
> > >
> > > It looks like a part of the rfkill initialization was done whenever
> > > BT was on or not. The following patch checks for BT presence before
> > > registering the rfkill to the input layer. Does it solve the problem for
> > > you?
> > >
> >
> > This patch fixes regression in 2.6.28. Patch has been tested and
> > acnowledged. Rafael, could you add it to list of open regressions?
> > Thank you!
>
> I will, if you provide me with a link to the patch and (optionally) to the
> original bug report.
>
Original bug report:
http://marc.info/?l=linux-kernel&m=122522113619025&w=2
Patch:
http://marc.info/?l=linux-kernel&m=122526843117478&w=2
Thank you!
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 197 bytes --]
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [patch] ACPI toshiba: only register rfkill if bt is enabled
2008-11-16 12:50 ` Andrey Borzenkov
@ 2008-11-16 16:44 ` Rafael J. Wysocki
0 siblings, 0 replies; 12+ messages in thread
From: Rafael J. Wysocki @ 2008-11-16 16:44 UTC (permalink / raw)
To: Andrey Borzenkov
Cc: Frederik Deweerdt, linux-acpi, linux-kernel, akpm, philipl
On Sunday, 16 of November 2008, Andrey Borzenkov wrote:
> On Sunday 16 November 2008, Rafael J. Wysocki wrote:
> > On Sunday, 16 of November 2008, Andrey Borzenkov wrote:
> > > On Wednesday 29 October 2008, Frederik Deweerdt wrote:
> > > > [resending as it didn't make it to the list, removing netdev from cc
> > > > also]
> > > > Hello Andrey,
> > > >
> > > > It looks like a part of the rfkill initialization was done whenever
> > > > BT was on or not. The following patch checks for BT presence before
> > > > registering the rfkill to the input layer. Does it solve the problem for
> > > > you?
> > > >
> > >
> > > This patch fixes regression in 2.6.28. Patch has been tested and
> > > acnowledged. Rafael, could you add it to list of open regressions?
> > > Thank you!
> >
> > I will, if you provide me with a link to the patch and (optionally) to the
> > original bug report.
> >
>
> Original bug report:
> http://marc.info/?l=linux-kernel&m=122522113619025&w=2
>
> Patch:
> http://marc.info/?l=linux-kernel&m=122526843117478&w=2
Thanks, added to the list as http://bugzilla.kernel.org/show_bug.cgi?id=12047
Best,
Rafael
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [patch] ACPI toshiba: only register rfkill if bt is enabled
2008-10-29 8:19 [patch] ACPI toshiba: only register rfkill if bt is enabled Frederik Deweerdt
` (2 preceding siblings ...)
2008-11-16 8:28 ` Andrey Borzenkov
@ 2008-12-01 19:30 ` Andrew Morton
2008-12-01 19:40 ` Arjan van de Ven
3 siblings, 1 reply; 12+ messages in thread
From: Andrew Morton @ 2008-12-01 19:30 UTC (permalink / raw)
To: Frederik Deweerdt
Cc: arvidjaar, linux-kernel, philipl, linux-acpi, Arjan van de Ven
This patch:
On Wed, 29 Oct 2008 08:19:34 +0000
Frederik Deweerdt <frederik.deweerdt@xprog.eu> wrote:
> Register toshiba RFKill Switch only if bluetooth is enabled
which is here: http://userweb.kernel.org/~akpm/mmotm/broken-out/acpi-toshiba-only-register-rfkill-if-bt-is-enabled.patch
fixes a post-2.6.27 regression. It doesn't apply any more, due to the
recent merge of this:
commit e251fa12fb5713e722029341c03b2f48bcd7b7be
Author: Arjan van de Ven <arjan@linux.intel.com>
Date: Fri Nov 28 08:19:09 2008 -0800
toshiba_acpi: close race in toshiba_acpi driver
in linux-next.
It is unclear to me whether these two patches fix the same bug, or
different ones.
Could you please take a look in linux-next and see what needs to be
done?
Thanks.
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [patch] ACPI toshiba: only register rfkill if bt is enabled
2008-12-01 19:30 ` Andrew Morton
@ 2008-12-01 19:40 ` Arjan van de Ven
2008-12-01 23:52 ` Frederik Deweerdt
0 siblings, 1 reply; 12+ messages in thread
From: Arjan van de Ven @ 2008-12-01 19:40 UTC (permalink / raw)
To: Andrew Morton
Cc: Frederik Deweerdt, arvidjaar, linux-kernel, philipl, linux-acpi
On Mon, 1 Dec 2008 11:30:52 -0800
Andrew Morton <akpm@linux-foundation.org> wrote:
> This patch:
>
> On Wed, 29 Oct 2008 08:19:34 +0000
> Frederik Deweerdt <frederik.deweerdt@xprog.eu> wrote:
>
> > Register toshiba RFKill Switch only if bluetooth is enabled
>
> which is here:
> http://userweb.kernel.org/~akpm/mmotm/broken-out/acpi-toshiba-only-register-rfkill-if-bt-is-enabled.patch
>
> fixes a post-2.6.27 regression. It doesn't apply any more, due to the
> recent merge of this:
>
> commit e251fa12fb5713e722029341c03b2f48bcd7b7be
> Author: Arjan van de Ven <arjan@linux.intel.com>
> Date: Fri Nov 28 08:19:09 2008 -0800
>
> toshiba_acpi: close race in toshiba_acpi driver
>
> in linux-next.
>
> It is unclear to me whether these two patches fix the same bug, or
> different ones.
>
they fix the same symptom, but are both needed.
My patch fixes "if it goes wrong and init fails, don't oops", the mmotm
patch fixes "lets not make init fail for <this case>". If there's some
other reason the init fails, my patch is still needed.
--
Arjan van de Ven Intel Open Source Technology Centre
For development, discussion and tips for power savings,
visit http://www.lesswatts.org
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [patch] ACPI toshiba: only register rfkill if bt is enabled
2008-12-01 19:40 ` Arjan van de Ven
@ 2008-12-01 23:52 ` Frederik Deweerdt
0 siblings, 0 replies; 12+ messages in thread
From: Frederik Deweerdt @ 2008-12-01 23:52 UTC (permalink / raw)
To: Arjan van de Ven
Cc: Andrew Morton, arvidjaar, linux-kernel, philipl, linux-acpi
On Mon, Dec 01, 2008 at 11:40:04AM -0800, Arjan van de Ven wrote:
> they fix the same symptom, but are both needed.
> My patch fixes "if it goes wrong and init fails, don't oops", the mmotm
> patch fixes "lets not make init fail for <this case>". If there's some
> other reason the init fails, my patch is still needed.
Confirmed, both are needed. You'll find a refreshed patch attached.
Regards,
Frederik
From: Frederik Deweerdt <frederik.deweerdt@xprog.eu>
Part of the rfkill initialization was done whenever BT was on or not. The
following patch checks for BT presence before registering the rfkill to
the input layer. Some minor cleanups (> 80 char lines) were also added in
the process.
On Tue, Oct 28, 2008 at 10:10:37PM +0300, Andrey Borzenkov wrote:
[...]
> [ 66.633036] toshiba_acpi: Toshiba Laptop ACPI Extras version 0.19
> [ 66.633054] toshiba_acpi: HCI method: \_SB_.VALD.GHCI
> [ 66.637764] input: Toshiba RFKill Switch as /devices/virtual/input/input3
[...]
> [ 113.920753] ------------[ cut here ]------------
> [ 113.920828] kernel BUG at /home/bor/src/linux-git/net/rfkill/rfkill.c:347!
> [ 113.920845] invalid opcode: 0000 [#1]
> [ 113.920877] last sysfs file: /sys/devices/pci0000:00/0000:00:04.0/host0/target0:0:0/0:0:0:0/block/sda/size
> [ 113.920900] Dumping ftrace buffer:
> [ 113.920919] (ftrace buffer empty)
> [ 113.920933] Modules linked in: af_packet irnet ppp_generic slhc ircomm_tty ircomm binfmt_misc loop dm_mirror dm_region_hash dm_log dm_round_robin dm_multipath dm_mod alim15x3 ide_core nvram toshiba cryptomgr aead crypto_blkcipher michael_mic crypto_algapi orinoco_cs orinoco hermes_dld hermes pcmcia firmware_class snd_ali5451 snd_ac97_codec ac97_bus snd_seq_dummy snd_seq_oss snd_seq_midi_event snd_seq snd_seq_device smsc_ircc2 snd_pcm_oss snd_pcm rtc_cmos irda snd_timer snd_mixer_oss rtc_core snd crc_ccitt yenta_socket rtc_lib rsrc_nonstatic i2c_ali1535 pcmcia_core pcspkr psmouse soundcore i2c_core evdev sr_mod snd_page_alloc alim1535_wdt cdrom fan sg video output toshiba_acpi rfkill thermal backlight ali_agp processor ac button input_polldev battery agpgart ohci_hcd usbcore reiserfs pata_ali libata sd_mod scsi_mod [last unloaded: scsi_wait_scan]
> [ 113.921765]
> [ 113.921785] Pid: 3272, comm: ipolldevd Not tainted (2.6.28-rc2-1avb #3) PORTEGE 4000
> [ 113.921801] EIP: 0060:[<dfaa4683>] EFLAGS: 00010246 CPU: 0
> [ 113.921854] EIP is at rfkill_force_state+0x53/0x90 [rfkill]
> [ 113.921870] EAX: 00000000 EBX: 00000000 ECX: 00000003 EDX: 00000000
> [ 113.921885] ESI: 00000000 EDI: ddd50300 EBP: d8d7af40 ESP: d8d7af24
> [ 113.921900] DS: 007b ES: 007b FS: 0000 GS: 0000 SS: 0068
> [ 113.921918] Process ipolldevd (pid: 3272, ti=d8d7a000 task=d8d93c90 task.ti=d8d7a000)
> [ 113.921933] Stack:
> [ 113.921945] d8d7af38 00000246 dfb029d8 dfb029c0 dfb029d8 dfb029c0 ddd50300 d8d7af5c
> [ 113.922014] dfb018e2 01000246 01000000 ddd50300 ddd50314 ddabb8a0 d8d7af68 dfb381c1
> [ 113.922098] 00000000 d8d7afa4 c012ec0a 00000000 00000002 00000000 c012eba8 ddabb8c0
> [ 113.922240] Call Trace:
> [ 113.922240] [<dfb018e2>] ? bt_poll_rfkill+0x5c/0x82 [toshiba_acpi]
> [ 113.922240] [<dfb381c1>] ? input_polled_device_work+0x11/0x40 [input_polldev]
> [ 113.922240] [<c012ec0a>] ? run_workqueue+0xea/0x1f0
> [ 113.922240] [<c012eba8>] ? run_workqueue+0x88/0x1f0
> [ 113.922240] [<dfb381b0>] ? input_polled_device_work+0x0/0x40 [input_polldev]
> [ 113.922240] [<c012f047>] ? worker_thread+0x87/0xf0
> [ 113.922240] [<c0132b00>] ? autoremove_wake_function+0x0/0x50
> [ 113.922240] [<c012efc0>] ? worker_thread+0x0/0xf0
> [ 113.922240] [<c013280f>] ? kthread+0x3f/0x80
> [ 113.922240] [<c01327d0>] ? kthread+0x0/0x80
> [ 113.922240] [<c01040d7>] ? kernel_thread_helper+0x7/0x10
> [ 113.922240] Code: 43 54 89 73 54 39 c6 74 11 89 d9 ba 01 00 00 00 b8 40 68 aa df e8 3e 35 69 e0 89 f8 e8 77 fd 85 e0 31 c0 83 c4 10 5b 5e 5f 5d c3 <0f> 0b eb fe 89 f6 8d bc 27 00 00 00 00 be f4 4d aa df bb 5f 01
> [ 113.922240] EIP: [<dfaa4683>] rfkill_force_state+0x53/0x90 [rfkill] SS:ESP 0068:d8d7af24
> [ 113.924700] ---[ end trace 0e404eb40cadd5f0 ]---
Signed-off-by: Frederik Deweerdt <frederik.deweerdt@gmail.com>
Tested-by: Andrey Borzenkov <arvidjaar@mail.ru>
Cc: Len Brown <len.brown@intel.com>
Cc: Richard Purdie <rpurdie@rpsys.net>
Acked-by: Philip Langdale <philipl@overt.org>
diff --git a/drivers/acpi/toshiba_acpi.c b/drivers/acpi/toshiba_acpi.c
index 25f531d..40e60fc 100644
--- a/drivers/acpi/toshiba_acpi.c
+++ b/drivers/acpi/toshiba_acpi.c
@@ -824,32 +824,36 @@ static int __init toshiba_acpi_init(void)
toshiba_acpi_exit();
return -ENOMEM;
}
- }
- /* Register input device for kill switch */
- toshiba_acpi.poll_dev = input_allocate_polled_device();
- if (!toshiba_acpi.poll_dev) {
- printk(MY_ERR "unable to allocate kill-switch input device\n");
- toshiba_acpi_exit();
- return -ENOMEM;
- }
- toshiba_acpi.poll_dev->private = &toshiba_acpi;
- toshiba_acpi.poll_dev->poll = bt_poll_rfkill;
- toshiba_acpi.poll_dev->poll_interval = 1000; /* msecs */
-
- toshiba_acpi.poll_dev->input->name = toshiba_acpi.rfk_name;
- toshiba_acpi.poll_dev->input->id.bustype = BUS_HOST;
- toshiba_acpi.poll_dev->input->id.vendor = 0x0930; /* Toshiba USB ID */
- set_bit(EV_SW, toshiba_acpi.poll_dev->input->evbit);
- set_bit(SW_RFKILL_ALL, toshiba_acpi.poll_dev->input->swbit);
- input_report_switch(toshiba_acpi.poll_dev->input, SW_RFKILL_ALL, TRUE);
- input_sync(toshiba_acpi.poll_dev->input);
-
- ret = input_register_polled_device(toshiba_acpi.poll_dev);
- if (ret) {
- printk(MY_ERR "unable to register kill-switch input device\n");
- toshiba_acpi_exit();
- return ret;
+ /* Register input device for kill switch */
+ toshiba_acpi.poll_dev = input_allocate_polled_device();
+ if (!toshiba_acpi.poll_dev) {
+ printk(MY_ERR
+ "unable to allocate kill-switch input device\n");
+ toshiba_acpi_exit();
+ return -ENOMEM;
+ }
+ toshiba_acpi.poll_dev->private = &toshiba_acpi;
+ toshiba_acpi.poll_dev->poll = bt_poll_rfkill;
+ toshiba_acpi.poll_dev->poll_interval = 1000; /* msecs */
+
+ toshiba_acpi.poll_dev->input->name = toshiba_acpi.rfk_name;
+ toshiba_acpi.poll_dev->input->id.bustype = BUS_HOST;
+ /* Toshiba USB ID */
+ toshiba_acpi.poll_dev->input->id.vendor = 0x0930;
+ set_bit(EV_SW, toshiba_acpi.poll_dev->input->evbit);
+ set_bit(SW_RFKILL_ALL, toshiba_acpi.poll_dev->input->swbit);
+ input_report_switch(toshiba_acpi.poll_dev->input,
+ SW_RFKILL_ALL, TRUE);
+ input_sync(toshiba_acpi.poll_dev->input);
+
+ ret = input_register_polled_device(toshiba_acpi.poll_dev);
+ if (ret) {
+ printk(MY_ERR
+ "unable to register kill-switch input device\n");
+ toshiba_acpi_exit();
+ return ret;
+ }
}
return 0;
^ permalink raw reply related [flat|nested] 12+ messages in thread
end of thread, other threads:[~2008-12-01 23:52 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-10-29 8:19 [patch] ACPI toshiba: only register rfkill if bt is enabled Frederik Deweerdt
2008-10-29 13:10 ` Frans Pop
2008-10-29 15:30 ` Philip Langdale
2008-10-29 17:05 ` Andrey Borzenkov
2008-11-16 8:28 ` Andrey Borzenkov
2008-11-16 12:40 ` Rafael J. Wysocki
2008-11-16 12:50 ` Andrey Borzenkov
2008-11-16 16:44 ` Rafael J. Wysocki
2008-12-01 19:30 ` Andrew Morton
2008-12-01 19:40 ` Arjan van de Ven
2008-12-01 23:52 ` Frederik Deweerdt
-- strict thread matches above, loose matches on Subject: below --
2008-10-28 19:10 2.6.28-rc2 - kernel BUG at /home/bor/src/linux-git/net/rfkill/rfkill.c:347! Andrey Borzenkov
2008-10-28 23:51 ` [patch] ACPI toshiba: only register rfkill if bt is enabled Frederik Deweerdt
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox