Re: [PATCH] hw/intc: sifive_plic: Fix multi-socket plic configuraiton

2022-07-27 Thread Atish Kumar Patra
On Wed, Jul 27, 2022 at 5:23 AM Alistair Francis wrote: > On Sat, Jul 23, 2022 at 7:22 PM Atish Patra wrote: > > > > Since commit 40244040a7ac, multi-socket configuration with plic is > > broken as the hartid for second socket is calculated incorrectly. > > The hartid stored in addr_config

Re: [PATCH] hw/intc: sifive_plic: Fix multi-socket plic configuraiton

2022-07-27 Thread Alistair Francis
On Sat, Jul 23, 2022 at 7:22 PM Atish Patra wrote: > > Since commit 40244040a7ac, multi-socket configuration with plic is > broken as the hartid for second socket is calculated incorrectly. > The hartid stored in addr_config already includes the offset > for the base hartid for that socket.

Re: [PATCH] hw/intc: sifive_plic: Fix multi-socket plic configuraiton

2022-07-27 Thread Alistair Francis
On Tue, Jul 26, 2022 at 8:30 AM Atish Kumar Patra wrote: > > > > > On Sun, Jul 24, 2022 at 6:14 PM Alistair Francis wrote: >> >> On Sat, Jul 23, 2022 at 7:22 PM Atish Patra wrote: >> > >> > Since commit 40244040a7ac, multi-socket configuration with plic is >> > broken as the hartid for second

Re: [PATCH] hw/intc: sifive_plic: Fix multi-socket plic configuraiton

2022-07-25 Thread Atish Kumar Patra
On Sun, Jul 24, 2022 at 6:14 PM Alistair Francis wrote: > On Sat, Jul 23, 2022 at 7:22 PM Atish Patra wrote: > > > > Since commit 40244040a7ac, multi-socket configuration with plic is > > broken as the hartid for second socket is calculated incorrectly. > > The hartid stored in addr_config

Re: [PATCH] hw/intc: sifive_plic: Fix multi-socket plic configuraiton

2022-07-24 Thread Alistair Francis
On Sat, Jul 23, 2022 at 7:22 PM Atish Patra wrote: > > Since commit 40244040a7ac, multi-socket configuration with plic is > broken as the hartid for second socket is calculated incorrectly. > The hartid stored in addr_config already includes the offset > for the base hartid for that socket.

[PATCH] hw/intc: sifive_plic: Fix multi-socket plic configuraiton

2022-07-23 Thread Atish Patra
Since commit 40244040a7ac, multi-socket configuration with plic is broken as the hartid for second socket is calculated incorrectly. The hartid stored in addr_config already includes the offset for the base hartid for that socket. Adding it again would lead to segfault while creating the plic