[Bug 8498], [Bug 8510], and Re: Can't s2ram 22-rc2

All of lore.kernel.org
 help / color / mirror / Atom feed

* [Bug 8498], [Bug 8510], and Re: Can't s2ram 22-rc2
       [not found] <20070519164504.GA10675@kroah.com>
@ 2007-05-21 14:51 ` Alan Stern
  2007-05-21 14:51 ` Alan Stern
  1 sibling, 0 replies; 8+ messages in thread
From: Alan Stern @ 2007-05-21 14:51 UTC (permalink / raw)
  To: Andrey Borzenkov, Soeren Sonnenburg, Avuton Olrich, Greg KH
  Cc: Jiri Slaby, Kernel development list, Andrew Morton,
	Linux-pm mailing list, USB development list

On Sat, 19 May 2007, Greg KH wrote:

> On Sat, May 19, 2007 at 11:16:44AM -0400, Alan Stern wrote:
> > Greg:
> > 
> > The patch in $SUBJECT (already in your development tree) fixes a problem
> > with system suspend in 2.6.22-rc1, as described in
> > 
> > 	http://bugzilla.kernel.org/show_bug.cgi?id=8498
> > 
> > (For people on the PM development list, it's worth pointing out that all
> > the patch does is make a particular kernel thread freezable.)
> > 
> > Can you please expedite getting it sent up to Linus?
> 
> Will do, I have a number of USB bugfixes to go to Linus, just been
> traveling too much.  Next week I'll be able to get to it...


On Sun, 20 May 2007, Andrey Borzenkov wrote:

> No. Unfortunately. Here is dmesg with patch and USB_DEBUG
> 
> swsusp: Marking nosave pages: 000000000009f000 - 0000000000100000
> swsusp: Basic memory bitmaps created
> Stopping tasks ... done.
> Shrinking memory... done (61536 pages freed)
> Freed 246144 kbytes in 1.99 seconds (123.69 MB/s)
> Suspending console(s)
> hub 1-0:1.0: hub_suspend
> ohci_hcd 0000:00:02.0: suspend root hub
> usb usb1: usb suspend
> usb usb1: usb resume
> usb usb1: finish resume
> hub 1-0:1.0: hub_resume
> ohci_hcd 0000:00:02.0: wakeup root hub
> sd 0:0:0:0: [sda] Synchronizing SCSI cache
> pnp: Device 00:09 disabled.
> Trying to free already-free IRQ 11
> ACPI: PCI interrupt for device 0000:00:06.0 disabled
> ACPI: Unable to derive IRQ for device 0000:00:04.0
> pci_device_suspend(): usb_hcd_pci_suspend+0x0/0x230 [usbcore]() returns -16
> suspend_device(): pci_device_suspend+0x0/0x70() returns -16
> Could not suspend device 0000:00:02.0: error -16


On Mon, 21 May 2007, Rafael J. Wysocki wrote:

> > pci_device_suspend(): usb_hcd_pci_suspend+0x0/0x170() returns -16
> > suspend_device(): pci_device_suspend+0x0/0x60() returns -16
> > Could not suspend device 0000:00:1d.0: error -16
> > 
> > 00:1d.0 USB Controller: Intel Corporation 82801G (ICH7 Family) USB UHCI #1 (rev 02)
> > 
> > lspci/dmesg are in this mail:
> > http://marc.info/?l=linux-kernel&m=117973679209855&w=2
> 
> Well, I think Alan should have a look at this too.


It turns out that the patch I originally wrote to fix this is in
conflict with one of Raphael's patches (make freezeable workqueues
singlethread) already added to 2.6.22-rc2.  So here's an updated
version for that kernel.

Andrey, Soeren, and Avuton: Please try this patch with 2.6.22-rc2 or 
later and see if it fixes your problems.

Greg, if this works then I'll send it in the proper form for a patch, 
and you can use it to replace

	usb-make-the-autosuspend-workqueue-thread-freezable.patch

Alan Stern


Index: 2.6.22-rc2/drivers/usb/core/usb.c
===================================================================
--- 2.6.22-rc2.orig/drivers/usb/core/usb.c
+++ 2.6.22-rc2/drivers/usb/core/usb.c
@@ -205,7 +205,7 @@ struct device_type usb_device_type = {
 
 static int ksuspend_usb_init(void)
 {
-	ksuspend_usb_wq = create_singlethread_workqueue("ksuspend_usbd");
+	ksuspend_usb_wq = create_freezeable_workqueue("ksuspend_usbd");
 	if (!ksuspend_usb_wq)
 		return -ENOMEM;
 	return 0;

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Bug 8498], [Bug 8510], and Re: Can't s2ram 22-rc2
       [not found] <20070519164504.GA10675@kroah.com>
  2007-05-21 14:51 ` [Bug 8498], [Bug 8510], and Re: Can't s2ram 22-rc2 Alan Stern
@ 2007-05-21 14:51 ` Alan Stern
  2007-05-21 15:10   ` Soeren Sonnenburg
                     ` (5 more replies)
  1 sibling, 6 replies; 8+ messages in thread
From: Alan Stern @ 2007-05-21 14:51 UTC (permalink / raw)
  To: Andrey Borzenkov, Soeren Sonnenburg, Avuton Olrich, Greg KH
  Cc: Rafael J. Wysocki, Jiri Slaby, Andrew Morton,
	USB development list, Linux-pm mailing list,
	Kernel development list

On Sat, 19 May 2007, Greg KH wrote:

> On Sat, May 19, 2007 at 11:16:44AM -0400, Alan Stern wrote:
> > Greg:
> > 
> > The patch in $SUBJECT (already in your development tree) fixes a problem
> > with system suspend in 2.6.22-rc1, as described in
> > 
> > 	http://bugzilla.kernel.org/show_bug.cgi?id=8498
> > 
> > (For people on the PM development list, it's worth pointing out that all
> > the patch does is make a particular kernel thread freezable.)
> > 
> > Can you please expedite getting it sent up to Linus?
> 
> Will do, I have a number of USB bugfixes to go to Linus, just been
> traveling too much.  Next week I'll be able to get to it...


On Sun, 20 May 2007, Andrey Borzenkov wrote:

> No. Unfortunately. Here is dmesg with patch and USB_DEBUG
> 
> swsusp: Marking nosave pages: 000000000009f000 - 0000000000100000
> swsusp: Basic memory bitmaps created
> Stopping tasks ... done.
> Shrinking memory... done (61536 pages freed)
> Freed 246144 kbytes in 1.99 seconds (123.69 MB/s)
> Suspending console(s)
> hub 1-0:1.0: hub_suspend
> ohci_hcd 0000:00:02.0: suspend root hub
> usb usb1: usb suspend
> usb usb1: usb resume
> usb usb1: finish resume
> hub 1-0:1.0: hub_resume
> ohci_hcd 0000:00:02.0: wakeup root hub
> sd 0:0:0:0: [sda] Synchronizing SCSI cache
> pnp: Device 00:09 disabled.
> Trying to free already-free IRQ 11
> ACPI: PCI interrupt for device 0000:00:06.0 disabled
> ACPI: Unable to derive IRQ for device 0000:00:04.0
> pci_device_suspend(): usb_hcd_pci_suspend+0x0/0x230 [usbcore]() returns -16
> suspend_device(): pci_device_suspend+0x0/0x70() returns -16
> Could not suspend device 0000:00:02.0: error -16


On Mon, 21 May 2007, Rafael J. Wysocki wrote:

> > pci_device_suspend(): usb_hcd_pci_suspend+0x0/0x170() returns -16
> > suspend_device(): pci_device_suspend+0x0/0x60() returns -16
> > Could not suspend device 0000:00:1d.0: error -16
> > 
> > 00:1d.0 USB Controller: Intel Corporation 82801G (ICH7 Family) USB UHCI #1 (rev 02)
> > 
> > lspci/dmesg are in this mail:
> > http://marc.info/?l=linux-kernel&m=117973679209855&w=2
> 
> Well, I think Alan should have a look at this too.


It turns out that the patch I originally wrote to fix this is in
conflict with one of Raphael's patches (make freezeable workqueues
singlethread) already added to 2.6.22-rc2.  So here's an updated
version for that kernel.

Andrey, Soeren, and Avuton: Please try this patch with 2.6.22-rc2 or 
later and see if it fixes your problems.

Greg, if this works then I'll send it in the proper form for a patch, 
and you can use it to replace

	usb-make-the-autosuspend-workqueue-thread-freezable.patch

Alan Stern


Index: 2.6.22-rc2/drivers/usb/core/usb.c
===================================================================
--- 2.6.22-rc2.orig/drivers/usb/core/usb.c
+++ 2.6.22-rc2/drivers/usb/core/usb.c
@@ -205,7 +205,7 @@ struct device_type usb_device_type = {
 
 static int ksuspend_usb_init(void)
 {
-	ksuspend_usb_wq = create_singlethread_workqueue("ksuspend_usbd");
+	ksuspend_usb_wq = create_freezeable_workqueue("ksuspend_usbd");
 	if (!ksuspend_usb_wq)
 		return -ENOMEM;
 	return 0;


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [Bug 8498], [Bug 8510], and Re: Can't s2ram 22-rc2
  2007-05-21 14:51 ` Alan Stern
@ 2007-05-21 15:10   ` Soeren Sonnenburg
  2007-05-21 15:10   ` Soeren Sonnenburg
                     ` (4 subsequent siblings)
  5 siblings, 0 replies; 8+ messages in thread
From: Soeren Sonnenburg @ 2007-05-21 15:10 UTC (permalink / raw)
  To: Alan Stern
  Cc: Andrey Borzenkov, Avuton Olrich, Greg KH, Rafael J. Wysocki,
	Jiri Slaby, Andrew Morton, USB development list,
	Linux-pm mailing list, Kernel development list

On Mon, 2007-05-21 at 10:51 -0400, Alan Stern wrote:
> On Sat, 19 May 2007, Greg KH wrote:

> It turns out that the patch I originally wrote to fix this is in
> conflict with one of Raphael's patches (make freezeable workqueues
> singlethread) already added to 2.6.22-rc2.  So here's an updated
> version for that kernel.
> 
> Andrey, Soeren, and Avuton: Please try this patch with 2.6.22-rc2 or 
> later and see if it fixes your problems.
> 
> Greg, if this works then I'll send it in the proper form for a patch, 
> and you can use it to replace
> 
> 	usb-make-the-autosuspend-workqueue-thread-freezable.patch


works perfect, i.e. the machine survived 3 s2ram cycles and rebooted
cleanly :-))

apply! apply!

Soeren

> Alan Stern
> 
> 
> Index: 2.6.22-rc2/drivers/usb/core/usb.c
> ===================================================================
> --- 2.6.22-rc2.orig/drivers/usb/core/usb.c
> +++ 2.6.22-rc2/drivers/usb/core/usb.c
> @@ -205,7 +205,7 @@ struct device_type usb_device_type = {
>  
>  static int ksuspend_usb_init(void)
>  {
> -	ksuspend_usb_wq = create_singlethread_workqueue("ksuspend_usbd");
> +	ksuspend_usb_wq = create_freezeable_workqueue("ksuspend_usbd");
>  	if (!ksuspend_usb_wq)
>  		return -ENOMEM;
>  	return 0;
> 
-- 
Sometimes, there's a moment as you're waking, when you become aware of
the real world around you, but you're still dreaming.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [Bug 8498], [Bug 8510], and Re: Can't s2ram 22-rc2
  2007-05-21 14:51 ` Alan Stern
  2007-05-21 15:10   ` Soeren Sonnenburg
@ 2007-05-21 15:10   ` Soeren Sonnenburg
  2007-05-21 15:43   ` Andrey Borzenkov
                     ` (3 subsequent siblings)
  5 siblings, 0 replies; 8+ messages in thread
From: Soeren Sonnenburg @ 2007-05-21 15:10 UTC (permalink / raw)
  To: Alan Stern
  Cc: Jiri Slaby, Avuton Olrich, Kernel development list, Andrew Morton,
	Andrey Borzenkov, Linux-pm mailing list, USB development list

On Mon, 2007-05-21 at 10:51 -0400, Alan Stern wrote:
> On Sat, 19 May 2007, Greg KH wrote:

> It turns out that the patch I originally wrote to fix this is in
> conflict with one of Raphael's patches (make freezeable workqueues
> singlethread) already added to 2.6.22-rc2.  So here's an updated
> version for that kernel.
> 
> Andrey, Soeren, and Avuton: Please try this patch with 2.6.22-rc2 or 
> later and see if it fixes your problems.
> 
> Greg, if this works then I'll send it in the proper form for a patch, 
> and you can use it to replace
> 
> 	usb-make-the-autosuspend-workqueue-thread-freezable.patch


works perfect, i.e. the machine survived 3 s2ram cycles and rebooted
cleanly :-))

apply! apply!

Soeren

> Alan Stern
> 
> 
> Index: 2.6.22-rc2/drivers/usb/core/usb.c
> ===================================================================
> --- 2.6.22-rc2.orig/drivers/usb/core/usb.c
> +++ 2.6.22-rc2/drivers/usb/core/usb.c
> @@ -205,7 +205,7 @@ struct device_type usb_device_type = {
>  
>  static int ksuspend_usb_init(void)
>  {
> -	ksuspend_usb_wq = create_singlethread_workqueue("ksuspend_usbd");
> +	ksuspend_usb_wq = create_freezeable_workqueue("ksuspend_usbd");
>  	if (!ksuspend_usb_wq)
>  		return -ENOMEM;
>  	return 0;
> 
-- 
Sometimes, there's a moment as you're waking, when you become aware of
the real world around you, but you're still dreaming.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [Bug 8498], [Bug 8510], and Re: Can't s2ram 22-rc2
  2007-05-21 14:51 ` Alan Stern
  2007-05-21 15:10   ` Soeren Sonnenburg
  2007-05-21 15:10   ` Soeren Sonnenburg
@ 2007-05-21 15:43   ` Andrey Borzenkov
  2007-05-21 15:43   ` Andrey Borzenkov
                     ` (2 subsequent siblings)
  5 siblings, 0 replies; 8+ messages in thread
From: Andrey Borzenkov @ 2007-05-21 15:43 UTC (permalink / raw)
  To: Alan Stern
  Cc: Soeren Sonnenburg, Avuton Olrich, Greg KH, Rafael J. Wysocki,
	Jiri Slaby, Andrew Morton, USB development list,
	Linux-pm mailing list, Kernel development list

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

On Monday 21 May 2007, Alan Stern wrote:
> Andrey, Soeren, and Avuton: Please try this patch with 2.6.22-rc2 or
> later and see if it fixes your problems.
>

It does; thank you!

> Greg, if this works then I'll send it in the proper form for a patch,
> and you can use it to replace
>
> 	usb-make-the-autosuspend-workqueue-thread-freezable.patch
>
> Alan Stern
>
>
> Index: 2.6.22-rc2/drivers/usb/core/usb.c
> ===================================================================
> --- 2.6.22-rc2.orig/drivers/usb/core/usb.c
> +++ 2.6.22-rc2/drivers/usb/core/usb.c
> @@ -205,7 +205,7 @@ struct device_type usb_device_type = {
>
>  static int ksuspend_usb_init(void)
>  {
> -	ksuspend_usb_wq = create_singlethread_workqueue("ksuspend_usbd");
> +	ksuspend_usb_wq = create_freezeable_workqueue("ksuspend_usbd");
>  	if (!ksuspend_usb_wq)
>  		return -ENOMEM;
>  	return 0;



[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [Bug 8498], [Bug 8510], and Re: Can't s2ram 22-rc2
  2007-05-21 14:51 ` Alan Stern
                     ` (2 preceding siblings ...)
  2007-05-21 15:43   ` Andrey Borzenkov
@ 2007-05-21 15:43   ` Andrey Borzenkov
  2007-05-21 16:12   ` Avuton Olrich
  2007-05-21 16:12   ` Avuton Olrich
  5 siblings, 0 replies; 8+ messages in thread
From: Andrey Borzenkov @ 2007-05-21 15:43 UTC (permalink / raw)
  To: Alan Stern
  Cc: Soeren Sonnenburg, Jiri Slaby, Avuton Olrich,
	Kernel development list, Andrew Morton, Linux-pm mailing list,
	USB development list


[-- Attachment #1.1: Type: text/plain, Size: 870 bytes --]

On Monday 21 May 2007, Alan Stern wrote:
> Andrey, Soeren, and Avuton: Please try this patch with 2.6.22-rc2 or
> later and see if it fixes your problems.
>

It does; thank you!

> Greg, if this works then I'll send it in the proper form for a patch,
> and you can use it to replace
>
> 	usb-make-the-autosuspend-workqueue-thread-freezable.patch
>
> Alan Stern
>
>
> Index: 2.6.22-rc2/drivers/usb/core/usb.c
> ===================================================================
> --- 2.6.22-rc2.orig/drivers/usb/core/usb.c
> +++ 2.6.22-rc2/drivers/usb/core/usb.c
> @@ -205,7 +205,7 @@ struct device_type usb_device_type = {
>
>  static int ksuspend_usb_init(void)
>  {
> -	ksuspend_usb_wq = create_singlethread_workqueue("ksuspend_usbd");
> +	ksuspend_usb_wq = create_freezeable_workqueue("ksuspend_usbd");
>  	if (!ksuspend_usb_wq)
>  		return -ENOMEM;
>  	return 0;



[-- Attachment #1.2: Type: application/pgp-signature, Size: 189 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]



^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [Bug 8498], [Bug 8510], and Re: Can't s2ram 22-rc2
  2007-05-21 14:51 ` Alan Stern
                     ` (3 preceding siblings ...)
  2007-05-21 15:43   ` Andrey Borzenkov
@ 2007-05-21 16:12   ` Avuton Olrich
  2007-05-21 16:12   ` Avuton Olrich
  5 siblings, 0 replies; 8+ messages in thread
From: Avuton Olrich @ 2007-05-21 16:12 UTC (permalink / raw)
  To: Alan Stern
  Cc: Soeren Sonnenburg, Jiri Slaby, Kernel development list,
	Andrew Morton, Andrey Borzenkov, Linux-pm mailing list,
	USB development list

On 5/21/07, Alan Stern <stern@rowland.harvard.edu> wrote:
> Andrey, Soeren, and Avuton: Please try this patch with 2.6.22-rc2 or
> later and see if it fixes your problems.

It does, I already synced the -rc1 patch that was at the bug link for myself.

Thanks
-- 
avuton
--
 Anyone who quotes me in their sig is an idiot. -- Rusty Russell.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [Bug 8498], [Bug 8510], and Re: Can't s2ram 22-rc2
  2007-05-21 14:51 ` Alan Stern
                     ` (4 preceding siblings ...)
  2007-05-21 16:12   ` Avuton Olrich
@ 2007-05-21 16:12   ` Avuton Olrich
  5 siblings, 0 replies; 8+ messages in thread
From: Avuton Olrich @ 2007-05-21 16:12 UTC (permalink / raw)
  To: Alan Stern
  Cc: Andrey Borzenkov, Soeren Sonnenburg, Greg KH, Rafael J. Wysocki,
	Jiri Slaby, Andrew Morton, USB development list,
	Linux-pm mailing list, Kernel development list

On 5/21/07, Alan Stern <stern@rowland.harvard.edu> wrote:
> Andrey, Soeren, and Avuton: Please try this patch with 2.6.22-rc2 or
> later and see if it fixes your problems.

It does, I already synced the -rc1 patch that was at the bug link for myself.

Thanks
-- 
avuton
--
 Anyone who quotes me in their sig is an idiot. -- Rusty Russell.

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2007-05-21 16:12 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <20070519164504.GA10675@kroah.com>
2007-05-21 14:51 ` [Bug 8498], [Bug 8510], and Re: Can't s2ram 22-rc2 Alan Stern
2007-05-21 14:51 ` Alan Stern
2007-05-21 15:10   ` Soeren Sonnenburg
2007-05-21 15:10   ` Soeren Sonnenburg
2007-05-21 15:43   ` Andrey Borzenkov
2007-05-21 15:43   ` Andrey Borzenkov
2007-05-21 16:12   ` Avuton Olrich
2007-05-21 16:12   ` Avuton Olrich

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.