From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([208.118.235.92]:34599)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <borntraeger@de.ibm.com>) id 1SyzmD-0003Hy-Cv
	for qemu-devel@nongnu.org; Wed, 08 Aug 2012 02:29:54 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <borntraeger@de.ibm.com>) id 1SyzmC-0002kO-91
	for qemu-devel@nongnu.org; Wed, 08 Aug 2012 02:29:53 -0400
Received: from e06smtp18.uk.ibm.com ([195.75.94.114]:33007)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <borntraeger@de.ibm.com>) id 1SyzmC-0002kG-0S
	for qemu-devel@nongnu.org; Wed, 08 Aug 2012 02:29:52 -0400
Received: from /spool/local
	by e06smtp18.uk.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use
	Only! Violators will be prosecuted
	for <qemu-devel@nongnu.org> from <borntraeger@de.ibm.com>;
	Wed, 8 Aug 2012 07:29:50 +0100
Message-ID: <50220759.1090504@de.ibm.com>
Date: Wed, 08 Aug 2012 08:29:45 +0200
From: Christian Borntraeger <borntraeger@de.ibm.com>
MIME-Version: 1.0
References: <1344307320-25094-2-git-send-email-zhlcindy@linux.vnet.ibm.com>
	<1344341985-10065-2-git-send-email-borntraeger@de.ibm.com>
	<CAFEAcA-mLAvgUwSfyT52oKrKa-WG454UfVgoMVxwQ3hfogJEgA@mail.gmail.com>
	<CAAu8pHuSMr25x=pF99+Y7DvZPOD+5qa+4NctGGCzkXsXduuySA@mail.gmail.com>
In-Reply-To: <CAAu8pHuSMr25x=pF99+Y7DvZPOD+5qa+4NctGGCzkXsXduuySA@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Subject: Re: [Qemu-devel] [RFC/PATCH 1/1] USB code fenced for s390
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Blue Swirl <blauwirbel@gmail.com>
Cc: Peter Maydell <peter.maydell@linaro.org>, aliguori@us.ibm.com, qemu-ppc@nongnu.org, agraf@suse.de, Li Zhang <zhlcindy@linux.vnet.ibm.com>, qemu-devel@nongnu.org, Jens Freimann <jfrei@linux.vnet.ibm.com>, Li Zhang <zhlcindy@gmail.com>, Eugene Dvurechenski <jno@linux.vnet.ibm.com>, afaerber@suse.de, david@gibson.dropbear.id.au

On 07/08/12 22:22, Blue Swirl wrote:
> On Tue, Aug 7, 2012 at 12:26 PM, Peter Maydell <peter.maydell@linaro.org> wrote:
>> On 7 August 2012 13:19, Christian Borntraeger <borntraeger@de.ibm.com> wrote:
>>> +#if defined(TARGET_HAS_USB) && (TARGET_HAS_USB == 1)
>>>      /* init USB devices */
>>>      if (usb_enabled) {
>>>          if (foreach_device_config(DEV_USB, usb_parse) < 0)
>>>              exit(1);
>>>      }
>>> +#endif
>>
>> Whether there is USB or not is a property of the machine model,
>> not the target CPU architecture, so a TARGET_HAS_USB define
>> is definitely the wrong approach.
> 
> Yes. I'd add CONFIG_USB=y to files in default-configs/ except for
> s390*, Sparc32 and user emulators. Unless it's enough for USB to have
> a dependency on PCI?

Yes, that seems to be the right place - thanks.  A dependency on pci would also
do for us. 

On the other hand, if there are really plans to add a virtio-usb then we should
not disable usb support at all - especially since disabling it does cause some
trouble with libvirt and the current state does not cause problems (we assumed that
libvirt will detect usb capability like it does for other parameters, but it doesnt).
In other words this patch was mostly cosmetics. So lets just drop this patch, go ahead
with Li's patch set and later on we can see if we should do anything regarding s390.

Christian