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

Re: [Xen-devel] [PATCH 0/1] Dom0less guest device tree format


  • To: Stefano Stabellini <sstabellini@xxxxxxxxxx>
  • From: Will Abele <will.abele@xxxxxxxxxx>
  • Date: Tue, 9 Jul 2019 13:17:13 +0000
  • Accept-language: en-US
  • Arc-authentication-results: i=1; mx.microsoft.com 1;spf=pass smtp.mailfrom=starlab.io;dmarc=pass action=none header.from=starlab.io;dkim=pass header.d=starlab.io;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=KxQHcaISl484CpSeooDGkydEoHmVUY9GQ0Hidx9QnYs=; b=Dg34JB0nX9dWPrnPjZsr6Kl6aICADGynIG1kCk5Q4yDwIRp2XG3LuflndLOefomOgx6egzoVzPQ2/N+K2SVzzW4yQY5MnqUmqBa2Z/1zPDAC+fG7tQf+LfEGaV4z+1ycjEu6GARyb302bjChhws0yrTqDzJtFL82TWtRXxQ8WOkYXvGJGd5g2Nkx4e8sSE5NbxGYBBqOBJyO3sX7sFpi0b0MYX/RdBQkJKiihU01thHSOTZ1UW/gwTYQFxt9mhX6HiwJ4SRwmNH7SD+fdTmqIkB5rSQtU8+6ACZb03waObcSLS5Eyi1rm5Db53fmltlpeb8ITLvM0YbsazZkqow+TA==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=mhn4mQ+/lEz5ODPkVHE3ROLEyssrrPnlradDMMob9V4Di6U4XTrlzCyPj1IHg3w+VuNw5hNlyxY6R8PNnrE/hAb/QOXpPhJTEI4N/Xo0Ki6CcKgLzpR5BT6zdrVP42+yhYv+1GdMa/odkJeO1o1nRzPe4wbCR7w5Jz0Tk7m75eTFWeeR7cyEpFFm1fjUdfG5dTTnapP6aIlRilVuXw/aaatC/Rr/B5MER0XZILTYdkvNeXyqP4Wh6mJct+Akhxr/7gvhtxy9ufvD4fKmMkfqelBTjepRYrbnIAIDJMhO07P2ZzlUVwIhKlXaov6FlZJiDrRenF9H+xi4HJo9Vg73Dw==
  • Authentication-results: spf=none (sender IP is ) smtp.mailfrom=will.abele@xxxxxxxxxx;
  • Cc: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Julien Grall <Julien.Grall@xxxxxxx>, nd <nd@xxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>
  • Delivery-date: Tue, 09 Jul 2019 13:17:41 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Thread-index: AQHVNCT8i/e9XHQvKUuqip9pKzLNp6a95d0AgAAAdQD//+zngIADfVwAgAD4XIA=
  • Thread-topic: [PATCH 0/1] Dom0less guest device tree format

The 07/08/2019 17:28, Stefano Stabellini wrote:
> On Sat, 6 Jul 2019, Will Abele wrote:
> > The 07/06/2019 18:19, Julien Grall wrote:
> > > 
> > > 
> > > On 06/07/2019 19:17, Julien Grall wrote:
> > > > 
> > > > 
> > > > On 06/07/2019 19:02, Will Abele wrote:
> > > >> From: Will Abele <will.abele@xxxxxxxxxx>
> > > >>
> > > >> Hi,
> > > > 
> > > > Hi,
> > > > 
> > > >> I've been using dom0less Xen on the Hikey 960 with a 4.14 Linux 
> > > >> Kernel. I had
> > > >> trouble getting the 4.14 Linux Kernel to boot as a dom0less domU 
> > > >> because it was
> > > >> misinterpreting the device tree version. Linux 4.14 and earlier 
> > > >> interpret device
> > > >> trees with a "/" in the root node as version 16. Xen produces a 
> > > >> version 17
> > > >> device tree, so the root node needs to be "" to work with 4.14 and 
> > > >> earlier Linux
> > > >> Kernels. Linux 4.15 and later assume that the version is 17, so this 
> > > >> patch does
> > > >> not have any impact.
> > > >>
> > > >> Please let me know if you need any more information or have 
> > > >> suggestions for
> > > >> other ways to handle this.
> > > > 
> > > > I don't understand where the version comes from. I also don't 
> > > > understand 
> > > > how you inferred that Xen is creating a version 17 device-tree.
> > > > 
> > > > Do you have link to the paragraph in the specifications?
> > > 
> > > Also, please expand what is the exact error. So we can understand 
> > > whether this is the right fix.
> > > 
> > > Cheers,
> > > 
> > > -- 
> > > Julien Grall
> > 
> > -- 
> > 
> > Hi Julien,
> > 
> > Thanks for the prompt response.
> > 
> > I said in my message that Linux was interpreting the device tree as version 
> > 16.
> > Looking through the code again, I realize it was being interpreted as 
> > earlier
> > than 16. As mentioned in Linux commit 
> > a7e4cfb0a7ca4773e7d0dd1d9c018ab27a15360e,
> > Linux had already broken support for FDT versions earlier than 16.
> > populate_node() in drivers/of/fdt.c would stop parsing the fdt at the root 
> > node
> > if it thought the fdt version was earlier than 16.
> > 
> > Xen sets the FDT version to 17 in fdt_create().
> > 
> > The issue I was having was that Linux panicked while initializing interrupts
> > because it could not find an interrupt controller. It couldn't find the
> > interrupt controller because it didn't process that part of the device tree.
> 
> Thank you, Will! And it is great to hear that you are using dom0less :)
> 
> I couldn't find the specific reference to the spec, but I could verify
> that the patch fixes the issue for Linux 4.14, while it is unneeded for
> newer Linux versions (they still work with the patch). Also we already
> start empty device tree using "" instead of "/" in a few other places. I
> would love to have the right reference in the commit message though.
> 
> FYI we also have another instance of fdt_begin_node(fdt, "/") in
> xen/arch/arm/acpi/domain_build.c that needs fixing and could be done in
> this patch.

Thanks, Stefano! I really appreciate all of your work on dom0less. We're very
excited to use it.

Thanks for pointing out the other instance. I'll fix that, change the commit
message to reference the spec, and send a v2 in a moment.

Thanks,
Will

-- 
Will Abele
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

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