[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH-4.16 v2] xen/efi: Fix Grub2 boot on arm64
- To: Stefano Stabellini <sstabellini@xxxxxxxxxx>
- From: Jan Beulich <jbeulich@xxxxxxxx>
- Date: Tue, 9 Nov 2021 12:00:58 +0100
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.com; 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-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=zjJuk/KrpS4ruiwa4Pg3T5VONjH3xlHdxkrjsajOmys=; b=G4KqdR6Mu69p7L1kNNKd5Y+ILb06iuQBHwqbYxVO47284VQMhw+B+9w3mYX/UOUSaXkT3eh63ZaVDQbPrt9iBmuuyFdx9lyohqaCUiGBpFHaIzjKtDbEGwB9Mk3bBrb1+tdqdvxQjBE90qkd3F4pqpgghyQgGbXhx2E06b6ClcvYal+XDYadLzSrnyiQDTRE71rh0NBbCtNo8ez5aQJq7Fgtpq6G5sw5Z9DDbzWTBJgip5EpBtwGtyDL+WoA0Ib+JkUhWUaE+ePpIAY5jDOiq1K5vn4LuFaaNpyLG66nJ3A+3R7fJjoR+5sAqjLwyKHHWzctpe+C5hch8ucCFEq9Bw==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=IfPSHc5TirtQORw4bo9hUpqIjQalBL2UAkCtx38agerV/MMajIYD5ud/0hTUS9GOMq0+1jGEsmUVDg551NAJXtoQA2ta4cTKPcKMfz226E0jrrNdELaf3jPeml31uTv3NAfU/uV1XKsbPfkhAwpfOaD9NTixZiDYroDtVxXeRf1tMIkcjtuhGRUUS3c/C6A8ZKATsx5DwJLlUWCi9z+4HGKSh2a8A9U8+qtRSS/dnSYjjur2hpwCLWTCkgg1AR7lGDi8tre8Qg4SV6f/uEQGrvOVwvLFIgbHAVz714weC+eIg4Vpn4ozzIjKOZzNTpmh1Ht7pehBFJ+/nqoqkG7ZmA==
- Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com;
- Cc: Bertrand Marquis <bertrand.marquis@xxxxxxx>, wei.chen@xxxxxxx, Ian Jackson <iwj@xxxxxxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx, Luca Fancellu <luca.fancellu@xxxxxxx>
- Delivery-date: Tue, 09 Nov 2021 11:01:22 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On 09.11.2021 03:11, Stefano Stabellini wrote:
> On Mon, 8 Nov 2021, Jan Beulich wrote:
>> On 05.11.2021 16:33, Stefano Stabellini wrote:
>>> My main concern is performance and resource utilization. With v3 of the
>>> patch get_parent_handle will get called for every module to be loaded.
>>> With dom0less, it could easily get called 10 times or more. Taking a
>>> look at get_parent_handle, the Xen side doesn't seem small and I have
>>> no idea how the EDK2 side looks. I am just worried that it would
>>> actually have an impact on boot times (also depending on the bootloader
>>> implementation).
>>
>> The biggest part of the function deals with determining the "residual" of
>> the file name. That part looks to be of no interest at all to
>> allocate_module_file() (whether that's actually correct I can't tell). I
>> don't see why this couldn't be made conditional (e.g. by passing in NULL
>> for "leaf").
>
> I understand the idea of passing NULL instead of "leaf", but I tried
> having a look and I can't tell what we would be able to skip in
> get_parent_handle.
My bad - I did overlook that dir_handle gets updated even past the
initial loop.
> Should we have a global variable to keep the dir_handle open during
> dom0less module loading?
If that's contained within Arm-specific code, I (obviously) don't mind.
Otherwise I remain to be convinced.
Jan
|