From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1jnfOf-0008KH-E2 for barebox@lists.infradead.org; Tue, 23 Jun 2020 09:39:18 +0000 Date: Tue, 23 Jun 2020 11:39:15 +0200 From: Sascha Hauer Message-ID: <20200623093915.GX11869@pengutronix.de> References: <20200622151525.21082-1-a.fatoum@pengutronix.de> <20200623082330.GW11869@pengutronix.de> <510a3c4d-7c1d-751e-0cb4-be40507f98d4@pengutronix.de> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <510a3c4d-7c1d-751e-0cb4-be40507f98d4@pengutronix.de> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "barebox" Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: Re: [PATCH v2] startup: inhibit watchdogs on non-interactive autoboot abort To: Ahmad Fatoum Cc: barebox@lists.infradead.org On Tue, Jun 23, 2020 at 10:34:26AM +0200, Ahmad Fatoum wrote: > Hello Sascha, > > On 6/23/20 10:23 AM, Sascha Hauer wrote: > > On Mon, Jun 22, 2020 at 05:15:25PM +0200, Ahmad Fatoum wrote: > >> nv.autoboot=abort has been added as development aid to have barebox stop > >> at the shell prompt automatically. It makes sense to inhibit all watchdogs > >> in this mode, so the user can later use the shell in peace. > > > > As a person who is frequently annoyed by watchdogs I really like this > > change. > > :] > > > We could add a new option to the wd command to call > > watchdog_inhibit_all() for the case when the autoboot timeout has been > > interrupted by the user. > > > > Anyway, this could be done as a separate patch. > > I thought about having $global.autoboot_watchdog_inhibit_abort={nv,all}, > but didn't go through with it because I couldn't find a name I like. > > Is this what you are suggesting, but with wd setting an internal parameter > instead? I thought about a "wd -i" command that disables watchdogs or automatically pings them. I would use this in case I want to do something with barebox, but don't want to change nv.autoboot to "abort" Sascha -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox