All of lore.kernel.org
 help / color / mirror / Atom feed
From: Juan Quintela <quintela@redhat.com>
To: "Michael S. Tsirkin" <mst@redhat.com>
Cc: qemu-devel@nongnu.org, lvivier@redhat.com, dgilbert@redhat.com,
	peterx@redhat.com
Subject: Re: [Qemu-devel] [PATCH v2 00/16] Virtio devices split from virtio-pci
Date: Thu, 29 Nov 2018 14:46:01 +0100	[thread overview]
Message-ID: <87h8g00zfq.fsf@trasno.org> (raw)
In-Reply-To: <20181128101603-mutt-send-email-mst@kernel.org> (Michael S. Tsirkin's message of "Wed, 28 Nov 2018 10:16:44 -0500")

"Michael S. Tsirkin" <mst@redhat.com> wrote:
> On Wed, Nov 28, 2018 at 08:46:15AM +0100, Juan Quintela wrote:
>> "Michael S. Tsirkin" <mst@redhat.com> wrote:
>> > On Mon, Nov 26, 2018 at 08:59:53PM +0100, Juan Quintela wrote:
>> >> Hi
>> >> 
>> >> V2:
>> >> 
>> >> - Rebase on top of master
>> >> 
>> >> Please review.
>> >> 
>> >> Later, Juan.
>> >
>> > And just making sure the point is so that one
>> > can disable virtio-pci and still build virtio?
>> >
>> > And the point of that is mostly as a cleanup on
>> > generic principles, right? Or is there a project that
>> > actually wants to do it?
>> 
>> This allows "make check" to work with them disabled.  Without this
>> changes, introspection fails, because we have the "name" (kind of)
>> "virtio-pci-blk", but not the real virtio device, so it tries to use it
>> while it is not there.  See the equivalent change for virtio-ccw-*.
>> 
>> Thanks, Juan.
>> 
>> PD.  Without this change, you can build a qemu with virtio devices
>>      disabled (i.e. not compiled in), but it fails "make check".  With
>>      the changes, it also works for "make check".
>
> And just to complete the picture the point of that is I guess enabling
> testing of random configs?

Not yet.
But for developing, I try _not_ to compile the bits that I am not
using.  Here I just went for being general.  After this patches, you can
disable  all CONFIG_VIRTIO_* stuff, and "make check" still works.
(Ok CONFIG_VIRTIO_GPU/VGA can't be disable, it needs changes inside the
tests themselves).

Later, Juan.

      reply	other threads:[~2018-11-29 13:46 UTC|newest]

Thread overview: 46+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-11-26 19:59 [Qemu-devel] [PATCH v2 00/16] Virtio devices split from virtio-pci Juan Quintela
2018-11-26 19:59 ` [Qemu-devel] [PATCH v2 01/16] virtio: split vhost vsock bits " Juan Quintela
2018-11-28 17:33   ` Laurent Vivier
2018-11-29  6:30   ` Thomas Huth
2018-11-29  6:32     ` Thomas Huth
2018-11-30  8:56       ` Juan Quintela
2018-11-26 19:59 ` [Qemu-devel] [PATCH v2 02/16] virtio: split virtio input host " Juan Quintela
2018-11-28 18:00   ` Laurent Vivier
2018-12-12 17:43     ` Juan Quintela
2018-11-28 18:21   ` Laurent Vivier
2018-11-26 19:59 ` [Qemu-devel] [PATCH v2 03/16] virtio: split virtio input " Juan Quintela
2018-11-28 18:20   ` Laurent Vivier
2018-11-26 19:59 ` [Qemu-devel] [PATCH v2 04/16] virtio: split virtio rng " Juan Quintela
2018-11-28 20:06   ` Laurent Vivier
2018-12-12 17:43     ` Juan Quintela
2018-11-26 19:59 ` [Qemu-devel] [PATCH v2 05/16] virtio: split virtio balloon " Juan Quintela
2018-11-28 20:09   ` Laurent Vivier
2018-11-26 19:59 ` [Qemu-devel] [PATCH v2 06/16] virtio: split virtio 9p " Juan Quintela
2018-11-28 20:15   ` Laurent Vivier
2018-12-12 17:44     ` Juan Quintela
2018-11-26 20:00 ` [Qemu-devel] [PATCH v2 07/16] virtio: split vhost user blk " Juan Quintela
2018-11-28 20:21   ` Laurent Vivier
2018-11-26 20:00 ` [Qemu-devel] [PATCH v2 08/16] virtio: split vhost user scsi " Juan Quintela
2018-11-28 20:23   ` Laurent Vivier
2018-11-26 20:00 ` [Qemu-devel] [PATCH v2 09/16] virtio: split vhost " Juan Quintela
2018-11-28 20:25   ` Laurent Vivier
2018-11-26 20:00 ` [Qemu-devel] [PATCH v2 10/16] virtio: split virtio " Juan Quintela
2018-11-28 20:28   ` Laurent Vivier
2018-11-26 20:00 ` [Qemu-devel] [PATCH v2 11/16] virtio: split virtio blk bits rom virtio-pci Juan Quintela
2018-11-28 20:31   ` Laurent Vivier
2018-11-26 20:00 ` [Qemu-devel] [PATCH v2 12/16] virtio: split virtio net " Juan Quintela
2018-11-28 20:33   ` Laurent Vivier
2018-11-26 20:00 ` [Qemu-devel] [PATCH v2 13/16] virtio: split virtio serial " Juan Quintela
2018-11-28 20:36   ` Laurent Vivier
2018-11-26 20:00 ` [Qemu-devel] [PATCH v2 14/16] virtio: split virtio gpu bits rom virtio-pci.h Juan Quintela
2018-11-28 20:39   ` Laurent Vivier
2018-11-26 20:00 ` [Qemu-devel] [PATCH v2 15/16] virtio: split virtio crypto " Juan Quintela
2018-11-28 20:41   ` Laurent Vivier
2018-11-26 20:00 ` [Qemu-devel] [PATCH v2 16/16] virtio: virtio 9p really requires CONFIG_VIRTFS to work Juan Quintela
2018-11-28 20:48   ` Laurent Vivier
2018-12-13 11:44     ` Juan Quintela
2018-11-27 11:38 ` [Qemu-devel] [PATCH v2 00/16] Virtio devices split from virtio-pci Dr. David Alan Gilbert
2018-11-28  3:13 ` Michael S. Tsirkin
2018-11-28  7:46   ` Juan Quintela
2018-11-28 15:16     ` Michael S. Tsirkin
2018-11-29 13:46       ` Juan Quintela [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87h8g00zfq.fsf@trasno.org \
    --to=quintela@redhat.com \
    --cc=dgilbert@redhat.com \
    --cc=lvivier@redhat.com \
    --cc=mst@redhat.com \
    --cc=peterx@redhat.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.