linux-efi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Kees Cook <kees@kernel.org>
To: Ard Biesheuvel <ardb@kernel.org>
Cc: Val Packett <val@packett.cool>,
	oe-lkp@lists.linux.dev, lkp@intel.com,
	linux-hardening@vger.kernel.org, Tony Luck <tony.luck@intel.com>,
	"Guilherme G. Piccoli" <gpiccoli@igalia.com>,
	linux-efi@vger.kernel.org, linux-kernel@vger.kernel.org,
	oliver.sang@intel.com
Subject: Re: [PATCH] efi: pstore: Support late setup with TEE-backed efivars ops
Date: Wed, 10 Dec 2025 06:28:56 -0800	[thread overview]
Message-ID: <202512100627.8B2915C87@keescook> (raw)
In-Reply-To: <CAMj1kXGfTNQcX-hoAy63PNU_-eb03LJLYmQGXvBB3SsGM+e1hA@mail.gmail.com>

On Wed, Dec 10, 2025 at 06:13:17PM +0900, Ard Biesheuvel wrote:
> On Wed, 10 Dec 2025 at 17:41, kernel test robot <oliver.sang@intel.com> wrote:
> >
> >
> >
> > Hello,
> >
> > kernel test robot noticed "WARNING:at_kernel/locking/rwsem.c:#__down_write_trylock" on:
> >
> > commit: 6f473fefec79a8ba24013a5676a93934ee5ac922 ("[PATCH] efi: pstore: Support late setup with TEE-backed efivars ops")
> > url: https://github.com/intel-lab-lkp/linux/commits/Val-Packett/efi-pstore-Support-late-setup-with-TEE-backed-efivars-ops/20251203-123406
> > base: https://git.kernel.org/cgit/linux/kernel/git/kees/linux.git for-next/pstore
> > patch link: https://lore.kernel.org/all/20251203042850.14210-1-val@packett.cool/
> > patch subject: [PATCH] efi: pstore: Support late setup with TEE-backed efivars ops
> >
> > in testcase: boot
> >
> > config: i386-randconfig-015-20251207
> > compiler: clang-20
> > test machine: qemu-system-i386 -enable-kvm -cpu SandyBridge -smp 2 -m 4G
> >
> > (please refer to attached dmesg/kmsg for entire log/backtrace)
> >
> >
> > +---------------------------------------------------------+------------+------------+
> > |                                                         | b692553573 | 6f473fefec |
> > +---------------------------------------------------------+------------+------------+
> > | WARNING:at_kernel/locking/rwsem.c:#__down_write_trylock | 0          | 18         |
> > | EIP:__down_write_trylock                                | 0          | 18         |
> > | WARNING:at_kernel/locking/rwsem.c:#up_write             | 0          | 18         |
> > | EIP:up_write                                            | 0          | 18         |
> > +---------------------------------------------------------+------------+------------+
> >
> 
> This seems to be a non-EFI boot, in which case the notifier chain is
> never initialized.
> 
> Kees, can you drop this from your branch so we can get it fixed? I
> also had some review feedback that hasn't been taken into account yet.

I didn't take it -- this appears to be patch-based 0day randconfig. (I'd
rather it went via EFI anyway, too.)

-Kees

-- 
Kees Cook

      reply	other threads:[~2025-12-10 14:28 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-12-03  4:28 [PATCH] efi: pstore: Support late setup with TEE-backed efivars ops Val Packett
2025-12-04  9:49 ` Ilias Apalodimas
2025-12-04 20:57   ` Val Packett
2025-12-05 10:27     ` Ard Biesheuvel
2025-12-05 10:56       ` Ilias Apalodimas
2025-12-04 23:36 ` Kees Cook
2025-12-10  8:40 ` kernel test robot
2025-12-10  9:13   ` Ard Biesheuvel
2025-12-10 14:28     ` Kees Cook [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=202512100627.8B2915C87@keescook \
    --to=kees@kernel.org \
    --cc=ardb@kernel.org \
    --cc=gpiccoli@igalia.com \
    --cc=linux-efi@vger.kernel.org \
    --cc=linux-hardening@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lkp@intel.com \
    --cc=oe-lkp@lists.linux.dev \
    --cc=oliver.sang@intel.com \
    --cc=tony.luck@intel.com \
    --cc=val@packett.cool \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).