* [Qemu-devel] [PATCH qemu] spapr: Add /system-id @ 2015-11-09 6:47 Alexey Kardashevskiy 2015-11-09 8:47 ` David Gibson 0 siblings, 1 reply; 9+ messages in thread From: Alexey Kardashevskiy @ 2015-11-09 6:47 UTC (permalink / raw) To: qemu-devel; +Cc: Alexey Kardashevskiy, qemu-ppc, David Gibson Section B.6.2.1 Root Node Properties of PAPR specification defines a set of properties which shall be present in the device tree root, one of these properties is "system-id" which "should be unique across all systems and all manufacturers". Since UUID is meant to be unique, it makes sense to use it as "system-id". This adds "system-id" property to the device tree root when not empty. Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru> --- This might be expected by AIX so here is the patch. I am really not sure if it makes sense to initialize property when UUID is all zeroes as the requirement is "unique" and zero-uuid is not. --- hw/ppc/spapr.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c index de77528..e8b407d 100644 --- a/hw/ppc/spapr.c +++ b/hw/ppc/spapr.c @@ -374,6 +374,9 @@ static void *spapr_create_fdt_skel(hwaddr initrd_base, qemu_uuid[14], qemu_uuid[15]); _FDT((fdt_property_string(fdt, "vm,uuid", buf))); + if (qemu_uuid_set) { + _FDT((fdt_property_string(fdt, "system-id", buf))); + } g_free(buf); if (qemu_get_vm_name()) { -- 2.5.0.rc3 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [Qemu-devel] [PATCH qemu] spapr: Add /system-id 2015-11-09 6:47 [Qemu-devel] [PATCH qemu] spapr: Add /system-id Alexey Kardashevskiy @ 2015-11-09 8:47 ` David Gibson 2015-11-18 7:45 ` Alexey Kardashevskiy 0 siblings, 1 reply; 9+ messages in thread From: David Gibson @ 2015-11-09 8:47 UTC (permalink / raw) To: Alexey Kardashevskiy; +Cc: qemu-ppc, qemu-devel [-- Attachment #1: Type: text/plain, Size: 1619 bytes --] On Mon, Nov 09, 2015 at 05:47:17PM +1100, Alexey Kardashevskiy wrote: > Section B.6.2.1 Root Node Properties of PAPR specification defines > a set of properties which shall be present in the device tree root, > one of these properties is "system-id" which "should be unique across > all systems and all manufacturers". Since UUID is meant to be unique, > it makes sense to use it as "system-id". > > This adds "system-id" property to the device tree root when not empty. > > Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru> > --- > > This might be expected by AIX so here is the patch. > I am really not sure if it makes sense to initialize property when > UUID is all zeroes as the requirement is "unique" and zero-uuid is > not. Yeah, I think it would be better to omit system-id entirely when a UUID hasn't been supplied. > > --- > hw/ppc/spapr.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c > index de77528..e8b407d 100644 > --- a/hw/ppc/spapr.c > +++ b/hw/ppc/spapr.c > @@ -374,6 +374,9 @@ static void *spapr_create_fdt_skel(hwaddr initrd_base, > qemu_uuid[14], qemu_uuid[15]); > > _FDT((fdt_property_string(fdt, "vm,uuid", buf))); > + if (qemu_uuid_set) { > + _FDT((fdt_property_string(fdt, "system-id", buf))); > + } > g_free(buf); > > if (qemu_get_vm_name()) { -- David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 819 bytes --] ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Qemu-devel] [PATCH qemu] spapr: Add /system-id 2015-11-09 8:47 ` David Gibson @ 2015-11-18 7:45 ` Alexey Kardashevskiy 2015-11-18 10:49 ` David Gibson 0 siblings, 1 reply; 9+ messages in thread From: Alexey Kardashevskiy @ 2015-11-18 7:45 UTC (permalink / raw) To: David Gibson; +Cc: qemu-ppc, qemu-devel On 11/09/2015 07:47 PM, David Gibson wrote: > On Mon, Nov 09, 2015 at 05:47:17PM +1100, Alexey Kardashevskiy wrote: >> Section B.6.2.1 Root Node Properties of PAPR specification defines >> a set of properties which shall be present in the device tree root, >> one of these properties is "system-id" which "should be unique across >> all systems and all manufacturers". Since UUID is meant to be unique, >> it makes sense to use it as "system-id". >> >> This adds "system-id" property to the device tree root when not empty. >> >> Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru> >> --- >> >> This might be expected by AIX so here is the patch. >> I am really not sure if it makes sense to initialize property when >> UUID is all zeroes as the requirement is "unique" and zero-uuid is >> not. > > Yeah, I think it would be better to omit system-id entirely when a > UUID hasn't been supplied. so this did not go anywhere yet, did it? >> >> --- >> hw/ppc/spapr.c | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c >> index de77528..e8b407d 100644 >> --- a/hw/ppc/spapr.c >> +++ b/hw/ppc/spapr.c >> @@ -374,6 +374,9 @@ static void *spapr_create_fdt_skel(hwaddr initrd_base, >> qemu_uuid[14], qemu_uuid[15]); >> >> _FDT((fdt_property_string(fdt, "vm,uuid", buf))); >> + if (qemu_uuid_set) { >> + _FDT((fdt_property_string(fdt, "system-id", buf))); >> + } >> g_free(buf); >> >> if (qemu_get_vm_name()) { > -- Alexey ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Qemu-devel] [PATCH qemu] spapr: Add /system-id 2015-11-18 7:45 ` Alexey Kardashevskiy @ 2015-11-18 10:49 ` David Gibson 2015-11-25 15:15 ` [Qemu-devel] [Qemu-ppc] " Alexander Graf 0 siblings, 1 reply; 9+ messages in thread From: David Gibson @ 2015-11-18 10:49 UTC (permalink / raw) To: Alexey Kardashevskiy; +Cc: qemu-ppc, qemu-devel [-- Attachment #1: Type: text/plain, Size: 1266 bytes --] On Wed, Nov 18, 2015 at 06:45:39PM +1100, Alexey Kardashevskiy wrote: > On 11/09/2015 07:47 PM, David Gibson wrote: > >On Mon, Nov 09, 2015 at 05:47:17PM +1100, Alexey Kardashevskiy wrote: > >>Section B.6.2.1 Root Node Properties of PAPR specification defines > >>a set of properties which shall be present in the device tree root, > >>one of these properties is "system-id" which "should be unique across > >>all systems and all manufacturers". Since UUID is meant to be unique, > >>it makes sense to use it as "system-id". > >> > >>This adds "system-id" property to the device tree root when not empty. > >> > >>Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru> > >>--- > >> > >>This might be expected by AIX so here is the patch. > >>I am really not sure if it makes sense to initialize property when > >>UUID is all zeroes as the requirement is "unique" and zero-uuid is > >>not. > > > >Yeah, I think it would be better to omit system-id entirely when a > >UUID hasn't been supplied. > > > so this did not go anywhere yet, did it? No. -- David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 819 bytes --] ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Qemu-devel] [Qemu-ppc] [PATCH qemu] spapr: Add /system-id 2015-11-18 10:49 ` David Gibson @ 2015-11-25 15:15 ` Alexander Graf 2015-11-26 0:49 ` David Gibson 0 siblings, 1 reply; 9+ messages in thread From: Alexander Graf @ 2015-11-25 15:15 UTC (permalink / raw) To: David Gibson, Alexey Kardashevskiy; +Cc: qemu-ppc, qemu-devel On 18.11.15 11:49, David Gibson wrote: > On Wed, Nov 18, 2015 at 06:45:39PM +1100, Alexey Kardashevskiy wrote: >> On 11/09/2015 07:47 PM, David Gibson wrote: >>> On Mon, Nov 09, 2015 at 05:47:17PM +1100, Alexey Kardashevskiy wrote: >>>> Section B.6.2.1 Root Node Properties of PAPR specification defines >>>> a set of properties which shall be present in the device tree root, >>>> one of these properties is "system-id" which "should be unique across >>>> all systems and all manufacturers". Since UUID is meant to be unique, >>>> it makes sense to use it as "system-id". >>>> >>>> This adds "system-id" property to the device tree root when not empty. >>>> >>>> Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru> >>>> --- >>>> >>>> This might be expected by AIX so here is the patch. >>>> I am really not sure if it makes sense to initialize property when >>>> UUID is all zeroes as the requirement is "unique" and zero-uuid is >>>> not. >>> >>> Yeah, I think it would be better to omit system-id entirely when a >>> UUID hasn't been supplied. >> >> >> so this did not go anywhere yet, did it? > > No. So where is it stuck? Alex ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Qemu-devel] [Qemu-ppc] [PATCH qemu] spapr: Add /system-id 2015-11-25 15:15 ` [Qemu-devel] [Qemu-ppc] " Alexander Graf @ 2015-11-26 0:49 ` David Gibson 2015-11-26 4:29 ` Alexey Kardashevskiy 0 siblings, 1 reply; 9+ messages in thread From: David Gibson @ 2015-11-26 0:49 UTC (permalink / raw) To: Alexander Graf; +Cc: Alexey Kardashevskiy, qemu-ppc, qemu-devel [-- Attachment #1: Type: text/plain, Size: 1577 bytes --] On Wed, Nov 25, 2015 at 04:15:01PM +0100, Alexander Graf wrote: > > > On 18.11.15 11:49, David Gibson wrote: > > On Wed, Nov 18, 2015 at 06:45:39PM +1100, Alexey Kardashevskiy wrote: > >> On 11/09/2015 07:47 PM, David Gibson wrote: > >>> On Mon, Nov 09, 2015 at 05:47:17PM +1100, Alexey Kardashevskiy wrote: > >>>> Section B.6.2.1 Root Node Properties of PAPR specification defines > >>>> a set of properties which shall be present in the device tree root, > >>>> one of these properties is "system-id" which "should be unique across > >>>> all systems and all manufacturers". Since UUID is meant to be unique, > >>>> it makes sense to use it as "system-id". > >>>> > >>>> This adds "system-id" property to the device tree root when not empty. > >>>> > >>>> Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru> > >>>> --- > >>>> > >>>> This might be expected by AIX so here is the patch. > >>>> I am really not sure if it makes sense to initialize property when > >>>> UUID is all zeroes as the requirement is "unique" and zero-uuid is > >>>> not. > >>> > >>> Yeah, I think it would be better to omit system-id entirely when a > >>> UUID hasn't been supplied. > >> > >> > >> so this did not go anywhere yet, did it? > > > > No. > > So where is it stuck? I was waiting for a respin which didn't set the property when a UUID hadn't been given. -- David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 819 bytes --] ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Qemu-devel] [Qemu-ppc] [PATCH qemu] spapr: Add /system-id 2015-11-26 0:49 ` David Gibson @ 2015-11-26 4:29 ` Alexey Kardashevskiy 2015-11-26 11:04 ` Greg Kurz 2015-11-26 23:46 ` David Gibson 0 siblings, 2 replies; 9+ messages in thread From: Alexey Kardashevskiy @ 2015-11-26 4:29 UTC (permalink / raw) To: David Gibson, Alexander Graf; +Cc: qemu-ppc, qemu-devel On 11/26/2015 11:49 AM, David Gibson wrote: > On Wed, Nov 25, 2015 at 04:15:01PM +0100, Alexander Graf wrote: >> >> >> On 18.11.15 11:49, David Gibson wrote: >>> On Wed, Nov 18, 2015 at 06:45:39PM +1100, Alexey Kardashevskiy wrote: >>>> On 11/09/2015 07:47 PM, David Gibson wrote: >>>>> On Mon, Nov 09, 2015 at 05:47:17PM +1100, Alexey Kardashevskiy wrote: >>>>>> Section B.6.2.1 Root Node Properties of PAPR specification defines >>>>>> a set of properties which shall be present in the device tree root, >>>>>> one of these properties is "system-id" which "should be unique across >>>>>> all systems and all manufacturers". Since UUID is meant to be unique, >>>>>> it makes sense to use it as "system-id". >>>>>> >>>>>> This adds "system-id" property to the device tree root when not empty. >>>>>> >>>>>> Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru> >>>>>> --- >>>>>> >>>>>> This might be expected by AIX so here is the patch. >>>>>> I am really not sure if it makes sense to initialize property when >>>>>> UUID is all zeroes as the requirement is "unique" and zero-uuid is >>>>>> not. >>>>> >>>>> Yeah, I think it would be better to omit system-id entirely when a >>>>> UUID hasn't been supplied. >>>> >>>> >>>> so this did not go anywhere yet, did it? >>> >>> No. >> >> So where is it stuck? > > I was waiting for a respin which didn't set the property when a UUID > hadn't been given. > This is the original patch: + if (qemu_uuid_set) { + _FDT((fdt_property_string(fdt, "system-id", buf))); + } I does not set property if qemu_uuid_set==false already. What did I miss? -- Alexey ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Qemu-devel] [Qemu-ppc] [PATCH qemu] spapr: Add /system-id 2015-11-26 4:29 ` Alexey Kardashevskiy @ 2015-11-26 11:04 ` Greg Kurz 2015-11-26 23:46 ` David Gibson 1 sibling, 0 replies; 9+ messages in thread From: Greg Kurz @ 2015-11-26 11:04 UTC (permalink / raw) To: Alexey Kardashevskiy; +Cc: qemu-devel, qemu-ppc, Alexander Graf, David Gibson On Thu, 26 Nov 2015 15:29:07 +1100 Alexey Kardashevskiy <aik@ozlabs.ru> wrote: > On 11/26/2015 11:49 AM, David Gibson wrote: > > On Wed, Nov 25, 2015 at 04:15:01PM +0100, Alexander Graf wrote: > >> > >> > >> On 18.11.15 11:49, David Gibson wrote: > >>> On Wed, Nov 18, 2015 at 06:45:39PM +1100, Alexey Kardashevskiy wrote: > >>>> On 11/09/2015 07:47 PM, David Gibson wrote: > >>>>> On Mon, Nov 09, 2015 at 05:47:17PM +1100, Alexey Kardashevskiy wrote: > >>>>>> Section B.6.2.1 Root Node Properties of PAPR specification defines > >>>>>> a set of properties which shall be present in the device tree root, > >>>>>> one of these properties is "system-id" which "should be unique across > >>>>>> all systems and all manufacturers". Since UUID is meant to be unique, > >>>>>> it makes sense to use it as "system-id". > >>>>>> > >>>>>> This adds "system-id" property to the device tree root when not empty. > >>>>>> > >>>>>> Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru> > >>>>>> --- > >>>>>> > >>>>>> This might be expected by AIX so here is the patch. > >>>>>> I am really not sure if it makes sense to initialize property when > >>>>>> UUID is all zeroes as the requirement is "unique" and zero-uuid is > >>>>>> not. > >>>>> > >>>>> Yeah, I think it would be better to omit system-id entirely when a > >>>>> UUID hasn't been supplied. > >>>> > >>>> > >>>> so this did not go anywhere yet, did it? > >>> > >>> No. > >> > >> So where is it stuck? > > > > I was waiting for a respin which didn't set the property when a UUID > > hadn't been given. > > > > This is the original patch: > > > + if (qemu_uuid_set) { > + _FDT((fdt_property_string(fdt, "system-id", buf))); > + } > > I does not set property if qemu_uuid_set==false already. What did I miss? > It looks like the confusion lies in David's answer to your question about nil UUID... BTW does it make sense to assign nil UUID to a system ? -- Greg ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Qemu-devel] [Qemu-ppc] [PATCH qemu] spapr: Add /system-id 2015-11-26 4:29 ` Alexey Kardashevskiy 2015-11-26 11:04 ` Greg Kurz @ 2015-11-26 23:46 ` David Gibson 1 sibling, 0 replies; 9+ messages in thread From: David Gibson @ 2015-11-26 23:46 UTC (permalink / raw) To: Alexey Kardashevskiy; +Cc: qemu-ppc, Alexander Graf, qemu-devel [-- Attachment #1: Type: text/plain, Size: 2154 bytes --] On Thu, Nov 26, 2015 at 03:29:07PM +1100, Alexey Kardashevskiy wrote: > On 11/26/2015 11:49 AM, David Gibson wrote: > >On Wed, Nov 25, 2015 at 04:15:01PM +0100, Alexander Graf wrote: > >> > >> > >>On 18.11.15 11:49, David Gibson wrote: > >>>On Wed, Nov 18, 2015 at 06:45:39PM +1100, Alexey Kardashevskiy wrote: > >>>>On 11/09/2015 07:47 PM, David Gibson wrote: > >>>>>On Mon, Nov 09, 2015 at 05:47:17PM +1100, Alexey Kardashevskiy wrote: > >>>>>>Section B.6.2.1 Root Node Properties of PAPR specification defines > >>>>>>a set of properties which shall be present in the device tree root, > >>>>>>one of these properties is "system-id" which "should be unique across > >>>>>>all systems and all manufacturers". Since UUID is meant to be unique, > >>>>>>it makes sense to use it as "system-id". > >>>>>> > >>>>>>This adds "system-id" property to the device tree root when not empty. > >>>>>> > >>>>>>Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru> > >>>>>>--- > >>>>>> > >>>>>>This might be expected by AIX so here is the patch. > >>>>>>I am really not sure if it makes sense to initialize property when > >>>>>>UUID is all zeroes as the requirement is "unique" and zero-uuid is > >>>>>>not. > >>>>> > >>>>>Yeah, I think it would be better to omit system-id entirely when a > >>>>>UUID hasn't been supplied. > >>>> > >>>> > >>>>so this did not go anywhere yet, did it? > >>> > >>>No. > >> > >>So where is it stuck? > > > >I was waiting for a respin which didn't set the property when a UUID > >hadn't been given. > > > > This is the original patch: > > > + if (qemu_uuid_set) { > + _FDT((fdt_property_string(fdt, "system-id", buf))); > + } > > I does not set property if qemu_uuid_set==false already. What did I miss? Oh, sorry, I misread the original patch. I thought you were saying that if UUID wasn't specified, it would appear as a system-id of all 0s. Applied to ppc-for-2.6. -- David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 819 bytes --] ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2015-11-26 23:56 UTC | newest] Thread overview: 9+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2015-11-09 6:47 [Qemu-devel] [PATCH qemu] spapr: Add /system-id Alexey Kardashevskiy 2015-11-09 8:47 ` David Gibson 2015-11-18 7:45 ` Alexey Kardashevskiy 2015-11-18 10:49 ` David Gibson 2015-11-25 15:15 ` [Qemu-devel] [Qemu-ppc] " Alexander Graf 2015-11-26 0:49 ` David Gibson 2015-11-26 4:29 ` Alexey Kardashevskiy 2015-11-26 11:04 ` Greg Kurz 2015-11-26 23:46 ` David Gibson
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).