All of lore.kernel.org
 help / color / mirror / Atom feed
From: Long Li <longli@microsoft.com>
To: Jason Gunthorpe <jgg@ziepe.ca>
Cc: KY Srinivasan <kys@microsoft.com>,
	Haiyang Zhang <haiyangz@microsoft.com>,
	Stephen Hemminger <sthemmin@microsoft.com>,
	Wei Liu <wei.liu@kernel.org>, Dexuan Cui <decui@microsoft.com>,
	"David S. Miller" <davem@davemloft.net>,
	Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
	Leon Romanovsky <leon@kernel.org>,
	"linux-hyperv@vger.kernel.org" <linux-hyperv@vger.kernel.org>,
	"netdev@vger.kernel.org" <netdev@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"linux-rdma@vger.kernel.org" <linux-rdma@vger.kernel.org>
Subject: RE: [PATCH 09/12] net: mana: Move header files to a common location
Date: Tue, 17 May 2022 19:34:21 +0000	[thread overview]
Message-ID: <PH7PR21MB3263114D9D57A3F385FCA6EBCECE9@PH7PR21MB3263.namprd21.prod.outlook.com> (raw)
In-Reply-To: <20220517150339.GI63055@ziepe.ca>

> Subject: Re: [PATCH 09/12] net: mana: Move header files to a common location
> 
> On Tue, May 17, 2022 at 02:04:33AM -0700, longli@linuxonhyperv.com wrote:
> > From: Long Li <longli@microsoft.com>
> >
> > In preparation to add MANA RDMA driver, move all the required header
> > files to a common location for use by both Ethernet and RDMA drivers.
> >
> > Signed-off-by: Long Li <longli@microsoft.com>
> > ---
> >  MAINTAINERS                                                   | 1 +
> >  drivers/net/ethernet/microsoft/mana/gdma_main.c               | 2 +-
> >  drivers/net/ethernet/microsoft/mana/hw_channel.c              | 4 ++--
> >  drivers/net/ethernet/microsoft/mana/mana_bpf.c                | 2 +-
> >  drivers/net/ethernet/microsoft/mana/mana_en.c                 | 2 +-
> >  drivers/net/ethernet/microsoft/mana/mana_ethtool.c            | 2 +-
> >  drivers/net/ethernet/microsoft/mana/shm_channel.c             | 2 +-
> >  {drivers/net/ethernet/microsoft => include/linux}/mana/gdma.h | 0
> >  .../ethernet/microsoft => include/linux}/mana/hw_channel.h    | 0
> >  {drivers/net/ethernet/microsoft => include/linux}/mana/mana.h | 0
> >  .../ethernet/microsoft => include/linux}/mana/shm_channel.h   | 0
> >  11 files changed, 8 insertions(+), 7 deletions(-)  rename
> > {drivers/net/ethernet/microsoft => include/linux}/mana/gdma.h (100%)
> > rename {drivers/net/ethernet/microsoft =>
> > include/linux}/mana/hw_channel.h (100%)  rename
> > {drivers/net/ethernet/microsoft => include/linux}/mana/mana.h (100%)
> > rename {drivers/net/ethernet/microsoft =>
> > include/linux}/mana/shm_channel.h (100%)
> 
> I know mlx5 did it like this, but I wonder if include/net is more appropriate?
> 
> Or maybe include/aux/?

I can move the header files to include/net/mana, if that sounds okay and no objection to doing that.

Long

> 
> Jason

  reply	other threads:[~2022-05-17 19:49 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-05-17  9:04 [PATCH 00/12] Introduce Microsoft Azure Network Adapter (MANA) RDMA driver longli
2022-05-17  9:04 ` [PATCH 01/12] net: mana: Add support for auxiliary device longli
2022-05-19 18:58   ` kernel test robot
2022-05-17  9:04 ` [PATCH 02/12] net: mana: Record the physical address for doorbell page region longli
2022-05-17  9:04 ` [PATCH 03/12] net: mana: Handle vport sharing between devices longli
2022-05-17 15:19   ` Stephen Hemminger
2022-05-17 23:39     ` Long Li
2022-05-17  9:04 ` [PATCH 04/12] net: mana: Add functions for allocating doorbell page from GDMA longli
2022-05-17  9:04 ` [PATCH 05/12] net: mana: Set the DMA device max page size longli
2022-05-17 14:59   ` Jason Gunthorpe
2022-05-17 19:32     ` Long Li
2022-05-17 19:35       ` Jason Gunthorpe
2022-05-17 20:04         ` Long Li
2022-05-18  0:03           ` Jason Gunthorpe
2022-05-18  5:59             ` [EXTERNAL] " Ajay Sharma
2022-05-18 16:05               ` Jason Gunthorpe
2022-05-18 21:05                 ` Ajay Sharma
2022-05-18 23:11                   ` Jason Gunthorpe
2022-05-19  0:37                     ` Long Li
2022-05-17  9:04 ` [PATCH 06/12] net: mana: Define data structures for protection domain and memory registration longli
2022-05-17  9:04 ` [PATCH 07/12] net: mana: Export Work Queue functions for use by RDMA driver longli
2022-05-17  9:04 ` [PATCH 08/12] net: mana: Record port number in netdev longli
2022-05-17  9:04 ` [PATCH 09/12] net: mana: Move header files to a common location longli
2022-05-17 15:03   ` Jason Gunthorpe
2022-05-17 19:34     ` Long Li [this message]
2022-05-17  9:04 ` [PATCH 10/12] net: mana: Define max values for SGL entries longli
2022-05-17  9:04 ` [PATCH 11/12] net: mana: Define and process GDMA response code GDMA_STATUS_MORE_ENTRIES longli
2022-05-17  9:04 ` [PATCH 12/12] RDMA/mana_ib: Add a driver for Microsoft Azure Network Adapter longli
2022-05-17 15:24   ` Jason Gunthorpe
2022-05-19  5:57     ` Long Li
2022-05-19 12:49       ` Jason Gunthorpe
2022-05-19 17:50         ` Long Li
2022-05-17 23:40   ` kernel test robot
2022-05-18  1:14   ` kernel test robot
2022-05-19 21:42   ` kernel test robot

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=PH7PR21MB3263114D9D57A3F385FCA6EBCECE9@PH7PR21MB3263.namprd21.prod.outlook.com \
    --to=longli@microsoft.com \
    --cc=davem@davemloft.net \
    --cc=decui@microsoft.com \
    --cc=haiyangz@microsoft.com \
    --cc=jgg@ziepe.ca \
    --cc=kuba@kernel.org \
    --cc=kys@microsoft.com \
    --cc=leon@kernel.org \
    --cc=linux-hyperv@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-rdma@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=pabeni@redhat.com \
    --cc=sthemmin@microsoft.com \
    --cc=wei.liu@kernel.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.