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

Re: Moving Dom0less to common, Was: [PATCH 0/5] Fine granular configuration


  • To: Stefano Stabellini <sstabellini@xxxxxxxxxx>, Luca Fancellu <luca.fancellu@xxxxxxx>, <jbeulich@xxxxxxxx>, <andrew.cooper3@xxxxxxxxxx>, <roger.pau@xxxxxxxxxx>
  • From: Michal Orzel <michal.orzel@xxxxxxx>
  • Date: Thu, 10 Aug 2023 08:45:31 +0200
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=kernel.org smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); 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=R87u6pIYt02/RYsjsUdIkzY9niQNjr9GCe7BLv+qUqg=; b=ZN2+9J0+qMsALOO5vQ4OiU5M1SfDjUMygrKL5yFTmB4cwfQCOZOgihmDPQITdR3uSurJS2QgU87WRasVJNcAURXb4TPbsvVw+8rpKNTWoawnYOFwU8vMwe9c2XkTZ/Cwk4M6mBCzoMPnne0o+C+u2NrMfUwI14dLgzxAdxP3oB5Yv42iY+tiaQBuejRv1PzDhqETWQmz5o+ITxHWOoKS6IEHffPLVqL3xFPU4fOGDRhlgBMRPm3pddlpNKIruPeNUelu5gh++LxndNCiYENyovvAUX8DXUEW7UdNIhJtZFkfqHX/qmge+k0j/nwiC41zshNG38EFl99n8fzgapSNzQ==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=nNKGXb9/ZI4rUunDiwSpMRsrcksHNxVy3LKrT5hwGTyGt5LfxO2ldnm0pLOOX/ZZBc8kqAHFLZt5NwQyWfr9iTxirtKgdo6Yf3kXBNgoD6tLXYEC/wMib4r8fDAFkEmSdvLwmgXy6/QpS90hkWTvydguX8zOmMMCCEq7tEOftagplmB/2rDxsbTCfc95dBfribsu5xWbHpXAflG9jpWjy4TkFQzV4hE1ULs0VTfhZajlEcGTyUVRQEpnGXoe3z+37I2B1lCvtv8cV40oOIzwk9QUm1LsNQ6AdlE7lNPVGlC9PkXEdoNe5x3yaRPpGyje73ZhxSuwB1zWBlUHs3aunA==
  • Cc: <xen-devel@xxxxxxxxxxxxxxxxxxxx>, <wei.chen@xxxxxxx>, <rahul.singh@xxxxxxx>, Julien Grall <julien@xxxxxxx>, Bertrand Marquis <bertrand.marquis@xxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>, <george.dunlap@xxxxxxxxxx>, <christopher.w.clark@xxxxxxxxx>, <dpsmith@xxxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Thu, 10 Aug 2023 06:46:16 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>


On 09/08/2023 21:51, Stefano Stabellini wrote:
> 
> 
> Adding hyperlaunch and x86 maintainers.
> 
> 
> Hi Luca,
> 
> This patch series is very timely. You might not have realized but we
> were just discussing with the hyperlaunch guys to move dom0less code to
> common.
> 
> Your series is a great step in the right direction. However, we don't
> want to move the dom0less code twice because it would make 'git blame'
> very hard to use.
> 
> I wonder if I could ask you to move the code to xen/common/ instead. I
> am not asking you to make it buildable on x86, as that would take more
> work and also something along the lines of this patch series from
> Christopher:
> https://patchew.org/Xen/20230701071835.41599-1-christopher.w.clark@xxxxxxxxx/
The question is whether we want to move everything related to dom0less 
parse/build to common.
There are parts that make sense only for Arm. Also, a plain move might not be 
what
hyperlaunch wants and what Daniel described in his plans for the future. FWIU, 
hyperlaunch
wants to operate on flattened device tree whereas the majority of dom0less 
build/parse code
is done using unflattened device tree (FDT is used at the very early stage of 
the boot and is contained
in bootfdt.c). Maintaining two sets of parsers is not the best idea.

@Daniel,Christopher, what is the reason of using FDT and not DT on hyperlaunch?
Is is because there are parts of configuration that you want to probe early 
(e.g. XSM module)?

~Michal



 


Rackspace

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