netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Or Gerlitz <gerlitz.or@gmail.com>
To: Salil Mehta <salil.mehta@huawei.com>
Cc: David Miller <davem@davemloft.net>,
	yisen.zhuang@huawei.com, lipeng321@huawei.com,
	mehta.salil.lnk@gmail.com,
	Linux Netdev List <netdev@vger.kernel.org>,
	Andy Gospodarek <andy@greyhouse.net>,
	Simon Horman <simon.horman@netronome.com>,
	Jiri Pirko <jiri@mellanox.com>
Subject: Re: [PATCH net-next 0/8] Hisilicon Network Subsystem 3 VF Ethernet Driver
Date: Sun, 3 Dec 2017 15:22:42 +0200	[thread overview]
Message-ID: <CAJ3xEMgU2N8UStvXdQLL4jp-og-aiZbnzgJtvo+XxU2qu7S+aA@mail.gmail.com> (raw)
In-Reply-To: <20171203123307.19820-1-salil.mehta@huawei.com>

On Sun, Dec 3, 2017 at 2:32 PM, Salil Mehta <salil.mehta@huawei.com> wrote:
> This patch-set contains the support of the HNS3 (Hisilicon Network Subsystem 3)
> Virtual Function Ethernet driver for hip08 family of SoCs. The Physical Function
> driver is already part of the Linux mainline.
>
> This VF driver has its Hardware Compatibility Layer and has commom/unified ENET
> layer/client/ethtool code with the PF driver. The basic architecture of VF driver
> is derivative of the PF driver. Just like PF driver, this driver is also PCI
> Express based.


Hi,

I wasn't sure if/what PF side control plane user space APIs are
supported by this series,
can you elaborate on that?  Please note that since 4.8, the kernel
supports switchdev
mode for SRIOV drivers and it's highly desirable for new drivers to
support that. The old
mode (called legacy) is inferior in functionality and design. We
didn't do a clear statement
asking new drivers not to support the legacy mode, but still, it would
be good to support
the new mode from day one.

Or.

  parent reply	other threads:[~2017-12-03 13:22 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-03 12:32 [PATCH net-next 0/8] Hisilicon Network Subsystem 3 VF Ethernet Driver Salil Mehta
2017-12-03 12:33 ` [PATCH net-next 1/8] net: hns3: Add HNS3 VF IMP(Integrated Management Proc) cmd interface Salil Mehta
     [not found]   ` <20171203123307.19820-2-salil.mehta-hv44wF8Li93QT0dZR+AlfA@public.gmane.org>
2017-12-05 16:37     ` David Miller
2017-12-05 17:11       ` Salil Mehta
2017-12-03 12:33 ` [PATCH net-next 2/8] net: hns3: Add mailbox support to VF driver Salil Mehta
2017-12-03 12:33 ` [PATCH net-next 3/8] net: hns3: Add HNS3 VF HCL(Hardware Compatibility Layer) Support Salil Mehta
2017-12-03 12:33 ` [PATCH net-next 4/8] net: hns3: Add HNS3 VF driver to kernel build framework Salil Mehta
2017-12-03 12:33 ` [PATCH net-next 5/8] net: hns3: Unified HNS3 {VF|PF} Ethernet Driver for hip08 SoC Salil Mehta
     [not found] ` <20171203123307.19820-1-salil.mehta-hv44wF8Li93QT0dZR+AlfA@public.gmane.org>
2017-12-03 12:33   ` [PATCH net-next 6/8] net: hns3: Add mailbox support to PF driver Salil Mehta
2017-12-03 12:33 ` [PATCH net-next 7/8] net: hns3: Change PF to add ring-vect binding & resetQ to mailbox Salil Mehta
2017-12-03 12:33 ` [PATCH net-next 8/8] net: hns3: Add mailbox interrupt handling to PF driver Salil Mehta
2017-12-03 13:22 ` Or Gerlitz [this message]
     [not found]   ` <CAFOm3gZ8ayvRowh-UdR7WukWNpXvaVmq2xCxwK_wf=fErji37g@mail.gmail.com>
     [not found]     ` <F4CC6FACFEB3C54C9141D49AD221F7F93BB5F85E@FRAEML521-MBX.china.huawei.com>
2017-12-04 10:48       ` [PATCH net-next 0/8] Hisilicon Network Subsystem 3 VF Ethernet Driver Salil Mehta
2017-12-04 14:07         ` Or Gerlitz
2017-12-04 14:28           ` Salil Mehta

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=CAJ3xEMgU2N8UStvXdQLL4jp-og-aiZbnzgJtvo+XxU2qu7S+aA@mail.gmail.com \
    --to=gerlitz.or@gmail.com \
    --cc=andy@greyhouse.net \
    --cc=davem@davemloft.net \
    --cc=jiri@mellanox.com \
    --cc=lipeng321@huawei.com \
    --cc=mehta.salil.lnk@gmail.com \
    --cc=netdev@vger.kernel.org \
    --cc=salil.mehta@huawei.com \
    --cc=simon.horman@netronome.com \
    --cc=yisen.zhuang@huawei.com \
    /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).