Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752239AbaDOVSz (ORCPT ); Tue, 15 Apr 2014 17:18:55 -0400 Received: from atrey.karlin.mff.cuni.cz ([195.113.26.193]:48335 "EHLO atrey.karlin.mff.cuni.cz" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750919AbaDOVSx (ORCPT ); Tue, 15 Apr 2014 17:18:53 -0400 Date: Tue, 15 Apr 2014 23:18:32 +0200 From: Pavel Machek To: Russell King - ARM Linux Cc: Sebastian Capella , One Thousand Gnomes , Linux Kernel , "linux-pm@vger.kernel.org" , "linaro-kernel@lists.linaro.org" , Len Brown , "Rafael J. Wysocki" , Ezequiel Garcia Subject: Re: [PATCH RFC] PM / Hibernate: no kernel_power_off when pm_power_off NULL Message-ID: <20140415211832.GA32213@amd.pavel.ucw.cz> References: <1395348795-8554-1-git-send-email-sebastian.capella@linaro.org> <1395348795-8554-2-git-send-email-sebastian.capella@linaro.org> <20140320212336.GA17368@amd.pavel.ucw.cz> <20140320213502.795a5d3c@alan.etchedpixels.co.uk> <20140415205453.GX24070@n2100.arm.linux.org.uk> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20140415205453.GX24070@n2100.arm.linux.org.uk> User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue 2014-04-15 21:54:53, Russell King - ARM Linux wrote: > On Tue, Apr 15, 2014 at 11:34:52AM -0700, Sebastian Capella wrote: > > Ping.. > > > > There appears to be disagreement on the correct path to take on this. > > > > Pavel and Alan recommend that arm's machine_power_off shall never return > > > > Russell suggests hibernation be modified to handle machine_power_off > > returning; that x86 architecture (and others as well) can have > > machine_power_off returning. > > > > Discussions available at the links below: > > https://lkml.org/lkml/2014/3/25/554 -- linux-arm discussion > > https://lkml.org/lkml/2014/3/20/649 -- linux-pm discussion > > > > Should I continue with the original hibernation patch from the > > linux-pm discussion? > > > > Does anyone have any response to Russel's commentsl? > > What I'm basically saying is that I see no reason for ARM to do something > different to what x86 does. > > What is pretty clear to me is that ARM is compatible with x86, which is > compatible with kernel/reboot.c, and it's the hibernate code which is > the odd one out. I'm pretty sure the original code did not return. Anyway, the best solution, given how many platforms are out there, would be to a) document that it should not return b) fix hibernation to handle the returning case, anyway. Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/