From mboxrd@z Thu Jan  1 00:00:00 1970
From: Julien Grall <julien.grall@linaro.org>
Subject: Re: [PATCH v3 03/14] xen: arm: allocate dom0 memory
 separately from preparing the dtb
Date: Mon, 11 Nov 2013 12:08:01 +0000
Message-ID: <5280C8A1.6050006@linaro.org>
References: <1383842678.3189.15.camel@kazak.uk.xensource.com>
	<1383842696-8598-3-git-send-email-ian.campbell@citrix.com>
	<527C9043.1050702@linaro.org>
	<1383903402.3189.44.camel@kazak.uk.xensource.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <1383903402.3189.44.camel@kazak.uk.xensource.com>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Ian Campbell <Ian.Campbell@citrix.com>
Cc: ian.jackson@eu.citrix.com, stefano.stabellini@eu.citrix.com, tim@xen.org, xen-devel@lists.xen.org
List-Id: xen-devel@lists.xenproject.org

On 11/08/2013 09:36 AM, Ian Campbell wrote:
> On Thu, 2013-11-07 at 23:18 -0800, Julien Grall wrote:
> I think
> 	/ {
> 		"memory" {
> 			#address-cells = 2;
> 	 		#size-cells = 2;
> 			device_type = "memory";
> 			reg = <...>;
> 		}
> 	}
> 
> Won't work because the #foo-cells only applies to children.
> 
> I could do
> 	/ {
> 		"memory" {
> 			#address-cells = 2;
> 	        	#size-cells = 2;
> 			"memory@foo" {
> 				device_type = "memory"
> 				reg = <...>;
> 	 		}
> 	  	}
> 	}
> 
> which puts the size under my control.

I have noticed that we have the same issue on the other nodes (gic,
timer,...). Perhaps we can create a node "xen" which contains all the
devices?

"xen" {
   #address-cells = 2;
   #size-cells = 2;
   memory {
     ...
   }
   timer {
     ...
   }
}

>>> This allows us to move kernel parsing before DTB setup.
>>
>> Why do you want to move the kernel parsing earlier? Xen don't use 
>> d->arch.type during dom0 building.
> 
> In this series prepare_dtb needs to know which kind of guest it is,
> which requires us to have parsed the kernel.

Thanks, I didn't pay attention that patch #4 uses the kernel type via
is_pv64_domain.

> 
> I also think it is an independently worthwhile change to separate
> prepping the dtb from the memory allocation, since it makes things
> cleaner overall (i.e. we can drop the overlap_check() which is quite a
> hacky way to do things which came about because of the intertwining of
> the dtb and memory alloc.
> 
>>>   static void kernel_elf_load(struct kernel_info *info)
>>>   {
>>> +    place_modules(info,
>>> +                  info->elf.parms.virt_kstart,
>>> +                  info->elf.parms.virt_kend);
>>> +
>>>       printk("Loading ELF image into guest memory\n");
>>>       info->elf.elf.dest_base = (void*)(unsigned long)info->elf.parms.virt_kstart;
>>>       info->elf.elf.dest_size =
>>>            info->elf.parms.virt_kend - info->elf.parms.virt_kstart;
>>> +
>>
>> spurious line?
> 
> Just aesthetics I think. Could drop it I guess.

Ok.


-- 
Julien Grall