* [Qemu-devel] [PATCH] spapr: Add "slb-size" property to CPU device tree nodes
@ 2015-10-01 13:30 Thomas Huth
2015-10-02 3:25 ` [Qemu-devel] [Qemu-ppc] " David Gibson
0 siblings, 1 reply; 2+ messages in thread
From: Thomas Huth @ 2015-10-01 13:30 UTC (permalink / raw)
To: qemu-ppc, David Gibson; +Cc: Alexander Graf, qemu-devel
According to a commit message in the Linux kernel (see here
https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=b60c31d85a2a
for example), the name of the property that carries the information
about the number of SLB entries should be called "slb-size", and
not "ibm,slb-size". The Linux kernel can deal with both names, but
to be on the safe side we should support the official name, too.
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
hw/ppc/spapr.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c
index a9b5f2a..ba6b273 100644
--- a/hw/ppc/spapr.c
+++ b/hw/ppc/spapr.c
@@ -625,6 +625,7 @@ static void spapr_populate_cpu_dt(CPUState *cs, void *fdt, int offset,
_FDT((fdt_setprop_cell(fdt, offset, "timebase-frequency", tbfreq)));
_FDT((fdt_setprop_cell(fdt, offset, "clock-frequency", cpufreq)));
+ _FDT((fdt_setprop_cell(fdt, offset, "slb-size", env->slb_nr)));
_FDT((fdt_setprop_cell(fdt, offset, "ibm,slb-size", env->slb_nr)));
_FDT((fdt_setprop_string(fdt, offset, "status", "okay")));
_FDT((fdt_setprop(fdt, offset, "64-bit", NULL, 0)));
--
1.8.3.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [Qemu-devel] [Qemu-ppc] [PATCH] spapr: Add "slb-size" property to CPU device tree nodes
2015-10-01 13:30 [Qemu-devel] [PATCH] spapr: Add "slb-size" property to CPU device tree nodes Thomas Huth
@ 2015-10-02 3:25 ` David Gibson
0 siblings, 0 replies; 2+ messages in thread
From: David Gibson @ 2015-10-02 3:25 UTC (permalink / raw)
To: Thomas Huth; +Cc: qemu-ppc, qemu-devel
[-- Attachment #1: Type: text/plain, Size: 775 bytes --]
On Thu, Oct 01, 2015 at 03:30:07PM +0200, Thomas Huth wrote:
> According to a commit message in the Linux kernel (see here
> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=b60c31d85a2a
> for example), the name of the property that carries the information
> about the number of SLB entries should be called "slb-size", and
> not "ibm,slb-size". The Linux kernel can deal with both names, but
> to be on the safe side we should support the official name, too.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
Merged to spapr-next, thanks.
--
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: Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2015-10-02 4:27 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-10-01 13:30 [Qemu-devel] [PATCH] spapr: Add "slb-size" property to CPU device tree nodes Thomas Huth
2015-10-02 3:25 ` [Qemu-devel] [Qemu-ppc] " 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).