All of lore.kernel.org
 help / color / mirror / Atom feed
* [Qemu-riscv] [PATCH for 4.0 v2 0/2] Update the QEMU PLIC addresses
@ 2019-03-27 18:51 Alistair Francis
  2019-03-27 18:51 ` [Qemu-riscv] [PATCH for 4.0 v2 1/2] riscv: plic: Fix incorrect irq calculation Alistair Francis
                   ` (3 more replies)
  0 siblings, 4 replies; 11+ messages in thread
From: Alistair Francis @ 2019-03-27 18:51 UTC (permalink / raw)
  To: qemu-devel, qemu-riscv; +Cc: palmer, Alistair Francis, alistair23

This series updates the PLIC address to match the documentation.

This fixes: https://github.com/riscv/opensbi/issues/97

V2:
 - Squash patches to ensure biesctability

Alistair Francis (2):
  riscv: plic: Fix incorrect irq calculation
  riscv: plic: Log guest errors

 hw/riscv/sifive_plic.c      | 16 +++++++++++-----
 hw/riscv/sifive_u.c         |  2 +-
 include/hw/riscv/sifive_e.h |  2 +-
 include/hw/riscv/sifive_u.h |  4 ++--
 include/hw/riscv/virt.h     |  2 +-
 5 files changed, 16 insertions(+), 10 deletions(-)

-- 
2.21.0



^ permalink raw reply	[flat|nested] 11+ messages in thread

* [Qemu-riscv] [PATCH for 4.0 v2 1/2] riscv: plic: Fix incorrect irq calculation
  2019-03-27 18:51 [Qemu-riscv] [PATCH for 4.0 v2 0/2] Update the QEMU PLIC addresses Alistair Francis
@ 2019-03-27 18:51 ` Alistair Francis
  2019-03-27 18:51 ` [Qemu-riscv] [PATCH for 4.0 v2 2/2] riscv: plic: Log guest errors Alistair Francis
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 11+ messages in thread
From: Alistair Francis @ 2019-03-27 18:51 UTC (permalink / raw)
  To: qemu-devel, qemu-riscv; +Cc: palmer, Alistair Francis, alistair23

This patch fixes four different things, to maintain bisectability they
have been merged into a single patch. The following fixes are below:

sifive_plic: Fix incorrect irq calculation
The irq is incorrectly calculated to be off by one. It has worked in the
past as the priority_base offset has also been set incorrectly. We are
about to fix the priority_base offset so first first the irq
calculation.

sifive_u: Fix PLIC priority base offset and numbering
According to the FU540 manual the PLIC source priority address starts at
an offset of 0x04 and not 0x00. The same manual also specifies that the
PLIC only has 53 source priorities. Fix these two incorrect header
files.

We also need to over extend the plic_gpios[] array as the PLIC sources
count from 1 and not 0.

riscv: sifive_e: Fix PLIC priority base offset
According to the FE31 manual the PLIC source priority address starts at
an offset of 0x04 and not 0x00.

riscv: virt: Fix PLIC priority base offset
Update the virt offsets based on the newly updated SiFive U and SiFive E
offsets.

Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
---
 hw/riscv/sifive_plic.c      | 4 ++--
 hw/riscv/sifive_u.c         | 2 +-
 include/hw/riscv/sifive_e.h | 2 +-
 include/hw/riscv/sifive_u.h | 4 ++--
 include/hw/riscv/virt.h     | 2 +-
 5 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/hw/riscv/sifive_plic.c b/hw/riscv/sifive_plic.c
index 1c703e1a37..70a85cd075 100644
--- a/hw/riscv/sifive_plic.c
+++ b/hw/riscv/sifive_plic.c
@@ -206,7 +206,7 @@ static uint64_t sifive_plic_read(void *opaque, hwaddr addr, unsigned size)
     if (addr >= plic->priority_base && /* 4 bytes per source */
         addr < plic->priority_base + (plic->num_sources << 2))
     {
-        uint32_t irq = (addr - plic->priority_base) >> 2;
+        uint32_t irq = ((addr - plic->priority_base) >> 2) + 1;
         if (RISCV_DEBUG_PLIC) {
             qemu_log("plic: read priority: irq=%d priority=%d\n",
                 irq, plic->source_priority[irq]);
@@ -279,7 +279,7 @@ static void sifive_plic_write(void *opaque, hwaddr addr, uint64_t value,
     if (addr >= plic->priority_base && /* 4 bytes per source */
         addr < plic->priority_base + (plic->num_sources << 2))
     {
-        uint32_t irq = (addr - plic->priority_base) >> 2;
+        uint32_t irq = ((addr - plic->priority_base) >> 2) + 1;
         plic->source_priority[irq] = value & 7;
         if (RISCV_DEBUG_PLIC) {
             qemu_log("plic: write priority: irq=%d priority=%d\n",
diff --git a/hw/riscv/sifive_u.c b/hw/riscv/sifive_u.c
index 5ecc47cea3..88381a7507 100644
--- a/hw/riscv/sifive_u.c
+++ b/hw/riscv/sifive_u.c
@@ -340,7 +340,7 @@ static void riscv_sifive_u_soc_realize(DeviceState *dev, Error **errp)
     const struct MemmapEntry *memmap = sifive_u_memmap;
     MemoryRegion *system_memory = get_system_memory();
     MemoryRegion *mask_rom = g_new(MemoryRegion, 1);
-    qemu_irq plic_gpios[SIFIVE_U_PLIC_NUM_SOURCES];
+    qemu_irq plic_gpios[SIFIVE_U_PLIC_NUM_SOURCES + 1];
     int i;
     Error *err = NULL;
     NICInfo *nd = &nd_table[0];
diff --git a/include/hw/riscv/sifive_e.h b/include/hw/riscv/sifive_e.h
index 7b6d8aed96..f715f8606f 100644
--- a/include/hw/riscv/sifive_e.h
+++ b/include/hw/riscv/sifive_e.h
@@ -70,7 +70,7 @@ enum {
 #define SIFIVE_E_PLIC_HART_CONFIG "M"
 #define SIFIVE_E_PLIC_NUM_SOURCES 127
 #define SIFIVE_E_PLIC_NUM_PRIORITIES 7
-#define SIFIVE_E_PLIC_PRIORITY_BASE 0x0
+#define SIFIVE_E_PLIC_PRIORITY_BASE 0x04
 #define SIFIVE_E_PLIC_PENDING_BASE 0x1000
 #define SIFIVE_E_PLIC_ENABLE_BASE 0x2000
 #define SIFIVE_E_PLIC_ENABLE_STRIDE 0x80
diff --git a/include/hw/riscv/sifive_u.h b/include/hw/riscv/sifive_u.h
index be13cc1304..d859ea20f6 100644
--- a/include/hw/riscv/sifive_u.h
+++ b/include/hw/riscv/sifive_u.h
@@ -68,9 +68,9 @@ enum {
 };
 
 #define SIFIVE_U_PLIC_HART_CONFIG "MS"
-#define SIFIVE_U_PLIC_NUM_SOURCES 127
+#define SIFIVE_U_PLIC_NUM_SOURCES 53
 #define SIFIVE_U_PLIC_NUM_PRIORITIES 7
-#define SIFIVE_U_PLIC_PRIORITY_BASE 0x0
+#define SIFIVE_U_PLIC_PRIORITY_BASE 0x04
 #define SIFIVE_U_PLIC_PENDING_BASE 0x1000
 #define SIFIVE_U_PLIC_ENABLE_BASE 0x2000
 #define SIFIVE_U_PLIC_ENABLE_STRIDE 0x80
diff --git a/include/hw/riscv/virt.h b/include/hw/riscv/virt.h
index f12deaebd6..568764b570 100644
--- a/include/hw/riscv/virt.h
+++ b/include/hw/riscv/virt.h
@@ -59,7 +59,7 @@ enum {
 #define VIRT_PLIC_HART_CONFIG "MS"
 #define VIRT_PLIC_NUM_SOURCES 127
 #define VIRT_PLIC_NUM_PRIORITIES 7
-#define VIRT_PLIC_PRIORITY_BASE 0x0
+#define VIRT_PLIC_PRIORITY_BASE 0x04
 #define VIRT_PLIC_PENDING_BASE 0x1000
 #define VIRT_PLIC_ENABLE_BASE 0x2000
 #define VIRT_PLIC_ENABLE_STRIDE 0x80
-- 
2.21.0



^ permalink raw reply related	[flat|nested] 11+ messages in thread

* [Qemu-riscv] [PATCH for 4.0 v2 2/2] riscv: plic: Log guest errors
  2019-03-27 18:51 [Qemu-riscv] [PATCH for 4.0 v2 0/2] Update the QEMU PLIC addresses Alistair Francis
  2019-03-27 18:51 ` [Qemu-riscv] [PATCH for 4.0 v2 1/2] riscv: plic: Fix incorrect irq calculation Alistair Francis
@ 2019-03-27 18:51 ` Alistair Francis
  2019-03-27 18:52   ` Alistair Francis
       [not found] ` <5c9bc77e.1c69fb81.526f.a250SMTPIN_ADDED_BROKEN@mx.google.com>
  2019-03-28  3:23 ` [Qemu-riscv] [PATCH for 4.0 v2 0/2] Update the QEMU PLIC addresses Palmer Dabbelt
  3 siblings, 1 reply; 11+ messages in thread
From: Alistair Francis @ 2019-03-27 18:51 UTC (permalink / raw)
  To: qemu-devel, qemu-riscv; +Cc: palmer, Alistair Francis, alistair23

Instead of using error_report() to print guest errors let's use
qemu_log_mask(LOG_GUEST_ERROR,...) to log the error.

Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
---
 hw/riscv/sifive_plic.c | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/hw/riscv/sifive_plic.c b/hw/riscv/sifive_plic.c
index 70a85cd075..7f373d6c9d 100644
--- a/hw/riscv/sifive_plic.c
+++ b/hw/riscv/sifive_plic.c
@@ -262,7 +262,9 @@ static uint64_t sifive_plic_read(void *opaque, hwaddr addr, unsigned size)
     }
 
 err:
-    error_report("plic: invalid register read: %08x", (uint32_t)addr);
+    qemu_log_mask(LOG_GUEST_ERROR,
+                  "%s: Invalid register read 0x%" HWADDR_PRIx "\n",
+                  __func__, addr);
     return 0;
 }
 
@@ -289,7 +291,9 @@ static void sifive_plic_write(void *opaque, hwaddr addr, uint64_t value,
     } else if (addr >= plic->pending_base && /* 1 bit per source */
                addr < plic->pending_base + (plic->num_sources >> 3))
     {
-        error_report("plic: invalid pending write: %08x", (uint32_t)addr);
+        qemu_log_mask(LOG_GUEST_ERROR,
+                      "%s: invalid pending write: 0x%" HWADDR_PRIx "",
+                      __func__, addr);
         return;
     } else if (addr >= plic->enable_base && /* 1 bit per source */
         addr < plic->enable_base + plic->num_addrs * plic->enable_stride)
@@ -339,7 +343,9 @@ static void sifive_plic_write(void *opaque, hwaddr addr, uint64_t value,
     }
 
 err:
-    error_report("plic: invalid register write: %08x", (uint32_t)addr);
+    qemu_log_mask(LOG_GUEST_ERROR,
+                  "%s: Invalid register write 0x%" HWADDR_PRIx "\n",
+                  __func__, addr);
 }
 
 static const MemoryRegionOps sifive_plic_ops = {
-- 
2.21.0



^ permalink raw reply related	[flat|nested] 11+ messages in thread

* Re: [Qemu-riscv] [PATCH for 4.0 v2 2/2] riscv: plic: Log guest errors
  2019-03-27 18:51 ` [Qemu-riscv] [PATCH for 4.0 v2 2/2] riscv: plic: Log guest errors Alistair Francis
@ 2019-03-27 18:52   ` Alistair Francis
  2019-03-28  3:23     ` Palmer Dabbelt
  0 siblings, 1 reply; 11+ messages in thread
From: Alistair Francis @ 2019-03-27 18:52 UTC (permalink / raw)
  To: Alistair Francis; +Cc: qemu-devel, qemu-riscv, palmer

On Wed, Mar 27, 2019 at 11:51 AM Alistair Francis
<Alistair.Francis@wdc.com> wrote:
>
> Instead of using error_report() to print guest errors let's use
> qemu_log_mask(LOG_GUEST_ERROR,...) to log the error.
>
> Signed-off-by: Alistair Francis <alistair.francis@wdc.com>

I dropped Philippe's review in this series.
This should be included from v1:

Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

Alistair

> ---
>  hw/riscv/sifive_plic.c | 12 +++++++++---
>  1 file changed, 9 insertions(+), 3 deletions(-)
>
> diff --git a/hw/riscv/sifive_plic.c b/hw/riscv/sifive_plic.c
> index 70a85cd075..7f373d6c9d 100644
> --- a/hw/riscv/sifive_plic.c
> +++ b/hw/riscv/sifive_plic.c
> @@ -262,7 +262,9 @@ static uint64_t sifive_plic_read(void *opaque, hwaddr addr, unsigned size)
>      }
>
>  err:
> -    error_report("plic: invalid register read: %08x", (uint32_t)addr);
> +    qemu_log_mask(LOG_GUEST_ERROR,
> +                  "%s: Invalid register read 0x%" HWADDR_PRIx "\n",
> +                  __func__, addr);
>      return 0;
>  }
>
> @@ -289,7 +291,9 @@ static void sifive_plic_write(void *opaque, hwaddr addr, uint64_t value,
>      } else if (addr >= plic->pending_base && /* 1 bit per source */
>                 addr < plic->pending_base + (plic->num_sources >> 3))
>      {
> -        error_report("plic: invalid pending write: %08x", (uint32_t)addr);
> +        qemu_log_mask(LOG_GUEST_ERROR,
> +                      "%s: invalid pending write: 0x%" HWADDR_PRIx "",
> +                      __func__, addr);
>          return;
>      } else if (addr >= plic->enable_base && /* 1 bit per source */
>          addr < plic->enable_base + plic->num_addrs * plic->enable_stride)
> @@ -339,7 +343,9 @@ static void sifive_plic_write(void *opaque, hwaddr addr, uint64_t value,
>      }
>
>  err:
> -    error_report("plic: invalid register write: %08x", (uint32_t)addr);
> +    qemu_log_mask(LOG_GUEST_ERROR,
> +                  "%s: Invalid register write 0x%" HWADDR_PRIx "\n",
> +                  __func__, addr);
>  }
>
>  static const MemoryRegionOps sifive_plic_ops = {
> --
> 2.21.0
>


^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [Qemu-riscv] [Qemu-devel] [PATCH for 4.0 v2 2/2] riscv: plic: Log guest errors
       [not found] ` <5c9bc77e.1c69fb81.526f.a250SMTPIN_ADDED_BROKEN@mx.google.com>
@ 2019-03-27 22:02   ` Philippe Mathieu-Daudé
  0 siblings, 0 replies; 11+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-03-27 22:02 UTC (permalink / raw)
  To: Alistair Francis
  Cc: qemu-devel@nongnu.org Developers, qemu-riscv, alistair23, palmer

[-- Attachment #1: Type: text/plain, Size: 2362 bytes --]

Le mer. 27 mars 2019 19:57, Alistair Francis <Alistair.Francis@wdc.com> a
écrit :

> Instead of using error_report() to print guest errors let's use
> qemu_log_mask(LOG_GUEST_ERROR,...) to log the error.
>
> Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
> ---
>  hw/riscv/sifive_plic.c | 12 +++++++++---
>  1 file changed, 9 insertions(+), 3 deletions(-)
>
> diff --git a/hw/riscv/sifive_plic.c b/hw/riscv/sifive_plic.c
> index 70a85cd075..7f373d6c9d 100644
> --- a/hw/riscv/sifive_plic.c
> +++ b/hw/riscv/sifive_plic.c
> @@ -262,7 +262,9 @@ static uint64_t sifive_plic_read(void *opaque, hwaddr
> addr, unsigned size)
>      }
>
>  err:
> -    error_report("plic: invalid register read: %08x", (uint32_t)addr);
> +    qemu_log_mask(LOG_GUEST_ERROR,
> +                  "%s: Invalid register read 0x%" HWADDR_PRIx "\n",
> +                  __func__, addr);
>      return 0;
>  }
>
> @@ -289,7 +291,9 @@ static void sifive_plic_write(void *opaque, hwaddr
> addr, uint64_t value,
>      } else if (addr >= plic->pending_base && /* 1 bit per source */
>                 addr < plic->pending_base + (plic->num_sources >> 3))
>      {
> -        error_report("plic: invalid pending write: %08x", (uint32_t)addr);
> +        qemu_log_mask(LOG_GUEST_ERROR,
> +                      "%s: invalid pending write: 0x%" HWADDR_PRIx "",
> +                      __func__, addr);
>          return;
>      } else if (addr >= plic->enable_base && /* 1 bit per source */
>          addr < plic->enable_base + plic->num_addrs * plic->enable_stride)
> @@ -339,7 +343,9 @@ static void sifive_plic_write(void *opaque, hwaddr
> addr, uint64_t value,
>      }
>
>  err:
> -    error_report("plic: invalid register write: %08x", (uint32_t)addr);
> +    qemu_log_mask(LOG_GUEST_ERROR,
> +                  "%s: Invalid register write 0x%" HWADDR_PRIx "\n",
> +                  __func__, addr);
>  }
>
>  static const MemoryRegionOps sifive_plic_ops = {
> --
> 2.21.0
>

I'm using the GMail embedded gapp and wonder if the deduplication features
is not sometimes abusive, when the same patch is sent increasing versions,
I reply to the last version I see but my reply looks like I replied to the
first.
I might be also misusing the gapp :)
Anyway:
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

>

[-- Attachment #2: Type: text/html, Size: 3379 bytes --]

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [Qemu-riscv] [PATCH for 4.0 v2 0/2] Update the QEMU PLIC addresses
  2019-03-27 18:51 [Qemu-riscv] [PATCH for 4.0 v2 0/2] Update the QEMU PLIC addresses Alistair Francis
                   ` (2 preceding siblings ...)
       [not found] ` <5c9bc77e.1c69fb81.526f.a250SMTPIN_ADDED_BROKEN@mx.google.com>
@ 2019-03-28  3:23 ` Palmer Dabbelt
  2019-04-03 23:32     ` [Qemu-riscv] " Alistair Francis
  3 siblings, 1 reply; 11+ messages in thread
From: Palmer Dabbelt @ 2019-03-28  3:23 UTC (permalink / raw)
  To: Alistair Francis; +Cc: qemu-devel, qemu-riscv, Alistair Francis, alistair23

On Wed, 27 Mar 2019 11:51:15 PDT (-0700), Alistair Francis wrote:
> This series updates the PLIC address to match the documentation.
>
> This fixes: https://github.com/riscv/opensbi/issues/97
>
> V2:
>  - Squash patches to ensure biesctability
>
> Alistair Francis (2):
>   riscv: plic: Fix incorrect irq calculation
>   riscv: plic: Log guest errors
>
>  hw/riscv/sifive_plic.c      | 16 +++++++++++-----
>  hw/riscv/sifive_u.c         |  2 +-
>  include/hw/riscv/sifive_e.h |  2 +-
>  include/hw/riscv/sifive_u.h |  4 ++--
>  include/hw/riscv/virt.h     |  2 +-
>  5 files changed, 16 insertions(+), 10 deletions(-)
>
> -- 
> 2.21.0

Thanks, I've got these on for-master.  I'll let them sit for a bit to see if 
there are any other comments, but

Reviewed-by: Palmer Dabbelt <palmer@sifive.com>


^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [Qemu-riscv] [PATCH for 4.0 v2 2/2] riscv: plic: Log guest errors
  2019-03-27 18:52   ` Alistair Francis
@ 2019-03-28  3:23     ` Palmer Dabbelt
  0 siblings, 0 replies; 11+ messages in thread
From: Palmer Dabbelt @ 2019-03-28  3:23 UTC (permalink / raw)
  To: alistair23; +Cc: Alistair Francis, qemu-devel, qemu-riscv

On Wed, 27 Mar 2019 11:52:53 PDT (-0700), alistair23@gmail.com wrote:
> On Wed, Mar 27, 2019 at 11:51 AM Alistair Francis
> <Alistair.Francis@wdc.com> wrote:
>>
>> Instead of using error_report() to print guest errors let's use
>> qemu_log_mask(LOG_GUEST_ERROR,...) to log the error.
>>
>> Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
>
> I dropped Philippe's review in this series.
> This should be included from v1:
>
> Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

I got it.

>
> Alistair
>
>> ---
>>  hw/riscv/sifive_plic.c | 12 +++++++++---
>>  1 file changed, 9 insertions(+), 3 deletions(-)
>>
>> diff --git a/hw/riscv/sifive_plic.c b/hw/riscv/sifive_plic.c
>> index 70a85cd075..7f373d6c9d 100644
>> --- a/hw/riscv/sifive_plic.c
>> +++ b/hw/riscv/sifive_plic.c
>> @@ -262,7 +262,9 @@ static uint64_t sifive_plic_read(void *opaque, hwaddr addr, unsigned size)
>>      }
>>
>>  err:
>> -    error_report("plic: invalid register read: %08x", (uint32_t)addr);
>> +    qemu_log_mask(LOG_GUEST_ERROR,
>> +                  "%s: Invalid register read 0x%" HWADDR_PRIx "\n",
>> +                  __func__, addr);
>>      return 0;
>>  }
>>
>> @@ -289,7 +291,9 @@ static void sifive_plic_write(void *opaque, hwaddr addr, uint64_t value,
>>      } else if (addr >= plic->pending_base && /* 1 bit per source */
>>                 addr < plic->pending_base + (plic->num_sources >> 3))
>>      {
>> -        error_report("plic: invalid pending write: %08x", (uint32_t)addr);
>> +        qemu_log_mask(LOG_GUEST_ERROR,
>> +                      "%s: invalid pending write: 0x%" HWADDR_PRIx "",
>> +                      __func__, addr);
>>          return;
>>      } else if (addr >= plic->enable_base && /* 1 bit per source */
>>          addr < plic->enable_base + plic->num_addrs * plic->enable_stride)
>> @@ -339,7 +343,9 @@ static void sifive_plic_write(void *opaque, hwaddr addr, uint64_t value,
>>      }
>>
>>  err:
>> -    error_report("plic: invalid register write: %08x", (uint32_t)addr);
>> +    qemu_log_mask(LOG_GUEST_ERROR,
>> +                  "%s: Invalid register write 0x%" HWADDR_PRIx "\n",
>> +                  __func__, addr);
>>  }
>>
>>  static const MemoryRegionOps sifive_plic_ops = {
>> --
>> 2.21.0
>>


^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [Qemu-devel] [PATCH for 4.0 v2 0/2] Update the QEMU PLIC addresses
  2019-03-28  3:23 ` [Qemu-riscv] [PATCH for 4.0 v2 0/2] Update the QEMU PLIC addresses Palmer Dabbelt
@ 2019-04-03 23:32     ` Alistair Francis
  0 siblings, 0 replies; 11+ messages in thread
From: Alistair Francis @ 2019-04-03 23:32 UTC (permalink / raw)
  To: Palmer Dabbelt
  Cc: Alistair Francis, qemu-devel@nongnu.org Developers, open list:RISC-V

On Wed, Mar 27, 2019 at 8:23 PM Palmer Dabbelt <palmer@sifive.com> wrote:
>
> On Wed, 27 Mar 2019 11:51:15 PDT (-0700), Alistair Francis wrote:
> > This series updates the PLIC address to match the documentation.
> >
> > This fixes: https://github.com/riscv/opensbi/issues/97
> >
> > V2:
> >  - Squash patches to ensure biesctability
> >
> > Alistair Francis (2):
> >   riscv: plic: Fix incorrect irq calculation
> >   riscv: plic: Log guest errors
> >
> >  hw/riscv/sifive_plic.c      | 16 +++++++++++-----
> >  hw/riscv/sifive_u.c         |  2 +-
> >  include/hw/riscv/sifive_e.h |  2 +-
> >  include/hw/riscv/sifive_u.h |  4 ++--
> >  include/hw/riscv/virt.h     |  2 +-
> >  5 files changed, 16 insertions(+), 10 deletions(-)
> >
> > --
> > 2.21.0
>
> Thanks, I've got these on for-master.  I'll let them sit for a bit to see if
> there are any other comments, but
>
> Reviewed-by: Palmer Dabbelt <palmer@sifive.com>

Ping! Just want to make sure these make it into 4.0

Alistair

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [Qemu-riscv] [PATCH for 4.0 v2 0/2] Update the QEMU PLIC addresses
@ 2019-04-03 23:32     ` Alistair Francis
  0 siblings, 0 replies; 11+ messages in thread
From: Alistair Francis @ 2019-04-03 23:32 UTC (permalink / raw)
  To: Palmer Dabbelt
  Cc: Alistair Francis, qemu-devel@nongnu.org Developers, open list:RISC-V

On Wed, Mar 27, 2019 at 8:23 PM Palmer Dabbelt <palmer@sifive.com> wrote:
>
> On Wed, 27 Mar 2019 11:51:15 PDT (-0700), Alistair Francis wrote:
> > This series updates the PLIC address to match the documentation.
> >
> > This fixes: https://github.com/riscv/opensbi/issues/97
> >
> > V2:
> >  - Squash patches to ensure biesctability
> >
> > Alistair Francis (2):
> >   riscv: plic: Fix incorrect irq calculation
> >   riscv: plic: Log guest errors
> >
> >  hw/riscv/sifive_plic.c      | 16 +++++++++++-----
> >  hw/riscv/sifive_u.c         |  2 +-
> >  include/hw/riscv/sifive_e.h |  2 +-
> >  include/hw/riscv/sifive_u.h |  4 ++--
> >  include/hw/riscv/virt.h     |  2 +-
> >  5 files changed, 16 insertions(+), 10 deletions(-)
> >
> > --
> > 2.21.0
>
> Thanks, I've got these on for-master.  I'll let them sit for a bit to see if
> there are any other comments, but
>
> Reviewed-by: Palmer Dabbelt <palmer@sifive.com>

Ping! Just want to make sure these make it into 4.0

Alistair


^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [Qemu-devel] [PATCH for 4.0 v2 0/2] Update the QEMU PLIC addresses
  2019-04-03 23:32     ` [Qemu-riscv] " Alistair Francis
@ 2019-04-04  1:00       ` Palmer Dabbelt
  -1 siblings, 0 replies; 11+ messages in thread
From: Palmer Dabbelt @ 2019-04-04  1:00 UTC (permalink / raw)
  To: alistair23; +Cc: Alistair Francis, qemu-devel, qemu-riscv

On Wed, 03 Apr 2019 16:32:11 PDT (-0700), alistair23@gmail.com wrote:
> On Wed, Mar 27, 2019 at 8:23 PM Palmer Dabbelt <palmer@sifive.com> wrote:
>>
>> On Wed, 27 Mar 2019 11:51:15 PDT (-0700), Alistair Francis wrote:
>> > This series updates the PLIC address to match the documentation.
>> >
>> > This fixes: https://github.com/riscv/opensbi/issues/97
>> >
>> > V2:
>> >  - Squash patches to ensure biesctability
>> >
>> > Alistair Francis (2):
>> >   riscv: plic: Fix incorrect irq calculation
>> >   riscv: plic: Log guest errors
>> >
>> >  hw/riscv/sifive_plic.c      | 16 +++++++++++-----
>> >  hw/riscv/sifive_u.c         |  2 +-
>> >  include/hw/riscv/sifive_e.h |  2 +-
>> >  include/hw/riscv/sifive_u.h |  4 ++--
>> >  include/hw/riscv/virt.h     |  2 +-
>> >  5 files changed, 16 insertions(+), 10 deletions(-)
>> >
>> > --
>> > 2.21.0
>>
>> Thanks, I've got these on for-master.  I'll let them sit for a bit to see if
>> there are any other comments, but
>>
>> Reviewed-by: Palmer Dabbelt <palmer@sifive.com>
>
> Ping! Just want to make sure these make it into 4.0

Thanks, this got lost in the shuffle.  I just forwarded along the PR.

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [Qemu-riscv] [PATCH for 4.0 v2 0/2] Update the QEMU PLIC addresses
@ 2019-04-04  1:00       ` Palmer Dabbelt
  0 siblings, 0 replies; 11+ messages in thread
From: Palmer Dabbelt @ 2019-04-04  1:00 UTC (permalink / raw)
  To: alistair23; +Cc: Alistair Francis, qemu-devel, qemu-riscv

On Wed, 03 Apr 2019 16:32:11 PDT (-0700), alistair23@gmail.com wrote:
> On Wed, Mar 27, 2019 at 8:23 PM Palmer Dabbelt <palmer@sifive.com> wrote:
>>
>> On Wed, 27 Mar 2019 11:51:15 PDT (-0700), Alistair Francis wrote:
>> > This series updates the PLIC address to match the documentation.
>> >
>> > This fixes: https://github.com/riscv/opensbi/issues/97
>> >
>> > V2:
>> >  - Squash patches to ensure biesctability
>> >
>> > Alistair Francis (2):
>> >   riscv: plic: Fix incorrect irq calculation
>> >   riscv: plic: Log guest errors
>> >
>> >  hw/riscv/sifive_plic.c      | 16 +++++++++++-----
>> >  hw/riscv/sifive_u.c         |  2 +-
>> >  include/hw/riscv/sifive_e.h |  2 +-
>> >  include/hw/riscv/sifive_u.h |  4 ++--
>> >  include/hw/riscv/virt.h     |  2 +-
>> >  5 files changed, 16 insertions(+), 10 deletions(-)
>> >
>> > --
>> > 2.21.0
>>
>> Thanks, I've got these on for-master.  I'll let them sit for a bit to see if
>> there are any other comments, but
>>
>> Reviewed-by: Palmer Dabbelt <palmer@sifive.com>
>
> Ping! Just want to make sure these make it into 4.0

Thanks, this got lost in the shuffle.  I just forwarded along the PR.


^ permalink raw reply	[flat|nested] 11+ messages in thread

end of thread, other threads:[~2019-04-04  1:01 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-03-27 18:51 [Qemu-riscv] [PATCH for 4.0 v2 0/2] Update the QEMU PLIC addresses Alistair Francis
2019-03-27 18:51 ` [Qemu-riscv] [PATCH for 4.0 v2 1/2] riscv: plic: Fix incorrect irq calculation Alistair Francis
2019-03-27 18:51 ` [Qemu-riscv] [PATCH for 4.0 v2 2/2] riscv: plic: Log guest errors Alistair Francis
2019-03-27 18:52   ` Alistair Francis
2019-03-28  3:23     ` Palmer Dabbelt
     [not found] ` <5c9bc77e.1c69fb81.526f.a250SMTPIN_ADDED_BROKEN@mx.google.com>
2019-03-27 22:02   ` [Qemu-riscv] [Qemu-devel] " Philippe Mathieu-Daudé
2019-03-28  3:23 ` [Qemu-riscv] [PATCH for 4.0 v2 0/2] Update the QEMU PLIC addresses Palmer Dabbelt
2019-04-03 23:32   ` [Qemu-devel] " Alistair Francis
2019-04-03 23:32     ` [Qemu-riscv] " Alistair Francis
2019-04-04  1:00     ` [Qemu-devel] " Palmer Dabbelt
2019-04-04  1:00       ` [Qemu-riscv] " Palmer Dabbelt

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.