All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rusty Russell <rusty@rustcorp.com.au>
To: Jamie Lokier <jamie@shareable.org>
Cc: Anthony Liguori <aliguori@us.ibm.com>,
	kvm@vger.kernel.org, "Michael S. Tsirkin" <mst@redhat.com>,
	qemu-devel <qemu-devel@nongnu.org>,
	virtualization@lists.linux-foundation.org
Subject: Re: [Qemu-devel] Using PCI config space to indicate config location
Date: Thu, 11 Oct 2012 10:38:43 +1030	[thread overview]
Message-ID: <87haq198fo.fsf@rustcorp.com.au> (raw)
In-Reply-To: <87txu36le8.fsf@rustcorp.com.au>

Rusty Russell <rusty@rustcorp.com.au> writes:
> Jamie Lokier <jamie@shareable.org> writes:
>
>> Rusty Russell wrote:
>>> I don't think it'll be that bad; reset clears the device to unknown,
>>> bar0 moves it from unknown->legacy mode, bar1/2/3 changes it from
>>> unknown->modern mode, and anything else is bad (I prefer being strict so
>>> we catch bad implementations from the beginning).
>>
>> Will that work, if the guest with kernel that uses modern mode, kexecs
>> to an older (but presumed reliable) kernel that only knows about legacy mode?
>>
>> I.e. will the replacement kernel, or (ideally) replacement driver on
>> the rare occasion that is needed on a running kernel, be able to reset
>> the device hard enough?
>
> Well, you need to reset the device, so yes.

MST said something which made me think harder about this case.

Either there needs to be a way to tell what mode the device is in, or
legacy reset has to work, even in modern mode.  The latter is
preferable, since it allows an older kernel to do the reset.

Now, since qemu would almost certainly have to add code to stop that
working, it'll probably be fine.  But I'd really like to add a "strict
mode" to qemu virtio which does extra sanity checking for driver
authors, and that might break this.  That's OK.

Thanks,
Rusty.

WARNING: multiple messages have this Message-ID (diff)
From: Rusty Russell <rusty@rustcorp.com.au>
To: Jamie Lokier <jamie@shareable.org>
Cc: Anthony Liguori <aliguori@us.ibm.com>,
	virtualization@lists.linux-foundation.org,
	qemu-devel <qemu-devel@nongnu.org>,
	kvm@vger.kernel.org, "Michael S. Tsirkin" <mst@redhat.com>
Subject: Re: [Qemu-devel] Using PCI config space to indicate config location
Date: Thu, 11 Oct 2012 10:38:43 +1030	[thread overview]
Message-ID: <87haq198fo.fsf@rustcorp.com.au> (raw)
In-Reply-To: <87txu36le8.fsf@rustcorp.com.au>

Rusty Russell <rusty@rustcorp.com.au> writes:
> Jamie Lokier <jamie@shareable.org> writes:
>
>> Rusty Russell wrote:
>>> I don't think it'll be that bad; reset clears the device to unknown,
>>> bar0 moves it from unknown->legacy mode, bar1/2/3 changes it from
>>> unknown->modern mode, and anything else is bad (I prefer being strict so
>>> we catch bad implementations from the beginning).
>>
>> Will that work, if the guest with kernel that uses modern mode, kexecs
>> to an older (but presumed reliable) kernel that only knows about legacy mode?
>>
>> I.e. will the replacement kernel, or (ideally) replacement driver on
>> the rare occasion that is needed on a running kernel, be able to reset
>> the device hard enough?
>
> Well, you need to reset the device, so yes.

MST said something which made me think harder about this case.

Either there needs to be a way to tell what mode the device is in, or
legacy reset has to work, even in modern mode.  The latter is
preferable, since it allows an older kernel to do the reset.

Now, since qemu would almost certainly have to add code to stop that
working, it'll probably be fine.  But I'd really like to add a "strict
mode" to qemu virtio which does extra sanity checking for driver
authors, and that might break this.  That's OK.

Thanks,
Rusty.

  reply	other threads:[~2012-10-11  1:10 UTC|newest]

Thread overview: 91+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-27  0:29 Proposal for virtio standardization Rusty Russell
2012-09-27  0:29 ` [Qemu-devel] " Rusty Russell
2012-09-27  0:29 ` Rusty Russell
2012-10-04 18:49 ` [Qemu-devel] " Anthony Liguori
2012-10-04 18:49 ` Anthony Liguori
2012-10-04 18:49   ` Anthony Liguori
2012-10-08  2:21   ` Using PCI config space to indicate config location Rusty Russell
2012-10-08  2:21   ` Rusty Russell
2012-10-08  2:21     ` [Qemu-devel] " Rusty Russell
2012-10-08 13:58     ` Anthony Liguori
2012-10-08 13:58       ` Anthony Liguori
2012-10-08 14:58       ` Gerd Hoffmann
2012-10-08 14:58         ` Gerd Hoffmann
2012-10-08 15:09         ` Anthony Liguori
2012-10-08 15:09           ` Anthony Liguori
2012-10-08 20:13           ` Gerd Hoffmann
2012-10-08 20:13             ` Gerd Hoffmann
2012-10-08 20:55             ` Anthony Liguori
2012-10-08 20:55               ` Anthony Liguori
2012-10-08 23:56               ` Rusty Russell
2012-10-09  1:51                 ` Anthony Liguori
2012-10-09  3:16                   ` Rusty Russell
2012-10-09  3:16                   ` Rusty Russell
2012-10-09  3:16                     ` Rusty Russell
2012-10-09 10:17                     ` Avi Kivity
2012-10-09 10:17                       ` Avi Kivity
2012-10-09 14:03                       ` Anthony Liguori
2012-10-09 14:03                         ` Anthony Liguori
2012-10-09 13:56                     ` Anthony Liguori
2012-10-09 13:56                     ` Anthony Liguori
2012-10-09 13:56                       ` Anthony Liguori
2012-10-10  3:44                       ` Rusty Russell
2012-10-10  3:44                         ` Rusty Russell
2012-10-10 11:37                         ` Michael S. Tsirkin
2012-10-10 11:37                           ` Michael S. Tsirkin
2012-10-09 21:09                     ` Jamie Lokier
2012-10-09 21:09                       ` [Qemu-devel] " Jamie Lokier
2012-10-10  3:44                       ` Rusty Russell
2012-10-10  3:44                         ` Rusty Russell
2012-10-11  0:08                         ` Rusty Russell [this message]
2012-10-11  0:08                           ` Rusty Russell
2012-10-11  0:08                         ` Rusty Russell
2012-10-09 21:09                     ` Jamie Lokier
2012-10-09  6:33                   ` Gerd Hoffmann
2012-10-09  6:33                     ` Gerd Hoffmann
2012-10-09 15:26                     ` Anthony Liguori
2012-10-09 15:26                     ` Anthony Liguori
2012-10-09 15:26                       ` Anthony Liguori
2012-10-09 20:24                       ` Gerd Hoffmann
2012-10-09 20:24                         ` Gerd Hoffmann
2012-10-10  2:54                         ` Rusty Russell
2012-10-10  2:54                         ` Rusty Russell
2012-10-10  2:54                           ` Rusty Russell
2012-10-10 13:36                           ` Anthony Liguori
2012-10-10 13:41                             ` Michael S. Tsirkin
2012-10-10 13:41                               ` Michael S. Tsirkin
2012-10-11  0:43                               ` Rusty Russell
2012-10-11  0:43                                 ` [Qemu-devel] " Rusty Russell
2012-10-11  0:43                               ` Rusty Russell
2012-10-10  8:34                 ` Michael S. Tsirkin
2012-10-10  8:34                   ` Michael S. Tsirkin
2012-10-08 13:58     ` Anthony Liguori
2012-10-10  8:30     ` Michael S. Tsirkin
2012-10-10  8:30       ` [Qemu-devel] " Michael S. Tsirkin
2012-10-11  1:18       ` Rusty Russell
2012-10-11  1:18         ` [Qemu-devel] " Rusty Russell
2012-10-11 10:23         ` Michael S. Tsirkin
2012-10-11 10:23           ` [Qemu-devel] " Michael S. Tsirkin
2012-10-11 22:29           ` Rusty Russell
2012-10-11 22:29           ` Rusty Russell
2012-10-11 22:29             ` [Qemu-devel] " Rusty Russell
2012-10-12  9:33             ` Michael S. Tsirkin
2012-10-12  9:33               ` [Qemu-devel] " Michael S. Tsirkin
2012-10-12  9:51               ` Rusty Russell
2012-10-12  9:51                 ` [Qemu-devel] " Rusty Russell
2012-10-12 10:02                 ` Michael S. Tsirkin
2012-10-12 10:02                   ` [Qemu-devel] " Michael S. Tsirkin
2012-10-16 13:15                   ` Rusty Russell
2012-10-16 13:15                     ` [Qemu-devel] " Rusty Russell
2012-10-16 13:30                     ` Michael S. Tsirkin
2012-10-16 13:30                       ` [Qemu-devel] " Michael S. Tsirkin
2012-10-16 13:52                       ` Rusty Russell
2012-10-16 13:52                       ` Rusty Russell
2012-10-16 13:52                         ` [Qemu-devel] " Rusty Russell
2012-10-16 13:15                   ` Rusty Russell
2012-10-09 14:02 ` Proposal for virtio standardization Cornelia Huck
2012-10-09 14:02   ` [Qemu-devel] " Cornelia Huck
2012-10-10  3:46   ` Rusty Russell
2012-10-10  3:46     ` [Qemu-devel] " Rusty Russell
2012-10-10  3:46   ` Rusty Russell
2012-10-09 14:02 ` Cornelia Huck

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=87haq198fo.fsf@rustcorp.com.au \
    --to=rusty@rustcorp.com.au \
    --cc=aliguori@us.ibm.com \
    --cc=jamie@shareable.org \
    --cc=kvm@vger.kernel.org \
    --cc=mst@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=virtualization@lists.linux-foundation.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.