From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:54410) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fSvih-0004z7-OJ for qemu-devel@nongnu.org; Tue, 12 Jun 2018 22:41:12 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1fSvie-0000iQ-LV for qemu-devel@nongnu.org; Tue, 12 Jun 2018 22:41:11 -0400 Received: from mx3-rdu2.redhat.com ([66.187.233.73]:57296 helo=mx1.redhat.com) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1fSvie-0000i5-Gy for qemu-devel@nongnu.org; Tue, 12 Jun 2018 22:41:08 -0400 References: <1525734594-11134-1-git-send-email-sridhar.samudrala@intel.com> <20180605152344-mutt-send-email-mst@kernel.org> <20180612144743-mutt-send-email-mst@kernel.org> <7aa6dbe7-bac2-96f6-d96c-f51e5173c556@intel.com> From: Jason Wang Message-ID: <6c7666e6-dc00-d9ed-8814-68e8b3998eef@redhat.com> Date: Wed, 13 Jun 2018 10:41:01 +0800 MIME-Version: 1.0 In-Reply-To: <7aa6dbe7-bac2-96f6-d96c-f51e5173c556@intel.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH] qemu: Introduce VIRTIO_NET_F_STANDBY feature bit to virtio_net List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Samudrala, Sridhar" , "Michael S. Tsirkin" Cc: alexander.h.duyck@intel.com, virtio-dev@lists.oasis-open.org, qemu-devel@nongnu.org, jesse.brandeburg@intel.com, virtualization@lists.linux-foundation.org On 2018=E5=B9=B406=E6=9C=8813=E6=97=A5 08:20, Samudrala, Sridhar wrote: > On 6/12/2018 4:54 AM, Michael S. Tsirkin wrote: >> On Wed, Jun 06, 2018 at 10:29:03AM +0800, Jason Wang wrote: >>> >>> On 2018=E5=B9=B406=E6=9C=8805=E6=97=A5 20:33, Michael S. Tsirkin wrot= e: >>>> I don't think this is sufficient. >>>> >>>> If both primary and standby devices are present, a legacy guest=20 >>>> without >>>> support for the feature might see two devices with same mac and get >>>> confused. >>>> >>>> I think that we should only make primary visible after guest acked t= he >>>> backup feature bit. >>> I think we want exactly the reverse? E.g fail the negotiation when=20 >>> guest >>> does not ack backup feature. >>> >>> Otherwise legacy guest won't even have the chance to see primary=20 >>> device in >>> the guest. >> That's by design. >> >>>> And on reset or when backup is cleared in some other way, unplug the >>>> primary. >>> What if guest does not support hotplug? >> It shouldn't ack the backup feature then and it's a good point. >> We should both document this and check kernel config has >> hotplug support. Sridhar could you take a look pls? > > Right now we select NET_FAILOVER when virtio-net is enabled, Should we=20 > select > PCI_HOTPLUG when NET_FAILOVER is enabled?=20 Maybe I was wrong but it looks to me PCI_HOTPLUG is no sufficient since=20 it depends on other to work e.g HOTPLUG_PCI_ACPI. Thanks From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jason Wang Subject: Re: [Qemu-devel] [PATCH] qemu: Introduce VIRTIO_NET_F_STANDBY feature bit to virtio_net Date: Wed, 13 Jun 2018 10:41:01 +0800 Message-ID: <6c7666e6-dc00-d9ed-8814-68e8b3998eef@redhat.com> References: <1525734594-11134-1-git-send-email-sridhar.samudrala@intel.com> <20180605152344-mutt-send-email-mst@kernel.org> <20180612144743-mutt-send-email-mst@kernel.org> <7aa6dbe7-bac2-96f6-d96c-f51e5173c556@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <7aa6dbe7-bac2-96f6-d96c-f51e5173c556@intel.com> Content-Language: en-US List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: virtualization-bounces@lists.linux-foundation.org Errors-To: virtualization-bounces@lists.linux-foundation.org To: "Samudrala, Sridhar" , "Michael S. Tsirkin" Cc: alexander.h.duyck@intel.com, virtio-dev@lists.oasis-open.org, virtualization@lists.linux-foundation.org, qemu-devel@nongnu.org List-Id: virtualization@lists.linuxfoundation.org CgpPbiAyMDE45bm0MDbmnIgxM+aXpSAwODoyMCwgU2FtdWRyYWxhLCBTcmlkaGFyIHdyb3RlOgo+ IE9uIDYvMTIvMjAxOCA0OjU0IEFNLCBNaWNoYWVsIFMuIFRzaXJraW4gd3JvdGU6Cj4+IE9uIFdl ZCwgSnVuIDA2LCAyMDE4IGF0IDEwOjI5OjAzQU0gKzA4MDAsIEphc29uIFdhbmcgd3JvdGU6Cj4+ Pgo+Pj4gT24gMjAxOOW5tDA25pyIMDXml6UgMjA6MzMsIE1pY2hhZWwgUy4gVHNpcmtpbiB3cm90 ZToKPj4+PiBJIGRvbid0IHRoaW5rIHRoaXMgaXMgc3VmZmljaWVudC4KPj4+Pgo+Pj4+IElmIGJv dGggcHJpbWFyeSBhbmQgc3RhbmRieSBkZXZpY2VzIGFyZSBwcmVzZW50LCBhIGxlZ2FjeSBndWVz dCAKPj4+PiB3aXRob3V0Cj4+Pj4gc3VwcG9ydCBmb3IgdGhlIGZlYXR1cmUgbWlnaHQgc2VlIHR3 byBkZXZpY2VzIHdpdGggc2FtZSBtYWMgYW5kIGdldAo+Pj4+IGNvbmZ1c2VkLgo+Pj4+Cj4+Pj4g SSB0aGluayB0aGF0IHdlIHNob3VsZCBvbmx5IG1ha2UgcHJpbWFyeSB2aXNpYmxlIGFmdGVyIGd1 ZXN0IGFja2VkIHRoZQo+Pj4+IGJhY2t1cCBmZWF0dXJlIGJpdC4KPj4+IEkgdGhpbmsgd2Ugd2Fu dCBleGFjdGx5IHRoZSByZXZlcnNlPyBFLmcgZmFpbCB0aGUgbmVnb3RpYXRpb24gd2hlbiAKPj4+ IGd1ZXN0Cj4+PiBkb2VzIG5vdCBhY2sgYmFja3VwIGZlYXR1cmUuCj4+Pgo+Pj4gT3RoZXJ3aXNl IGxlZ2FjeSBndWVzdCB3b24ndCBldmVuIGhhdmUgdGhlIGNoYW5jZSB0byBzZWUgcHJpbWFyeSAK Pj4+IGRldmljZSBpbgo+Pj4gdGhlIGd1ZXN0Lgo+PiBUaGF0J3MgYnkgZGVzaWduLgo+Pgo+Pj4+ IEFuZCBvbiByZXNldCBvciB3aGVuIGJhY2t1cCBpcyBjbGVhcmVkIGluIHNvbWUgb3RoZXIgd2F5 LCB1bnBsdWcgdGhlCj4+Pj4gcHJpbWFyeS4KPj4+IFdoYXQgaWYgZ3Vlc3QgZG9lcyBub3Qgc3Vw cG9ydCBob3RwbHVnPwo+PiBJdCBzaG91bGRuJ3QgYWNrIHRoZSBiYWNrdXAgZmVhdHVyZSB0aGVu IGFuZCBpdCdzIGEgZ29vZCBwb2ludC4KPj4gV2Ugc2hvdWxkIGJvdGggZG9jdW1lbnQgdGhpcyBh bmQgY2hlY2sga2VybmVsIGNvbmZpZyBoYXMKPj4gaG90cGx1ZyBzdXBwb3J0LiBTcmlkaGFyIGNv dWxkIHlvdSB0YWtlIGEgbG9vayBwbHM/Cj4KPiBSaWdodCBub3cgd2Ugc2VsZWN0IE5FVF9GQUlM T1ZFUiB3aGVuIHZpcnRpby1uZXQgaXMgZW5hYmxlZCwgU2hvdWxkIHdlIAo+IHNlbGVjdAo+IFBD SV9IT1RQTFVHIHdoZW4gTkVUX0ZBSUxPVkVSIGlzIGVuYWJsZWQ/IAoKTWF5YmUgSSB3YXMgd3Jv bmcgYnV0IGl0IGxvb2tzIHRvIG1lIFBDSV9IT1RQTFVHIGlzIG5vIHN1ZmZpY2llbnQgc2luY2Ug Cml0IGRlcGVuZHMgb24gb3RoZXIgdG8gd29yayBlLmcgSE9UUExVR19QQ0lfQUNQSS4KClRoYW5r cwpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpWaXJ0dWFs aXphdGlvbiBtYWlsaW5nIGxpc3QKVmlydHVhbGl6YXRpb25AbGlzdHMubGludXgtZm91bmRhdGlv bi5vcmcKaHR0cHM6Ly9saXN0cy5saW51eGZvdW5kYXRpb24ub3JnL21haWxtYW4vbGlzdGluZm8v dmlydHVhbGl6YXRpb24= From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: virtio-dev-return-4358-cohuck=redhat.com@lists.oasis-open.org Sender: List-Post: List-Help: List-Unsubscribe: List-Subscribe: Received: from lists.oasis-open.org (oasis-open.org [66.179.20.138]) by lists.oasis-open.org (Postfix) with ESMTP id 0D9D758191E1 for ; Tue, 12 Jun 2018 19:41:18 -0700 (PDT) References: <1525734594-11134-1-git-send-email-sridhar.samudrala@intel.com> <20180605152344-mutt-send-email-mst@kernel.org> <20180612144743-mutt-send-email-mst@kernel.org> <7aa6dbe7-bac2-96f6-d96c-f51e5173c556@intel.com> From: Jason Wang Message-ID: <6c7666e6-dc00-d9ed-8814-68e8b3998eef@redhat.com> Date: Wed, 13 Jun 2018 10:41:01 +0800 MIME-Version: 1.0 In-Reply-To: <7aa6dbe7-bac2-96f6-d96c-f51e5173c556@intel.com> Content-Type: text/plain; charset="utf-8"; format="flowed" Content-Transfer-Encoding: 8bit Content-Language: en-US Subject: [virtio-dev] Re: [Qemu-devel] [PATCH] qemu: Introduce VIRTIO_NET_F_STANDBY feature bit to virtio_net To: "Samudrala, Sridhar" , "Michael S. Tsirkin" Cc: alexander.h.duyck@intel.com, virtio-dev@lists.oasis-open.org, qemu-devel@nongnu.org, jesse.brandeburg@intel.com, virtualization@lists.linux-foundation.org List-ID: On 2018年06月13日 08:20, Samudrala, Sridhar wrote: > On 6/12/2018 4:54 AM, Michael S. Tsirkin wrote: >> On Wed, Jun 06, 2018 at 10:29:03AM +0800, Jason Wang wrote: >>> >>> On 2018年06月05日 20:33, Michael S. Tsirkin wrote: >>>> I don't think this is sufficient. >>>> >>>> If both primary and standby devices are present, a legacy guest >>>> without >>>> support for the feature might see two devices with same mac and get >>>> confused. >>>> >>>> I think that we should only make primary visible after guest acked the >>>> backup feature bit. >>> I think we want exactly the reverse? E.g fail the negotiation when >>> guest >>> does not ack backup feature. >>> >>> Otherwise legacy guest won't even have the chance to see primary >>> device in >>> the guest. >> That's by design. >> >>>> And on reset or when backup is cleared in some other way, unplug the >>>> primary. >>> What if guest does not support hotplug? >> It shouldn't ack the backup feature then and it's a good point. >> We should both document this and check kernel config has >> hotplug support. Sridhar could you take a look pls? > > Right now we select NET_FAILOVER when virtio-net is enabled, Should we > select > PCI_HOTPLUG when NET_FAILOVER is enabled? Maybe I was wrong but it looks to me PCI_HOTPLUG is no sufficient since it depends on other to work e.g HOTPLUG_PCI_ACPI. Thanks --------------------------------------------------------------------- To unsubscribe, e-mail: virtio-dev-unsubscribe@lists.oasis-open.org For additional commands, e-mail: virtio-dev-help@lists.oasis-open.org