On Sun, Oct 15, 2017 at 11:21 AM, Mark Cave-Ayland
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
Reviewed-by: Artyom Tarasenko
> ---
> 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