Re: [Qemu-devel] [PATCH] sun4u: fix assert when adding NICs which aren't the in-built model

[Artyom Tarasenko]

On Sun, Oct 15, 2017 at 11:21 AM, Mark Cave-Ayland
<address@hidden> wrote:
> Commit 8d93297 introduced a bug whereby non-inbuilt NICs are realized before
> setting the default MAC address causing an assert. Switch NIC creation
> over from pci_create_simple() to pci_create() which works exactly the
> same except omitting the realize as originally intended.
>
> Signed-off-by: Mark Cave-Ayland <address@hidden>

Reviewed-by: Artyom Tarasenko <address@hidden>

> ---
>  hw/sparc64/sun4u.c |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/hw/sparc64/sun4u.c b/hw/sparc64/sun4u.c
> index 2edd1cc..bf89252 100644
> --- a/hw/sparc64/sun4u.c
> +++ b/hw/sparc64/sun4u.c
> @@ -490,10 +490,10 @@ static void sun4uv_init(MemoryRegion *address_space_mem,
>                  memcpy(&macaddr, &nd->macaddr.a, sizeof(MACAddr));
>                  onboard_nic = true;
>              } else {
> -                pci_dev = pci_create_simple(pci_busB, -1, "sunhme");
> +                pci_dev = pci_create(pci_busB, -1, "sunhme");
>              }
>          } else {
> -            pci_dev = pci_create_simple(pci_busB, -1, nd->model);
> +            pci_dev = pci_create(pci_busB, -1, nd->model);
>          }
>
>          dev = &pci_dev->qdev;
> --
> 1.7.10.4
>



-- 
Regards,
Artyom Tarasenko

SPARC and PPC PReP under qemu blog: http://tyom.blogspot.com/search/label/qemu