From mboxrd@z Thu Jan 1 00:00:00 1970 From: Randy Dunlap Subject: Re: question on freeze and aio Date: Wed, 14 Feb 2007 09:25:10 -0800 Message-ID: <20070214092510.5468db34.rdunlap@xenotime.net> References: <20070213101859.GD12996@elf.ucw.cz> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-pm-bounces@lists.osdl.org Errors-To: linux-pm-bounces@lists.osdl.org To: Alan Stern Cc: linux-pm@lists.osdl.org List-Id: linux-pm@vger.kernel.org On Tue, 13 Feb 2007 16:01:30 -0500 (EST) Alan Stern wrote: > On Tue, 13 Feb 2007, Pavel Machek wrote: > = > > > No, workqueues are not frozen unless they are explicitly declared as > > > freezeable. Which currently is only done by XFS, AFAIKS. > > = > > Ok, then we have a bug in swsusp. Can someone locate _which_ workqueue > > is used for aio... and make it freezeable? > = > Here is an untested patch. Does anybody have a way to test it? > = > (BTW, should it be spelled "freezeable" or "freezable"?) www.dict.org says freezable. > Index: usb-2.6/fs/aio.c > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > --- usb-2.6.orig/fs/aio.c > +++ usb-2.6/fs/aio.c > @@ -73,7 +73,7 @@ static int __init aio_setup(void) > kioctx_cachep =3D kmem_cache_create("kioctx", sizeof(struct kioctx), > 0, SLAB_HWCACHE_ALIGN|SLAB_PANIC, NULL, NULL); > = > - aio_wq =3D create_workqueue("aio"); > + aio_wq =3D create_freezeable_workqueue("aio"); > = > pr_debug("aio_setup: sizeof(struct page) =3D %d\n", (int)sizeof(struct = page)); --- ~Randy *** Remember to use Documentation/SubmitChecklist when testing your code ***