linux-kernel - Re: [linux-pm] PM: cannot hibernate -- BUG at kernel/workqueue.c:3659

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <201201270210.37585.rjw@sisk.pl>
Date:	Fri, 27 Jan 2012 02:10:37 +0100
From:	"Rafael J. Wysocki" <rjw@...k.pl>
To:	"Srivatsa S. Bhat" <srivatsa.bhat@...ux.vnet.ibm.com>
Cc:	Jiri Slaby <jirislaby@...il.com>,
	"Linux-pm mailing list" <linux-pm@...ts.linux-foundation.org>,
	Jiri Slaby <jslaby@...e.cz>,
	LKML <linux-kernel@...r.kernel.org>, Baohua.Song@....com,
	Tejun Heo <tj@...nel.org>, "pavel@....cz" <pavel@....cz>
Subject: Re: [linux-pm] PM: cannot hibernate -- BUG 
 at	kernel/workqueue.c:3659

On Thursday, January 26, 2012, Srivatsa S. Bhat wrote:
> 
> Hi Rafael,
> 
> On 01/25/2012 09:01 PM, Srivatsa S. Bhat wrote:
> 
> > 
> > Ok, I will need to quote a part of the userspace utility to explain the
> > problem.
> > 
> > In suspend.c inside the suspend-utils userspace package, I see a loop such
> > as:
> > 
> >         error = freeze(snapshot_fd);
> > 	...
> >         attempts = 2;
> >         do {
> >                 if (set_image_size(snapshot_fd, image_size)) {
> >                         error = errno;
> >                         break;
> >                 }
> >                 if (atomic_snapshot(snapshot_fd, &in_suspend)) {
> >                         error = errno;
> >                         break;
> >                 }
> >                 if (!in_suspend) {
> >                         /* first unblank the console, see console_codes(4) */
> >                         printf("\e[13]");
> >                         printf("%s: returned to userspace\n", my_name);
> >                         free_snapshot(snapshot_fd);
> >                         break;
> >                 }
> > 
> >                 error = write_image(snapshot_fd, resume_fd, -1);
> >                 if (error) {
> >                         free_swap_pages(snapshot_fd);
> >                         free_snapshot(snapshot_fd);
> >                         image_size = 0;
> >                         error = -error;
> >                         if (error != ENOSPC)
> >                                 break;
> >                 } else {
> >                         splash.progress(100);
> > #ifdef CONFIG_BOTH
> >                         if (s2ram_kms || s2ram) {
> >                                 /* If we die (and allow system to continue)
> >                                  * between now and reset_signature(), very bad
> >                                  * things will happen. */
> >                                 error = suspend_to_ram(snapshot_fd);
> >                                 if (error)
> >                                         goto Shutdown;
> >                                 reset_signature(resume_fd);
> >                                 free_swap_pages(snapshot_fd);
> >                                 free_snapshot(snapshot_fd);
> >                                 if (!s2ram_kms)
> >                                         s2ram_resume();
> 
> 
> Your patch alters how SNAPSHOT_FREE (IOW, free_snapshot() in this utility) is
> handled. So, I was trying to see if there are any points of concern...
> 
> In the above code, s2ram_resume() gets invoked after free_snapshot(). Will that
> pose any problems because kernel threads would have been thawed at that point,
> after applying your patch?

No, it shouldn't.  s2ram_resume() only executes quirks needed to restore the
state of graphics if KMS is not being used.  That shouldn't interfere with
any kernel threads.

> And other than that, do you foresee any problems arising from the change caused
> to SNAPSHOT_FREE by your patch? I mean, s2ram/s2disk/suspend-utils package are
> not the only userspace utilities after all... so I just wanted to ensure that
> we don't over-fit our solution to this particular utility and end up breaking
> others...

I'm quite sure they are the only package using the interface in
kernel/power/user.c.  At least, I'm not aware of any other users. :-)

Thanks,
Rafael
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/