Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753137AbcDOATc (ORCPT ); Thu, 14 Apr 2016 20:19:32 -0400 Received: from mail-io0-f171.google.com ([209.85.223.171]:35633 "EHLO mail-io0-f171.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752956AbcDOATa (ORCPT ); Thu, 14 Apr 2016 20:19:30 -0400 MIME-Version: 1.0 In-Reply-To: References: <1460678121-30308-1-git-send-email-kys@microsoft.com> <1460678142-30353-1-git-send-email-kys@microsoft.com> <1460678142-30353-2-git-send-email-kys@microsoft.com> <5F69AC74-16B9-4EAD-8B4B-21D2EB6B3653@intel.com> Date: Thu, 14 Apr 2016 17:19:24 -0700 Message-ID: Subject: Re: [PATCH net-next 2/2] intel: ixgbevf: Support Windows hosts (Hyper-V) From: Alexander Duyck To: KY Srinivasan Cc: "Rustad, Mark D" , David Miller , netdev , "linux-kernel@vger.kernel.org" , "devel@linuxdriverproject.org" , "olaf@aepfle.de" , "apw@canonical.com" , "jasowang@redhat.com" , "eli@mellanox.com" , "jackm@mellanox.com" , "yevgenyp@mellanox.com" , "Ronciak, John" , "intel-wired-lan@linuxonhyperv.com" Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2475 Lines: 66 On Thu, Apr 14, 2016 at 5:11 PM, KY Srinivasan wrote: > > >> -----Original Message----- >> From: Rustad, Mark D [mailto:mark.d.rustad@intel.com] >> Sent: Thursday, April 14, 2016 5:07 PM >> To: KY Srinivasan >> Cc: David Miller ; netdev >> ; linux-kernel@vger.kernel.org; >> devel@linuxdriverproject.org; olaf@aepfle.de; apw@canonical.com; >> jasowang@redhat.com; eli@mellanox.com; jackm@mellanox.com; >> yevgenyp@mellanox.com; Ronciak, John ; intel- >> wired-lan@linuxonhyperv.com >> Subject: Re: [PATCH net-next 2/2] intel: ixgbevf: Support Windows hosts >> (Hyper-V) >> >> KY Srinivasan wrote: >> >> > >> > >> >> -----Original Message----- >> >> From: Rustad, Mark D [mailto:mark.d.rustad@intel.com] >> >> Sent: Thursday, April 14, 2016 4:01 PM >> >> To: KY Srinivasan >> >> Cc: David Miller ; netdev >> >> ; linux-kernel@vger.kernel.org; >> >> devel@linuxdriverproject.org; olaf@aepfle.de; apw@canonical.com; >> >> jasowang@redhat.com; eli@mellanox.com; jackm@mellanox.com; >> >> yevgenyp@mellanox.com; Ronciak, John ; >> intel- >> >> wired-lan@linuxonhyperv.com >> >> Subject: Re: [PATCH net-next 2/2] intel: ixgbevf: Support Windows hosts >> >> (Hyper-V) >> >> >> >> Some comments below: >> > >> > Mark, >> > >> > Thank you for the comments. I will address them and repost the patches. >> > >> > Regards, >> > >> > K. Y >> >> Please look closely at Alex's comments. I think they are much more >> important. > > I am looking at Alex's comments as I am writing this. > On additional thought that just occurred to me after looking over the other patches you submitted for the hv_netvsc is that you might just stub out the multicast, unicast, and vfta configuration calls for the hyperV interface since all that stuff should be handled by the other link in the bond anyway. Then you should be able to mostly contain all the changes other than the device IDs to the vf.c file which is really how this kind of change should work anyway. Also I was wondering. Since HyperV is using a proprietary device ID anyway do you really need the calls like the one below?: + if (x86_hyper == &x86_hyper_ms_hyperv) { If we can just identify HyperV via the device Id then we could drop the x86 arch specific bits and instead just build for all cases. - Alex