From: Kevin Wolf <kwolf@redhat.com>
To: Peter Lieven <pl@dlh.net>
Cc: Bernhard Kohl <bernhard.kohl@nsn.com>,
qemu-devel@nongnu.org,
"Nicholas A. Bellinger" <nab@linux-iscsi.org>,
Hannes Reinecke <hare@suse.de>
Subject: Re: [Qemu-devel] Re: [PATCH] lsi53c895a: add support for ABORT messages
Date: Wed, 09 Mar 2011 09:47:27 +0100 [thread overview]
Message-ID: <4D773E9F.6060009@redhat.com> (raw)
In-Reply-To: <34941DDA-4ADD-478F-A529-2B48F4A64541@dlh.net>
Am 09.03.2011 00:04, schrieb Peter Lieven:
>
> Am 07.10.2010 um 13:27 schrieb Kevin Wolf:
>
>> Am 06.09.2010 16:42, schrieb Bernhard Kohl:
>>> If these messages are not handled correctly the guest driver may hang.
>>>
>>> Always mandatory:
>>> - ABORT
>>> - BUS DEVICE RESET
>>>
>>> Mandatory if tagged queuing is implemented (which disks usually do):
>>> - ABORT TAG
>>> - CLEAR QUEUE
>>>
>>> Signed-off-by: Bernhard Kohl <bernhard.kohl@nsn.com>
>>
>> Nicholas, as you seem to have touched the lsi code recently, care to
>> review this one? Assuming that you are reasonably familiar with both the
>> hardware and the code, you should be quicker than me with this.
>
> Is there a reason why this patch was never added to the stable qemu-kvm release?
> At least int qemu-kvm-0.14.0 I still can't find it.
The reason is that it still didn't get a review.
Kevin
next prev parent reply other threads:[~2011-03-09 8:45 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-09-06 14:42 [Qemu-devel] [PATCH] lsi53c895a: add support for ABORT messages Bernhard Kohl
2010-10-07 11:27 ` [Qemu-devel] " Kevin Wolf
2011-03-08 23:04 ` Peter Lieven
2011-03-09 5:33 ` Stefan Hajnoczi
2011-03-09 8:47 ` Kevin Wolf [this message]
2011-03-09 9:25 ` Bernhard Kohl
2011-03-09 9:38 ` Peter Lieven
2011-04-26 14:46 ` [Qemu-devel] " Peter Lieven
2011-04-26 17:04 ` Michael Tokarev
2011-04-27 13:06 ` Peter Lieven
2011-04-01 20:16 ` [Qemu-devel] " Aurelien Jarno
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=4D773E9F.6060009@redhat.com \
--to=kwolf@redhat.com \
--cc=bernhard.kohl@nsn.com \
--cc=hare@suse.de \
--cc=nab@linux-iscsi.org \
--cc=pl@dlh.net \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).