[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] xen: Create EFI_VENDOR directory


  • To: Ian Jackson <iwj@xxxxxxxxxxxxxx>
  • From: Michael Young <m.a.young@xxxxxxxxxxxx>
  • Date: Tue, 23 Mar 2021 15:58:54 +0000 (GMT)
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=durham.ac.uk; dmarc=pass action=none header.from=durham.ac.uk; dkim=pass header.d=durham.ac.uk; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=qXJWkQg2RucF4w7bB9+7xhnq09sNwbk0FJm2HA8euMU=; b=cvNdn8kxYbo8E5KRE4b9wWWuHZoB3pVJqrJ23UnwEb0Ku5RArCsZwNeptM7ohcO5qOVWwp+1lILoA/HeyLB943C5o1q8M2wSn8Cu9tzAvdO4TNf1Fns1gvp2s4NhAbQIpI5gTiZkEiis/RJ0fXtwVlogM1YescezG7SqN+4VLnaJf2zD1CdnxBQAws3SsVBQ36xu+qZZ9PGpHPmx+sEZyEVLtdWLu/MBEoq+SjUl7nSwnHrjiEMRnvmjPy2AJiRxL8CQ3UmGZmh7ZQkBl/IMq9F5nza7QiHC8zKHP8DUd8ivb/CvF5sxFDQ1gvMUXHX29kEaHSh5+Mmxi2p0IN65Og==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=jQVlo5urUQXlqim8nlJ4yuXOAgp65C3FN7SlTHwSM+Cb7fP0l1USM0YyrUIb7E42z2aVOjNwtwAlLSt29RsG9+anjgGGyAdtAzFg3Eg91hp4mNe39xsgDrhwdOiFMGkOGzFn+bXtzwY9QbiRBh29PW//sR7XjoyM0UAifUVTYCrfr+4Y6CBuFor3KRjLWT0V5gjiY4FnGIqhquSDMqPPNxEgtBChVHqbnZd4+gAcS9wfCtHtOanGRYGeQEHsIjyvtou1OEUVjSEkaMy5CV/jLXjHp6xJWdoMtnO/C/YWeeipkeEgU645wtmoVMTBl63XN/zhlyXO6uJoYPP4Vl1tYw==
  • Authentication-results: xenproject.org; dkim=none (message not signed) header.d=none;xenproject.org; dmarc=none action=none header.from=durham.ac.uk;
  • Cc: Jan Beulich <jbeulich@xxxxxxxx>, Jason Andryuk <jandryuk@xxxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Tue, 23 Mar 2021 15:59:31 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>



On Tue, 23 Mar 2021, Ian Jackson wrote:

Jan Beulich writes ("Re: [PATCH] xen: Create EFI_VENDOR directory"):
On 23.03.2021 13:34, Jason Andryuk wrote:
...
On Fedora, RPMs drop EFI binaries directly into /boot/efi/EFI/fedora/.
grub, shim, fwupdate and xen are all packaged that way.  It seems
reasonable to have those important binaries tracked by the package
manager.

Does SuSE populate EFI_VENDOR from EFI_DIR when some boot loader
script is called?

Yes. And back at the time, when I consulted our EFI person, I was left
with the impression that this is the only reasonable approach. The
primary reason, as said, was that the EFI partition as a whole may get
rebuilt perhaps even from scratch at any point. Hence it's not
reasonable to expect package-managed files to live there.

I agree with this analysis but it is for people like Fedora to decide
how they want to build their packages.

There is also the case of ad-hoc packages (eg our "make debball")
which the user might reasonably choose to have dump things in the EFI
system partition.

Conversely, I see no downside to the mkdir.  Jan, is there some actual
harm in it ?  If not, we should be accomodating to people's build and
packaging strategies even if we don't entirely approve of them.

There is a request in https://bugzilla.redhat.com/show_bug.cgi?id=1750733 for xen on Fedora to install the efi file elsewhere and then copy it to /boot/efi post install. I could change the current Fedora set up for Fedora 35 (which should have xen-4.15) if there is a good reason to do so. I am not sure how useful the xen.efi file in /boot/efi is anyway for the the Fedora set up as it will generally use a xen*.gz file in /boot via grub.

        Michael Young



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.