[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 08/16] net: Remove VLANState
From: |
Laszlo Ersek |
Subject: |
Re: [Qemu-devel] [PATCH 08/16] net: Remove VLANState |
Date: |
Mon, 23 Jul 2012 22:57:58 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:10.0.6esrpre) Gecko/20120714 Thunderbird/10.0.6 |
On 07/20/12 14:01, Stefan Hajnoczi wrote:
> diff --git a/hw/exynos4_boards.c b/hw/exynos4_boards.c
> index e5c2a5f..4bb0a60 100644
> --- a/hw/exynos4_boards.c
> +++ b/hw/exynos4_boards.c
> @@ -81,7 +81,7 @@ static void lan9215_init(uint32_t base, qemu_irq irq)
> SysBusDevice *s;
>
> /* This should be a 9215 but the 9118 is close enough */
> - if (nd_table[0].vlan) {
> + if (nd_table[0].used) {
> qemu_check_nic_model(&nd_table[0], "lan9118");
> dev = qdev_create(NULL, "lan9118");
> qdev_set_nic_properties(dev, &nd_table[0]);
Right, relevant commits: 0ae18cee & 7697079b.
> @@ -970,7 +895,7 @@ void qmp_set_link(const char *name, bool up, Error **errp)
> done:
>
> if (!vc) {
> - error_set(errp, QERR_DEVICE_NOT_FOUND, name);
> + qerror_report(QERR_DEVICE_NOT_FOUND, name);
> return;
> }
Why? The only caller (at this point) seems to be hmp_set_link(), and it
handles the error (with monitor_printf()).
Thanks,
Laszlo
- [Qemu-devel] [PATCH 00/16] net: Move legacy QEMU VLAN code into net/hub.c, Stefan Hajnoczi, 2012/07/20
- [Qemu-devel] [PATCH 08/16] net: Remove VLANState, Stefan Hajnoczi, 2012/07/20
- Re: [Qemu-devel] [PATCH 08/16] net: Remove VLANState,
Laszlo Ersek <=
- [Qemu-devel] [PATCH 14/16] net: cleanup deliver/deliver_iov func pointers, Stefan Hajnoczi, 2012/07/20
- [Qemu-devel] [PATCH 07/16] net: Remove vlan code from net.c, Stefan Hajnoczi, 2012/07/20
- [Qemu-devel] [PATCH 06/16] net: Convert qdev_prop_vlan to peer with hub, Stefan Hajnoczi, 2012/07/20
- [Qemu-devel] [PATCH 11/16] net: Rename vc local variables to nc, Stefan Hajnoczi, 2012/07/20
- [Qemu-devel] [PATCH 16/16] hub: add the support for hub own flow control, Stefan Hajnoczi, 2012/07/20