[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 4/6] hypertrace: [softmmu] Add QEMU-side prox
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH v2 4/6] hypertrace: [softmmu] Add QEMU-side proxy to "guest_hypertrace" event |
Date: |
Wed, 14 Sep 2016 15:37:32 +0100 |
User-agent: |
Mutt/1.7.0 (2016-08-17) |
On Tue, Sep 06, 2016 at 04:43:00AM +0300, Michael S. Tsirkin wrote:
> On Mon, Sep 05, 2016 at 08:56:44PM +0200, Lluís Vilanova wrote:
> > diff --git a/include/hw/pci/pci.h b/include/hw/pci/pci.h
> > index 929ec2f..8973f57 100644
> > --- a/include/hw/pci/pci.h
> > +++ b/include/hw/pci/pci.h
> > @@ -80,6 +80,8 @@
> > #define PCI_DEVICE_ID_VIRTIO_RNG 0x1005
> > #define PCI_DEVICE_ID_VIRTIO_9P 0x1009
> >
> > +#define PCI_DEVICE_ID_HYPERTRACE 0x10f0
> > +
> > #define PCI_VENDOR_ID_REDHAT 0x1b36
> > #define PCI_DEVICE_ID_REDHAT_BRIDGE 0x0001
> > #define PCI_DEVICE_ID_REDHAT_SERIAL 0x0002
>
>
> There's no need to add IDs to this header.
> We should probably drop the ones we currently have in pci.h, too.
Defining the PCI IDs in public headers gives libqos test cases access to
them. If they are scoped to only be available in device emulation code
then the test cases need to duplicate them, which is undesirable IMO.
Stefan
signature.asc
Description: PGP signature
- [Qemu-devel] [PATCH v2 0/6] hypertrace: Lightweight guest-to-QEMU trace channel, Lluís Vilanova, 2016/09/05
- [Qemu-devel] [PATCH v2 2/6] hypertrace: Add tracing event "guest_hypertrace", Lluís Vilanova, 2016/09/05
- [Qemu-devel] [PATCH v2 1/6] hypertrace: Add documentation, Lluís Vilanova, 2016/09/05
- [Qemu-devel] [PATCH v2 4/6] hypertrace: [softmmu] Add QEMU-side proxy to "guest_hypertrace" event, Lluís Vilanova, 2016/09/05
- [Qemu-devel] [PATCH v2 3/6] hypertrace: [*-user] Add QEMU-side proxy to "guest_hypertrace" event, Lluís Vilanova, 2016/09/05
- [Qemu-devel] [PATCH v2 5/6] hypertrace: Add guest-side user-level library, Lluís Vilanova, 2016/09/05
- [Qemu-devel] [PATCH v2 6/6] hypertrace: Add guest-side Linux module, Lluís Vilanova, 2016/09/05
- Re: [Qemu-devel] [PATCH v2 0/6] hypertrace: Lightweight guest-to-QEMU trace channel, no-reply, 2016/09/05