From: Greg KH <gregkh@linuxfoundation.org>
To: KY Srinivasan <kys@microsoft.com>
Cc: "olaf@aepfle.de" <olaf@aepfle.de>,
"jasowang@redhat.com" <jasowang@redhat.com>,
Haiyang Zhang <haiyangz@microsoft.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"apw@canonical.com" <apw@canonical.com>,
"devel@linuxdriverproject.org" <devel@linuxdriverproject.org>,
"leann.ogasawara@canonical.com" <leann.ogasawara@canonical.com>
Subject: Re: [PATCH 02/15] hyperv: Add a function to detect hv_device
Date: Fri, 2 Dec 2016 17:02:59 +0100 [thread overview]
Message-ID: <20161202160259.GA6050@kroah.com> (raw)
In-Reply-To: <DM5PR03MB2490E658B9D8B479AD7AB402A08E0@DM5PR03MB2490.namprd03.prod.outlook.com>
On Fri, Dec 02, 2016 at 03:38:51PM +0000, KY Srinivasan wrote:
>
>
> > -----Original Message-----
> > From: Greg KH [mailto:gregkh@linuxfoundation.org]
> > Sent: Thursday, December 1, 2016 11:36 PM
> > To: KY Srinivasan <kys@microsoft.com>
> > Cc: linux-kernel@vger.kernel.org; devel@linuxdriverproject.org;
> > olaf@aepfle.de; apw@canonical.com; vkuznets@redhat.com;
> > jasowang@redhat.com; leann.ogasawara@canonical.com; Haiyang Zhang
> > <haiyangz@microsoft.com>
> > Subject: Re: [PATCH 02/15] hyperv: Add a function to detect hv_device
> >
> > On Fri, Dec 02, 2016 at 07:14:03AM +0000, KY Srinivasan wrote:
> > > > In other words, why do you need this and PCI or USB doesn't? Why is
> > > > hyperv "special"?
> > >
> > > On Hyper-V, each VF interface (SR-IOV interface)
> > > is paired with an instance of the
> > > synthetic interface that is managed by netvsc.
> > > When the VF interface comes up, we
> > > need to associate the VF instance with
> > > the corresponding netvsc instance. To do this
> > > without modifying the VF drivers, netvsc registers
> > > for netdev events.
> >
> > Why not modify the VF drivers? You have the full source to them...
> Greg,
>
> This is even worse. On Linux, VF drivers are hypervisor agnostic
> and I want to keep it that way.
Ok, I really don't know what to suggest, other than this is probably not
the way to do this as no other bus has to. As I don't see the code that
actually uses this anywhere, it's really impossible to have this
conversation at all :(
greg k-h
next prev parent reply other threads:[~2016-12-02 16:02 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-12-01 17:28 [PATCH 00/15] Drivers: hv: CPU management fixes and a new uio driver kys
2016-12-01 17:28 ` [PATCH 01/15] Drivers: hv: vmbus: Raise retry/wait limits in vmbus_post_msg() kys
2016-12-01 17:28 ` [PATCH 02/15] hyperv: Add a function to detect hv_device kys
2016-12-01 20:33 ` Greg KH
2016-12-02 5:46 ` KY Srinivasan
2016-12-01 20:35 ` Greg KH
2016-12-01 20:38 ` Greg KH
2016-12-02 6:02 ` KY Srinivasan
2016-12-02 6:48 ` Greg KH
2016-12-02 7:14 ` KY Srinivasan
2016-12-02 7:36 ` Greg KH
2016-12-02 15:38 ` KY Srinivasan
2016-12-02 16:02 ` Greg KH [this message]
2016-12-02 21:41 ` KY Srinivasan
2016-12-01 17:28 ` [PATCH 03/15] hyperv: Fix spelling of HV_UNKOWN kys
2016-12-01 17:28 ` [PATCH 04/15] Drivers: hv: vmbus: Prevent sending data on a rescinded channel kys
2016-12-01 20:36 ` Greg KH
2016-12-05 10:01 ` Dan Carpenter
2016-12-05 15:50 ` KY Srinivasan
2016-12-01 17:28 ` [PATCH 05/15] Drivers: hv: vmbus: Enhance the rescind callback functionality kys
2016-12-01 20:36 ` Greg KH
2016-12-01 17:28 ` [PATCH 06/15] hv: acquire vmbus_connection.channel_mutex in vmbus_free_channels() kys
2016-12-01 20:37 ` Greg KH
2016-12-02 6:11 ` KY Srinivasan
2016-12-01 17:28 ` [PATCH 07/15] hv: allocate synic pages for all present CPUs kys
2016-12-01 17:28 ` [PATCH 08/15] hv: init percpu_list in hv_synic_alloc() kys
2016-12-01 17:28 ` [PATCH 09/15] hv: change clockevents unbind tactics kys
2016-12-01 17:28 ` [PATCH 10/15] hv: switch to cpuhp state machine for synic init/cleanup kys
2016-12-01 17:28 ` [PATCH 11/15] hv: make CPU offlining prevention fine-grained kys
2016-12-01 17:28 ` [PATCH 12/15] hv: don't reset hv_context.tsc_page on crash kys
2016-12-01 17:28 ` [PATCH 13/15] vmbus: add support for dynamic device id's kys
2016-12-01 17:28 ` [PATCH 14/15] uio-hv-generic: new userspace i/o driver for VMBus kys
2016-12-01 17:28 ` [PATCH 15/15] Tools: hv: kvp: configurable external scripts path kys
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=20161202160259.GA6050@kroah.com \
--to=gregkh@linuxfoundation.org \
--cc=apw@canonical.com \
--cc=devel@linuxdriverproject.org \
--cc=haiyangz@microsoft.com \
--cc=jasowang@redhat.com \
--cc=kys@microsoft.com \
--cc=leann.ogasawara@canonical.com \
--cc=linux-kernel@vger.kernel.org \
--cc=olaf@aepfle.de \
/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).