From patchwork Fri Oct 24 19:02:51 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Johan Hovold X-Patchwork-Id: 5149251 Return-Path: X-Original-To: patchwork-linux-arm@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork2.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork2.web.kernel.org (Postfix) with ESMTP id C7DD7C11AC for ; Fri, 24 Oct 2014 19:08:52 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id B8A0A20265 for ; Fri, 24 Oct 2014 19:08:51 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.9]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id D39B92025B for ; Fri, 24 Oct 2014 19:08:50 +0000 (UTC) Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1XhkBm-0000Zg-Rn; Fri, 24 Oct 2014 19:06:18 +0000 Received: from mail-la0-x22b.google.com ([2a00:1450:4010:c03::22b]) by bombadil.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1XhkBi-0000UY-UP for linux-arm-kernel@lists.infradead.org; Fri, 24 Oct 2014 19:06:16 +0000 Received: by mail-la0-f43.google.com with SMTP id mc6so3197667lab.30 for ; Fri, 24 Oct 2014 12:05:52 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=sender:date:from:to:cc:subject:message-id:references:mime-version :content-type:content-disposition:in-reply-to:user-agent; bh=pZLebZnpF9ZWqjR0cbR8mim5LSsHe+6QyMparpKwBek=; b=stiqvU6vHE0V+bS5S8HNn8QJnJHwUZkHfGNxG1qz742wlplNk0ZtZwwpyoHFyf095m 6H91VopWaIr4PKF6n+YlquVWNIzY1xjajF3VCN1gE52kzYgUfu8UKwWT0BgL2/S6qTCc uRtD98kO90+uKB6ssayeNx7RLYHDnEk2nBh4BC1skOtSYIvW9CMg+WkB/8nP4uYNhOfk v9ENB/2RjqA3TlnWsJh5mAHBXJpqomDKYegZ3iP16Hq/+Dh9B3VrWaNKzM9VEoylxdal 6N4YP7iEQLg0EPTGqYsVBO3JPf4Mbcl2lLa5bfeW9JhhHhkia45+kMElYcD8Q8x+UpCY 5p6Q== X-Received: by 10.112.172.231 with SMTP id bf7mr4671746lbc.100.1414177552035; Fri, 24 Oct 2014 12:05:52 -0700 (PDT) Received: from xi.terra (s83-177-171-8.cust.tele2.se. [83.177.171.8]) by mx.google.com with ESMTPSA id e3sm2131801lam.26.2014.10.24.12.05.50 for (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 24 Oct 2014 12:05:51 -0700 (PDT) Received: from johan by xi.terra with local (Exim 4.84) (envelope-from ) id 1Xhk8R-0001Ne-Bu; Fri, 24 Oct 2014 21:02:51 +0200 Date: Fri, 24 Oct 2014 21:02:51 +0200 From: Johan Hovold To: Felipe Balbi , Andrew Morton Subject: Re: [PATCH v2 00/20] rtc: omap: fixes and power-off feature Message-ID: <20141024190251.GB19377@localhost> References: <1412881594-25678-1-git-send-email-johan@kernel.org> <1413913086-12730-1-git-send-email-johan@kernel.org> <20141024160845.GM26941@saruman> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20141024160845.GM26941@saruman> User-Agent: Mutt/1.5.22 (2013-10-16) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20141024_120615_163264_ACDB5D1C X-CRM114-Status: GOOD ( 23.18 ) X-Spam-Score: -0.7 (/) Cc: Alessandro Zummo , rtc-linux@googlegroups.com, devicetree@vger.kernel.org, Tony Lindgren , j-keerthy@ti.com, nsekhar@ti.com, Johan Hovold , linux-kernel@vger.kernel.org, t-kristo@ti.com, Lokesh Vutla , linux-arm-kernel@lists.infradead.org, =?iso-8859-1?Q?Beno=EEt?= Cousson , Russell King , linux-omap@vger.kernel.org, Guenter Roeck X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Spam-Status: No, score=-3.2 required=5.0 tests=BAYES_00,DKIM_SIGNED, RCVD_IN_DNSWL_NONE,RP_MATCHES_RCVD,T_DKIM_INVALID,UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP [ +CC: Russell ] On Fri, Oct 24, 2014 at 11:08:45AM -0500, Felipe Balbi wrote: > I tested this entire series with my BBB and it still works fine. However > I still get below panic. This time without any DRM errors: > > [ 63.087832] Kernel panic - not syncing: Attempted to kill init! exitcode=0x00000000 > [ 63.087832] > [ 63.097399] CPU: 0 PID: 1 Comm: systemd-shutdow Not tainted 3.18.0-rc1-00095-g8524e69 #556 > [ 63.106060] [] (unwind_backtrace) from [] (show_stack+0x20/0x24) > [ 63.114160] [] (show_stack) from [] (dump_stack+0x8c/0xa4) > [ 63.121706] [] (dump_stack) from [] (panic+0xa0/0x220) > [ 63.128895] [] (panic) from [] (do_exit+0x974/0x9d0) > [ 63.135900] [] (do_exit) from [] (SyS_reboot+0x14c/0x1e8) > [ 63.143361] [] (SyS_reboot) from [] (ret_fast_syscall+0x0/0x48) > [ 63.151596] ---[ end Kernel panic - not syncing: Attempted to kill init! exitcode=0x00000000 > > Then again, this also happens by simply calling poweroff without > enabling wakealarm. Bah, I forgot to look into that. I haven't seen this myself as I don't use systemd (which does the syscall from process 0). Some driver power-off implementations and some arch machine_power_off spin indefinitely (or use an mdelay and WARN) after trying to power off. I think this is really a bug in arch/arm that should be fixed analogously to how failed reboot is handled in machine_restart(). Care to try the patch below? I should still add a two-second delay to rtc-omap to avoid the arch error message. Andrew, can you update one patch in the series or should I just resend them all (with proper Tested-by tags)? > In any case, for the whole series: > > Tested-by: Felipe Balbi Thanks for testing! Johan >From aaa1d1d6171c895b6966ba5b738ac7946ada97c7 Mon Sep 17 00:00:00 2001 From: Johan Hovold Date: Fri, 24 Oct 2014 18:53:09 +0200 Subject: [PATCH] ARM: fix failed power-off handling Make sure to handle failed power off by printing an error message and halting (analogously to how failed reboot is handled). Power off can fail for example if the hardware has not been wired up correctly. This avoids a kernel panic when called from process 0: [ 63.087832] Kernel panic - not syncing: Attempted to kill init! exitcode=0x00000000 [ 63.087832] [ 63.097399] CPU: 0 PID: 1 Comm: systemd-shutdow Not tainted 3.18.0-rc1-00095-g8524e69 #556 [ 63.106060] [] (unwind_backtrace) from [] (show_stack+0x20/0x24) [ 63.114160] [] (show_stack) from [] (dump_stack+0x8c/0xa4) [ 63.121706] [] (dump_stack) from [] (panic+0xa0/0x220) [ 63.128895] [] (panic) from [] (do_exit+0x974/0x9d0) [ 63.135900] [] (do_exit) from [] (SyS_reboot+0x14c/0x1e8) [ 63.143361] [] (SyS_reboot) from [] (ret_fast_syscall+0x0/0x48) [ 63.151596] ---[ end Kernel panic - not syncing: Attempted to kill init! exitcode=0x00000000 Signed-off-by: Johan Hovold --- arch/arm/kernel/process.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/arch/arm/kernel/process.c b/arch/arm/kernel/process.c index a35f6ebbd2c2..68c38af5687c 100644 --- a/arch/arm/kernel/process.c +++ b/arch/arm/kernel/process.c @@ -212,6 +212,12 @@ void machine_power_off(void) if (pm_power_off) pm_power_off(); + + /* Give a grace period for failure to power off */ + mdelay(1000); + + pr_err("Power off failed -- system halted\n"); + while (1); } /*