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

[ImageBuilder][PATCH v4 0/1] Imagebuilder dynamic addresses and sizes


  • To: xen-devel@xxxxxxxxxxxxxxxxxxxx
  • From: "Andrei Cherechesu (OSS)" <andrei.cherechesu@xxxxxxxxxxx>
  • Date: Fri, 30 Sep 2022 18:58:48 +0300
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=oss.nxp.com; dmarc=pass action=none header.from=oss.nxp.com; dkim=pass header.d=oss.nxp.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=2/3yUG1c11tvYKcCImeHGQqZxgmPU/OgLyrCkHuKjaE=; b=OUW8vDWYreFzdITxXex62lMsF1syrfT1j2a+l2aQ8hjw4ePC4581ETswCq02T3pwhPWRfHe70YGCSI59mguhhgsNdkISsTm37PboUbsflXcMoOq4cdelxvtJ44k6ytu4VKImwdL9n6dqUGwQ4vyCSuBVlwUlaxpZa8IkEFRecAS4vOPD8IrG2mdKNTieYUTF8qaId+AmuYe0ied59Z1+te0KLNV065yxCQfaNSLevlcoiwM4gtEffyuK+13D7jEudTJfUhYDrtEC1nAJFRUxGz17hUB98+9XkUGrhe52fylEgfBezyR8d3W+QPLKqXXUFlQ18V9qGRJmy0CMs6ULTw==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=BXS5iPq5qlqtdXODDEJczn1rcgkZr0T3n29C2XfinOFA5exB3pdpEqxbJdIrwxxG/jDyPXRFU4cpBVbhtQxA0KBNiLJP/BnZBgVqWYUFnbIMltXiS0efz8r4XWpTL9iiXHQ+KWjpT8FjAoJlODWo/Ml9Zf+aDaYelwkKG1nIiGtuOrmDjpZ5R8Sy9CCoEDd57ykLg2gWgp258lneL/fw2cCGAGVV8Wyp7qXrk2y/7v9XVLkk/w8RJNodu2UvqtkterAOrv1NEWTvRI4GXfLppCnwLl+6y+lpj9b6lOakPGBTTmB0pIABnmLNYW/C26AqJAqKgdb5cLGmZMsokAuPNQ==
  • Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=oss.nxp.com;
  • Cc: viryaos-discuss@xxxxxxxxxxxxxxxxxxxxx, sstabellini@xxxxxxxxxx, Andrei Cherechesu <andrei.cherechesu@xxxxxxx>
  • Delivery-date: Fri, 30 Sep 2022 15:59:23 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

From: Andrei Cherechesu <andrei.cherechesu@xxxxxxx>

This sent patch adds support for dynamically computing the addresses
and sizes for loaded binaries via the boot script generated by
Imagebuilder.

Compared to the v3 version of the patch, this includes Stefano's
suggestions of not adding as many "if" statements on the
$dynamic_loading_opt parameter added by the patch, along with
keeping compatibility with the FDTEDIT case.

The sent patch extends Stefano's suggestion, by also keeping
the normal flow (without "-s" parameter) mostly unaltered: the addresses
inside the generated script are literals, and the $memaddr variable
is not re-computed after each binary loaded since it's unused. The
only difference in the normal flow is that the binaries' sizes and
addresses are stored after loading each binary. Also, removed the "0x"
prefix of $memaddr set on the first line in the generated script.

These mentioned changes, compared to Stefano's suggestion, need
an additional 2 "if" branches on $dynamic_loading_opt, which I
find a worthy tradeoff for keeping the normal flow mostly unaltered.
Otherwise, I'm also happy to go with Stefano's suggestion, but
the script generated on the normal flow would contain many
unnecessary steps.

Andrei Cherechesu (1):
  uboot-script-gen: Dynamically compute addr and size when loading
    binaries


 scripts/uboot-script-gen | 114 +++++++++++++++++++++++++++------------
 1 file changed, 80 insertions(+), 34 deletions(-)

-- 
2.35.1




 


Rackspace

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