From: Cao jin <caoj.fnst@cn.fujitsu.com>
To: qemu-devel@nongnu.org
Cc: "Michael S. Tsirkin" <mst@redhat.com>,
Jason Wang <jasowang@redhat.com>,
Markus Armbruster <armbru@redhat.com>,
Marcel Apfelbaum <marcel@redhat.com>,
Alex Williamson <alex.williamson@redhat.com>,
Hannes Reinecke <hare@suse.de>,
Dmitry Fleytman <dmitry@daynix.com>,
Paolo Bonzini <pbonzini@redhat.com>, John Snow <jsnow@redhat.com>,
Gerd Hoffmann <kraxel@redhat.com>
Subject: Re: [Qemu-devel] [PATCH v7 12/15] pci: Convert msi_init() to Error and fix callers to check it
Date: Mon, 6 Jun 2016 16:30:22 +0800 [thread overview]
Message-ID: <5755349E.6060904@cn.fujitsu.com> (raw)
In-Reply-To: <1465200062-13824-13-git-send-email-caoj.fnst@cn.fujitsu.com>
On 06/06/2016 04:00 PM, Cao jin wrote:
> diff --git a/hw/usb/hcd-xhci.c b/hw/usb/hcd-xhci.c
>
> + if (xhci->msi != ON_OFF_AUTO_OFF) {
> + ret = msi_init(dev, 0x70, xhci->numintrs, true, false, &err);
> + /* Any error other than -ENOTSUP(board's MSI support is broken)
> + * is a programming error */
> + assert(!ret || ret == -ENOTSUP);
> + if (ret && xhci->msi == ON_OFF_AUTO_ON) {
> + /* Can't satisfy user's explicit msi=on request, fail */
> + error_append_hint(&err, "You have to use msi=auto (default) or "
> + "si=off with this machine type.\n");
sorry, I made a typo "si=off", maybe it could be fixed when be applied
by maintainer
--
Yours Sincerely,
Cao jin
next prev parent reply other threads:[~2016-06-06 8:25 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-06-06 8:00 [Qemu-devel] [PATCH v7 00/15] Add param Error ** for msi_init() Cao jin
2016-06-06 8:00 ` [Qemu-devel] [PATCH v7 01/15] pci core: assert ENOSPC when add capability Cao jin
2016-06-06 8:00 ` [Qemu-devel] [PATCH v7 02/15] fix some coding style problems Cao jin
2016-06-06 8:00 ` [Qemu-devel] [PATCH v7 03/15] change pvscsi_init_msi() type to void Cao jin
2016-06-06 8:00 ` [Qemu-devel] [PATCH v7 04/15] megasas: Fix check for msi_init() failure Cao jin
2016-06-06 8:00 ` [Qemu-devel] [PATCH v7 05/15] mptsas: change .realize function name Cao jin
2016-06-06 8:00 ` [Qemu-devel] [PATCH v7 06/15] usb xhci: change msi/msix property type Cao jin
2016-06-06 8:00 ` [Qemu-devel] [PATCH v7 07/15] intel-hda: change msi " Cao jin
2016-06-06 8:00 ` [Qemu-devel] [PATCH v7 08/15] mptsas: " Cao jin
2016-06-06 8:00 ` [Qemu-devel] [PATCH v7 09/15] megasas: change msi/msix " Cao jin
2016-06-06 8:27 ` Hannes Reinecke
2016-06-06 8:43 ` Cao jin
2016-06-06 8:00 ` [Qemu-devel] [PATCH v7 10/15] pci bridge dev: change msi " Cao jin
2016-06-06 8:00 ` [Qemu-devel] [PATCH v7 11/15] msi_init: change return value to 0 on success Cao jin
2016-06-06 8:28 ` Hannes Reinecke
2016-06-06 9:19 ` Cao jin
2016-06-09 15:32 ` Markus Armbruster
2016-06-06 8:00 ` [Qemu-devel] [PATCH v7 12/15] pci: Convert msi_init() to Error and fix callers to check it Cao jin
2016-06-06 8:29 ` Hannes Reinecke
2016-06-06 8:30 ` Cao jin [this message]
2016-06-09 15:53 ` Markus Armbruster
2016-06-09 15:54 ` Markus Armbruster
2016-06-06 8:01 ` [Qemu-devel] [PATCH v7 13/15] megasas: remove unnecessary megasas_use_msi() Cao jin
2016-06-06 8:31 ` Hannes Reinecke
2016-06-06 8:01 ` [Qemu-devel] [PATCH v7 14/15] mptsas: remove unnecessary internal msi state flag Cao jin
2016-06-06 8:01 ` [Qemu-devel] [PATCH v7 15/15] vmw_pvscsi: " Cao jin
2016-06-09 15:48 ` Markus Armbruster
2016-06-08 10:15 ` [Qemu-devel] [PATCH v7 00/15] Add param Error ** for msi_init() Cao jin
2016-06-09 15:58 ` Markus Armbruster
2016-06-10 7:34 ` Cao jin
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=5755349E.6060904@cn.fujitsu.com \
--to=caoj.fnst@cn.fujitsu.com \
--cc=alex.williamson@redhat.com \
--cc=armbru@redhat.com \
--cc=dmitry@daynix.com \
--cc=hare@suse.de \
--cc=jasowang@redhat.com \
--cc=jsnow@redhat.com \
--cc=kraxel@redhat.com \
--cc=marcel@redhat.com \
--cc=mst@redhat.com \
--cc=pbonzini@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.