From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 0EDF620F4 for ; Sun, 25 Jun 2023 14:40:34 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 27665C433C0; Sun, 25 Jun 2023 14:40:33 +0000 (UTC) Authentication-Results: smtp.kernel.org; dkim=pass (1024-bit key) header.d=zx2c4.com header.i=@zx2c4.com header.b="JZi3W7u0" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=zx2c4.com; s=20210105; t=1687704030; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=W9bwUh3XZI0MEK9b1+FH1A6Siutg0zhBNWcDa+CeRtE=; b=JZi3W7u0qo7z03jALCC+GJ5v1KJePmwLLMpenda/7UBFHp+upwiG+3CkGbhPytkh3pRQRI l75ubZZWHVON7KrYWEYDLNzhrhr6ZLcg/LuQ7ku3/tOeHfV7kYJEn8KTeWMceGV2+yEvjk P576kJnixk9yuR/PlzfRQjXWHvPRXlc= Received: by mail.zx2c4.com (ZX2C4 Mail Server) with ESMTPSA id 788447a5 (TLSv1.3:TLS_AES_256_GCM_SHA384:256:NO); Sun, 25 Jun 2023 14:40:28 +0000 (UTC) Date: Sun, 25 Jun 2023 16:40:26 +0200 From: "Jason A. Donenfeld" To: Linus Torvalds Cc: Ard Biesheuvel , regressions@leemhuis.info, Andrew Lunn , Linux Stable , Linux Regressions , Bagas Sanjaya , Sami Korkalainen Subject: Re: [REGRESSION][BISECTED] Boot stall from merge tag 'net-next-6.2' Message-ID: References: <9517bb70-426c-0296-b426-f5b4f075f7c8@leemhuis.info> Precedence: bulk X-Mailing-List: regressions@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: On Fri, Jun 23, 2023 at 02:52:25PM -0700, Linus Torvalds wrote: > On Fri, 23 Jun 2023 at 13:31, Jason A. Donenfeld wrote: > > > > We always have to write when using so that we don't credit the same > > seed twice, so it's gotta be used at a stage when SetVariable is > > somewhat working. > > This code isn't even the code that "uses" the alleged entropy from > that EFI variable in the first place. That's the code in > efi_random_get_seed() in the EFI boot sequence, and appends it to the > bootup randomness buffers. > > And that code already seems to clear the EFI variable (or seems to > append to it). Oh, doh, yea, you're right. Sorry. My mistake. So indeed, we can probably get away with just delaying this until much later in boot, and doing this inside of a workqueue or similar, instead of in some special early boot context. Or maybe shutdown? Shutdown seems like it'd better handle potential firmware issues since hanging on shutdown is a lot better than hanging on boot. But it would be nice to keep this working during unclean shutdown, which maybe means doing it sometime after bootup is still better. > So this argument seems to be complete garbage - we absolutely do not > have to write it, and your patch already just wrote it in the wrong > place anyway. > > Don't make excuses. That code caused boot failures, it was all done in > the wrong place, and at entirely the wrong time. Yes, my point was entirely wrong. I was mistaken. But it wasn't an *excuse*. I was just momentarily confused. No malice here, I promise. Jason