Re: [BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-18 Thread Rafael J. Wysocki
On Wednesday, June 19, 2013 12:03:37 AM Jiang Liu wrote: > On 06/17/2013 07:40 PM, Rafael J. Wysocki wrote: > > On Monday, June 17, 2013 01:12:00 AM Jiang Liu wrote: > >> On 06/16/2013 06:54 AM, Rafael J. Wysocki wrote: > >>> On Saturday, June 15, 2013 11:20:40 PM Rafael J. Wysocki wrote: >

Re: [BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-18 Thread Rafael J. Wysocki
On Tuesday, June 18, 2013 11:36:50 PM Jiang Liu wrote: > On 06/17/2013 07:39 PM, Rafael J. Wysocki wrote: > > On Monday, June 17, 2013 01:01:51 AM Jiang Liu wrote: > >> On 06/16/2013 05:20 AM, Rafael J. Wysocki wrote: > >>> On Saturday, June 15, 2013 10:17:42 PM Rafael J. Wysocki wrote: > On

Re: [BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-18 Thread Jiang Liu
On 06/17/2013 07:40 PM, Rafael J. Wysocki wrote: > On Monday, June 17, 2013 01:12:00 AM Jiang Liu wrote: >> On 06/16/2013 06:54 AM, Rafael J. Wysocki wrote: >>> On Saturday, June 15, 2013 11:20:40 PM Rafael J. Wysocki wrote: On Saturday, June 15, 2013 10:17:42 PM Rafael J. Wysocki wrote: >>>

Re: [BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-18 Thread Jiang Liu
On 06/17/2013 07:39 PM, Rafael J. Wysocki wrote: > On Monday, June 17, 2013 01:01:51 AM Jiang Liu wrote: >> On 06/16/2013 05:20 AM, Rafael J. Wysocki wrote: >>> On Saturday, June 15, 2013 10:17:42 PM Rafael J. Wysocki wrote: On Saturday, June 15, 2013 09:44:28 AM Jiang Liu wrote: >> [...]

Re: [BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-18 Thread Jiang Liu
On 06/17/2013 07:39 PM, Rafael J. Wysocki wrote: On Monday, June 17, 2013 01:01:51 AM Jiang Liu wrote: On 06/16/2013 05:20 AM, Rafael J. Wysocki wrote: On Saturday, June 15, 2013 10:17:42 PM Rafael J. Wysocki wrote: On Saturday, June 15, 2013 09:44:28 AM Jiang Liu wrote: [...] When it

Re: [BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-18 Thread Jiang Liu
On 06/17/2013 07:40 PM, Rafael J. Wysocki wrote: On Monday, June 17, 2013 01:12:00 AM Jiang Liu wrote: On 06/16/2013 06:54 AM, Rafael J. Wysocki wrote: On Saturday, June 15, 2013 11:20:40 PM Rafael J. Wysocki wrote: On Saturday, June 15, 2013 10:17:42 PM Rafael J. Wysocki wrote: [...]

Re: [BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-18 Thread Rafael J. Wysocki
On Tuesday, June 18, 2013 11:36:50 PM Jiang Liu wrote: On 06/17/2013 07:39 PM, Rafael J. Wysocki wrote: On Monday, June 17, 2013 01:01:51 AM Jiang Liu wrote: On 06/16/2013 05:20 AM, Rafael J. Wysocki wrote: On Saturday, June 15, 2013 10:17:42 PM Rafael J. Wysocki wrote: On Saturday, June

Re: [BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-18 Thread Rafael J. Wysocki
On Wednesday, June 19, 2013 12:03:37 AM Jiang Liu wrote: On 06/17/2013 07:40 PM, Rafael J. Wysocki wrote: On Monday, June 17, 2013 01:12:00 AM Jiang Liu wrote: On 06/16/2013 06:54 AM, Rafael J. Wysocki wrote: On Saturday, June 15, 2013 11:20:40 PM Rafael J. Wysocki wrote: On Saturday,

Re: [BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-17 Thread Rafael J. Wysocki
On Monday, June 17, 2013 01:39:04 PM Rafael J. Wysocki wrote: > On Monday, June 17, 2013 01:01:51 AM Jiang Liu wrote: > > On 06/16/2013 05:20 AM, Rafael J. Wysocki wrote: > > > On Saturday, June 15, 2013 10:17:42 PM Rafael J. Wysocki wrote: > > >> On Saturday, June 15, 2013 09:44:28 AM Jiang Liu

Re: [BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-17 Thread Rafael J. Wysocki
On Monday, June 17, 2013 01:12:00 AM Jiang Liu wrote: > On 06/16/2013 06:54 AM, Rafael J. Wysocki wrote: > > On Saturday, June 15, 2013 11:20:40 PM Rafael J. Wysocki wrote: > >> On Saturday, June 15, 2013 10:17:42 PM Rafael J. Wysocki wrote: > > > > [...] > > > >> > >> Which sysfs interfaces do

Re: [BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-17 Thread Rafael J. Wysocki
On Monday, June 17, 2013 01:01:51 AM Jiang Liu wrote: > On 06/16/2013 05:20 AM, Rafael J. Wysocki wrote: > > On Saturday, June 15, 2013 10:17:42 PM Rafael J. Wysocki wrote: > >> On Saturday, June 15, 2013 09:44:28 AM Jiang Liu wrote: > [...] > >> When it returns from

Re: [BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-17 Thread Rafael J. Wysocki
On Monday, June 17, 2013 01:01:51 AM Jiang Liu wrote: On 06/16/2013 05:20 AM, Rafael J. Wysocki wrote: On Saturday, June 15, 2013 10:17:42 PM Rafael J. Wysocki wrote: On Saturday, June 15, 2013 09:44:28 AM Jiang Liu wrote: [...] When it returns from unregister_hotplug_dock_device(),

Re: [BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-17 Thread Rafael J. Wysocki
On Monday, June 17, 2013 01:12:00 AM Jiang Liu wrote: On 06/16/2013 06:54 AM, Rafael J. Wysocki wrote: On Saturday, June 15, 2013 11:20:40 PM Rafael J. Wysocki wrote: On Saturday, June 15, 2013 10:17:42 PM Rafael J. Wysocki wrote: [...] Which sysfs interfaces do you mean, by the

Re: [BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-17 Thread Rafael J. Wysocki
On Monday, June 17, 2013 01:39:04 PM Rafael J. Wysocki wrote: On Monday, June 17, 2013 01:01:51 AM Jiang Liu wrote: On 06/16/2013 05:20 AM, Rafael J. Wysocki wrote: On Saturday, June 15, 2013 10:17:42 PM Rafael J. Wysocki wrote: On Saturday, June 15, 2013 09:44:28 AM Jiang Liu wrote:

Re: [BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-16 Thread Jiang Liu
On 06/16/2013 06:54 AM, Rafael J. Wysocki wrote: > On Saturday, June 15, 2013 11:20:40 PM Rafael J. Wysocki wrote: >> On Saturday, June 15, 2013 10:17:42 PM Rafael J. Wysocki wrote: > > [...] > >> >> Which sysfs interfaces do you mean, by the way? >> >> If you mean "eject", then it takes

Re: [BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-16 Thread Jiang Liu
On 06/16/2013 05:20 AM, Rafael J. Wysocki wrote: > On Saturday, June 15, 2013 10:17:42 PM Rafael J. Wysocki wrote: >> On Saturday, June 15, 2013 09:44:28 AM Jiang Liu wrote: [...] >> When it returns from unregister_hotplug_dock_device(), nothing prevents it >> from accessing whatever it wants,

Re: [BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-16 Thread Jiang Liu
On 06/16/2013 04:17 AM, Rafael J. Wysocki wrote: > On Saturday, June 15, 2013 09:44:28 AM Jiang Liu wrote: >> On Sat 15 Jun 2013 06:21:02 AM CST, Rafael J. Wysocki wrote: [...] >>> Can we please relax a bit and possibly take a step back? >>> >>> So since your last reply to me wasn't particularly

Re: [BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-16 Thread Jiang Liu
On 06/16/2013 04:17 AM, Rafael J. Wysocki wrote: On Saturday, June 15, 2013 09:44:28 AM Jiang Liu wrote: On Sat 15 Jun 2013 06:21:02 AM CST, Rafael J. Wysocki wrote: [...] Can we please relax a bit and possibly take a step back? So since your last reply to me wasn't particularly helpful, I

Re: [BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-16 Thread Jiang Liu
On 06/16/2013 05:20 AM, Rafael J. Wysocki wrote: On Saturday, June 15, 2013 10:17:42 PM Rafael J. Wysocki wrote: On Saturday, June 15, 2013 09:44:28 AM Jiang Liu wrote: [...] When it returns from unregister_hotplug_dock_device(), nothing prevents it from accessing whatever it wants, because

Re: [BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-16 Thread Jiang Liu
On 06/16/2013 06:54 AM, Rafael J. Wysocki wrote: On Saturday, June 15, 2013 11:20:40 PM Rafael J. Wysocki wrote: On Saturday, June 15, 2013 10:17:42 PM Rafael J. Wysocki wrote: [...] Which sysfs interfaces do you mean, by the way? If you mean eject, then it takes acpi_scan_lock and

Re: [BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-15 Thread Rafael J. Wysocki
On Saturday, June 15, 2013 11:20:40 PM Rafael J. Wysocki wrote: > On Saturday, June 15, 2013 10:17:42 PM Rafael J. Wysocki wrote: [...] > > Which sysfs interfaces do you mean, by the way? > > If you mean "eject", then it takes acpi_scan_lock and hotplug_dock_devices() > should always be run

Re: [BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-15 Thread Rafael J. Wysocki
On Saturday, June 15, 2013 10:17:42 PM Rafael J. Wysocki wrote: > On Saturday, June 15, 2013 09:44:28 AM Jiang Liu wrote: > > On Sat 15 Jun 2013 06:21:02 AM CST, Rafael J. Wysocki wrote: > > > On Saturday, June 15, 2013 03:27:59 AM Jiang Liu wrote: > > >> This is a preparation for next patch to

Re: [BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-15 Thread Rafael J. Wysocki
On Saturday, June 15, 2013 09:44:28 AM Jiang Liu wrote: > On Sat 15 Jun 2013 06:21:02 AM CST, Rafael J. Wysocki wrote: > > On Saturday, June 15, 2013 03:27:59 AM Jiang Liu wrote: > >> This is a preparation for next patch to avoid breaking bisecting. > >> If next patch is applied without this one,

Re: [BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-15 Thread Rafael J. Wysocki
On Saturday, June 15, 2013 09:44:28 AM Jiang Liu wrote: On Sat 15 Jun 2013 06:21:02 AM CST, Rafael J. Wysocki wrote: On Saturday, June 15, 2013 03:27:59 AM Jiang Liu wrote: This is a preparation for next patch to avoid breaking bisecting. If next patch is applied without this one, it will

Re: [BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-15 Thread Rafael J. Wysocki
On Saturday, June 15, 2013 10:17:42 PM Rafael J. Wysocki wrote: On Saturday, June 15, 2013 09:44:28 AM Jiang Liu wrote: On Sat 15 Jun 2013 06:21:02 AM CST, Rafael J. Wysocki wrote: On Saturday, June 15, 2013 03:27:59 AM Jiang Liu wrote: This is a preparation for next patch to avoid

Re: [BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-15 Thread Rafael J. Wysocki
On Saturday, June 15, 2013 11:20:40 PM Rafael J. Wysocki wrote: On Saturday, June 15, 2013 10:17:42 PM Rafael J. Wysocki wrote: [...] Which sysfs interfaces do you mean, by the way? If you mean eject, then it takes acpi_scan_lock and hotplug_dock_devices() should always be run under

Re: [BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-14 Thread Jiang Liu
On Sat 15 Jun 2013 06:21:02 AM CST, Rafael J. Wysocki wrote: > On Saturday, June 15, 2013 03:27:59 AM Jiang Liu wrote: >> This is a preparation for next patch to avoid breaking bisecting. >> If next patch is applied without this one, it will cause deadlock >> as below: >> >> Case 1: >> [

Re: [BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-14 Thread Rafael J. Wysocki
On Saturday, June 15, 2013 03:27:59 AM Jiang Liu wrote: > This is a preparation for next patch to avoid breaking bisecting. > If next patch is applied without this one, it will cause deadlock > as below: > > Case 1: > [ 31.015593] Possible unsafe locking scenario: > > [ 31.018350]

[BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-14 Thread Jiang Liu
This is a preparation for next patch to avoid breaking bisecting. If next patch is applied without this one, it will cause deadlock as below: Case 1: [ 31.015593] Possible unsafe locking scenario: [ 31.018350]CPU0CPU1 [ 31.019691]

[BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-14 Thread Jiang Liu
This is a preparation for next patch to avoid breaking bisecting. If next patch is applied without this one, it will cause deadlock as below: Case 1: [ 31.015593] Possible unsafe locking scenario: [ 31.018350]CPU0CPU1 [ 31.019691]

Re: [BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-14 Thread Rafael J. Wysocki
On Saturday, June 15, 2013 03:27:59 AM Jiang Liu wrote: This is a preparation for next patch to avoid breaking bisecting. If next patch is applied without this one, it will cause deadlock as below: Case 1: [ 31.015593] Possible unsafe locking scenario: [ 31.018350]CPU0

Re: [BUGFIX v2 2/4] ACPI, DOCK: resolve possible deadlock scenarios

2013-06-14 Thread Jiang Liu
On Sat 15 Jun 2013 06:21:02 AM CST, Rafael J. Wysocki wrote: On Saturday, June 15, 2013 03:27:59 AM Jiang Liu wrote: This is a preparation for next patch to avoid breaking bisecting. If next patch is applied without this one, it will cause deadlock as below: Case 1: [ 31.015593] Possible