[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 05/22] hw/arm: Select VIRTIO_NET for virt machine
From: |
Peter Maydell |
Subject: |
Re: [PATCH 05/22] hw/arm: Select VIRTIO_NET for virt machine |
Date: |
Wed, 3 May 2023 19:32:45 +0100 |
On Wed, 3 May 2023 at 18:06, Paolo Bonzini <pbonzini@redhat.com> wrote:
>
> On 5/3/23 17:35, Alex Bennée wrote:
> >> You should select a device only if you cannot even start
> >> the machine without --nodefaults.
> >
> > Which is the case here right? We could skip tests that explicitly
> > instantiate a device but these are tests failing with default devices
> > the machine tries to instantiate.
>
> I'm sorry, I meant "select" directives are needed if you cannot even
> start the machine *with* --nodefaults.
>
> Devices that are added *without* --nodefaults should use "imply"
> directives instead, as is already the case.
Do we really want to build a QEMU that then barfs unless
you pass -nodefaults, though ? That doesn't seem very useful.
Something somewhere ought to be saying "if you want the
virt board then you almost certainly want these". Or
alternatively we should fall back to "don't create a
network device we don't have", maybe ?
-- PMM
- [PATCH 00/22] testing/next: cirrus, docker, docs, ci, configs, gitlab, Alex Bennée, 2023/05/03
- [PATCH 01/22] gitlab/cirrus: reduce scope of the FreeBSD testing matrix, Alex Bennée, 2023/05/03
- [PATCH 05/22] hw/arm: Select VIRTIO_NET for virt machine, Alex Bennée, 2023/05/03
- Re: [PATCH 05/22] hw/arm: Select VIRTIO_NET for virt machine, Paolo Bonzini, 2023/05/03
- Re: [PATCH 05/22] hw/arm: Select VIRTIO_NET for virt machine, Fabiano Rosas, 2023/05/03
- Re: [PATCH 05/22] hw/arm: Select VIRTIO_NET for virt machine, Alex Bennée, 2023/05/03
- Re: [PATCH 05/22] hw/arm: Select VIRTIO_NET for virt machine, Paolo Bonzini, 2023/05/03
- Re: [PATCH 05/22] hw/arm: Select VIRTIO_NET for virt machine,
Peter Maydell <=
- Re: [PATCH 05/22] hw/arm: Select VIRTIO_NET for virt machine, Thomas Huth, 2023/05/04
- Re: [PATCH 05/22] hw/arm: Select VIRTIO_NET for virt machine, Alex Bennée, 2023/05/04
- Re: [PATCH 05/22] hw/arm: Select VIRTIO_NET for virt machine, Paolo Bonzini, 2023/05/04
- Re: [PATCH 05/22] hw/arm: Select VIRTIO_NET for virt machine, Fabiano Rosas, 2023/05/04
- Re: [PATCH 05/22] hw/arm: Select VIRTIO_NET for virt machine, Paolo Bonzini, 2023/05/08
- Re: [PATCH 05/22] hw/arm: Select VIRTIO_NET for virt machine, Peter Maydell, 2023/05/09
- Re: [PATCH 05/22] hw/arm: Select VIRTIO_NET for virt machine, Paolo Bonzini, 2023/05/09
- Re: [PATCH 05/22] hw/arm: Select VIRTIO_NET for virt machine, Peter Maydell, 2023/05/09
- Re: [PATCH 05/22] hw/arm: Select VIRTIO_NET for virt machine, Paolo Bonzini, 2023/05/09
- Re: [PATCH 05/22] hw/arm: Select VIRTIO_NET for virt machine, Alex Bennée, 2023/05/09