netdev - RE: [Patch v7 10/12] net: mana: Define data structures for allocating doorbell page from GDMA

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <PH7PR21MB32633B6902D7577C279C5368CE2B9@PH7PR21MB3263.namprd21.prod.outlook.com>
Date:   Wed, 19 Oct 2022 18:16:54 +0000
From:   Long Li <longli@...rosoft.com>
To:     Yunsheng Lin <linyunsheng@...wei.com>,
        KY Srinivasan <kys@...rosoft.com>,
        Haiyang Zhang <haiyangz@...rosoft.com>,
        Stephen Hemminger <sthemmin@...rosoft.com>,
        Wei Liu <wei.liu@...nel.org>, Dexuan Cui <decui@...rosoft.com>,
        "David S. Miller" <davem@...emloft.net>,
        Jakub Kicinski <kuba@...nel.org>,
        Paolo Abeni <pabeni@...hat.com>,
        Jason Gunthorpe <jgg@...pe.ca>,
        Leon Romanovsky <leon@...nel.org>,
        "edumazet@...gle.com" <edumazet@...gle.com>,
        "shiraz.saleem@...el.com" <shiraz.saleem@...el.com>,
        Ajay Sharma <sharmaajay@...rosoft.com>
CC:     "linux-hyperv@...r.kernel.org" <linux-hyperv@...r.kernel.org>,
        "netdev@...r.kernel.org" <netdev@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "linux-rdma@...r.kernel.org" <linux-rdma@...r.kernel.org>
Subject: RE: [Patch v7 10/12] net: mana: Define data structures for allocating
 doorbell page from GDMA

> Subject: Re: [Patch v7 10/12] net: mana: Define data structures for allocating
> doorbell page from GDMA
> 
> On 2022/10/18 3:20, longli@...uxonhyperv.com wrote:
> > From: Long Li <longli@...rosoft.com>
> >
> > The RDMA device needs to allocate doorbell pages for each user context.
> > Define the GDMA data structures for use by the RDMA driver.
> >
> > Reviewed-by: Dexuan Cui <decui@...rosoft.com>
> > Signed-off-by: Long Li <longli@...rosoft.com>
> > Acked-by: Haiyang Zhang <haiyangz@...rosoft.com>
> > ---
> > Change log:
> > v4: use EXPORT_SYMBOL_NS
> > v7: move mana_gd_allocate_doorbell_page() and
> > mana_gd_destroy_doorbell_page() to the RDMA driver
> >
> >  include/net/mana/gdma.h | 25 +++++++++++++++++++++++++
> >  1 file changed, 25 insertions(+)
> >
> > diff --git a/include/net/mana/gdma.h b/include/net/mana/gdma.h index
> > a9b7930dfbf8..bc060b6fa54c 100644
> > --- a/include/net/mana/gdma.h
> > +++ b/include/net/mana/gdma.h
> > @@ -24,11 +24,15 @@ enum gdma_request_type {
> >  	GDMA_GENERATE_TEST_EQE		= 10,
> >  	GDMA_CREATE_QUEUE		= 12,
> >  	GDMA_DISABLE_QUEUE		= 13,
> > +	GDMA_ALLOCATE_RESOURCE_RANGE	= 22,
> > +	GDMA_DESTROY_RESOURCE_RANGE	= 24,
> >  	GDMA_CREATE_DMA_REGION		= 25,
> >  	GDMA_DMA_REGION_ADD_PAGES	= 26,
> >  	GDMA_DESTROY_DMA_REGION		= 27,
> >  };
> >
> > +#define GDMA_RESOURCE_DOORBELL_PAGE	27
> > +
> >  enum gdma_queue_type {
> >  	GDMA_INVALID_QUEUE,
> >  	GDMA_SQ,
> > @@ -587,6 +591,26 @@ struct gdma_register_device_resp {
> >  	u32 db_id;
> >  }; /* HW DATA */
> >
> > +struct gdma_allocate_resource_range_req {
> > +	struct gdma_req_hdr hdr;
> > +	u32 resource_type;
> > +	u32 num_resources;
> > +	u32 alignment;
> > +	u32 allocated_resources;
> > +};
> > +
> > +struct gdma_allocate_resource_range_resp {
> > +	struct gdma_resp_hdr hdr;
> > +	u32 allocated_resources;
> > +};
> > +
> > +struct gdma_destroy_resource_range_req {
> > +	struct gdma_req_hdr hdr;
> > +	u32 resource_type;
> > +	u32 num_resources;
> > +	u32 allocated_resources;
> > +};
> > +
> >  /* GDMA_CREATE_QUEUE */
> >  struct gdma_create_queue_req {
> >  	struct gdma_req_hdr hdr;
> > @@ -695,4 +719,5 @@ void mana_gd_free_memory(struct
> gdma_mem_info
> > *gmi);
> >
> >  int mana_gd_send_request(struct gdma_context *gc, u32 req_len, const
> void *req,
> >  			 u32 resp_len, void *resp);
> > +
> 
> Unrelated change.

Will remove this.

> 
> >  #endif /* _GDMA_H */
> >