[v1] hw/nvme: shadow doorbells broken on riscv64

[PATCH 0/1] hw/nvme: shadow doorbells broken on riscv64

Klaus Jensen posted 1 patch 1 year, 4 months ago

Diff against v2
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20221208082955.51732-1-its@irrelevant.dk

Maintainers: Keith Busch <kbusch@kernel.org>, Klaus Jensen <its@irrelevant.dk>

There is a newer version of this series

hw/nvme/ctrl.c | 8 ++++++++
1 file changed, 8 insertions(+)

Expand all Fold all

[PATCH 0/1] hw/nvme: shadow doorbells broken on riscv64

Posted by Klaus Jensen 1 year, 4 months ago

From: Klaus Jensen <k.jensen@samsung.com>

Guenter reports[1] that hw/nvme is broken on riscv64.

This is a regression since 7.1, so this does not warrent an rc5 for 7.2.
I'm sure Guenter can carry this patch in his tree, and maybe we can get
this out in a stable release.

I really wonder why this issue only shows up on riscv64. We have not
observed this on other platforms (yet).

  [1]: https://lore.kernel.org/qemu-devel/20221207174918.GA1151796@roeck-us.net/

Klaus Jensen (1):
  hw/nvme: fix missing cq eventidx update

 hw/nvme/ctrl.c | 8 ++++++++
 1 file changed, 8 insertions(+)

-- 
2.38.1

Re: [PATCH 0/1] hw/nvme: shadow doorbells broken on riscv64

Posted by Philippe Mathieu-Daudé 1 year, 4 months ago

On 8/12/22 09:29, Klaus Jensen wrote:
> From: Klaus Jensen <k.jensen@samsung.com>
> 
> Guenter reports[1] that hw/nvme is broken on riscv64.
> 
> This is a regression since 7.1, so this does not warrent an rc5 for 7.2.
> I'm sure Guenter can carry this patch in his tree, and maybe we can get
> this out in a stable release.

Delaying to 8.0 and adding a 'Cc: qemu-stable@' tag seems wise.

Cc'ing riscv list in case.

> I really wonder why this issue only shows up on riscv64. We have not
> observed this on other platforms (yet).
> 
>    [1]: https://lore.kernel.org/qemu-devel/20221207174918.GA1151796@roeck-us.net/
> 
> Klaus Jensen (1):
>    hw/nvme: fix missing cq eventidx update
> 
>   hw/nvme/ctrl.c | 8 ++++++++
>   1 file changed, 8 insertions(+)
>