From: Gal Pressman <galpress@amazon.com> To: Shiraz Saleem <shiraz.saleem@intel.com>, dledford@redhat.com, jgg@ziepe.ca, davem@davemloft.net Cc: linux-rdma@vger.kernel.org, netdev@vger.kernel.org, mustafa.ismail@intel.com, jeffrey.t.kirsher@intel.com Subject: Re: [RFC v1 04/19] RDMA/irdma: Add driver framework definitions Date: Sun, 24 Feb 2019 17:02:10 +0200 [thread overview] Message-ID: <5215c158-2935-ef45-db49-44caa84e07d3@amazon.com> (raw) In-Reply-To: <20190215171107.6464-5-shiraz.saleem@intel.com> On 15-Feb-19 19:10, Shiraz Saleem wrote: > +/* client interface functions */ > +static const struct i40e_client_ops i40e_ops = { > + .open = i40iw_open, > + .close = i40iw_close, > + .l2_param_change = i40iw_l2param_change, > + .virtchnl_receive = NULL, > + .vf_reset = NULL, > + .vf_enable = NULL, > + .vf_capable = NULL NULL assignments are redundant. > +}; > + > diff --git a/drivers/infiniband/hw/irdma/irdma_if.c b/drivers/infiniband/hw/irdma/irdma_if.c > new file mode 100644 > index 0000000..f7b89e9 > --- /dev/null > +++ b/drivers/infiniband/hw/irdma/irdma_if.c > @@ -0,0 +1,430 @@ > +// SPDX-License-Identifier: GPL-2.0 or Linux-OpenIB > +/* Copyright (c) 2019, Intel Corporation. */ > + > +#include <linux/module.h> > +#include <linux/moduleparam.h> > +#include <ice_idc.h> > +#include "main.h" > +#include "ws.h" > +#include "icrdma_hw.h" > + > +void irdma_add_dev_ref(struct irdma_sc_dev *dev) > +{ > + try_module_get(THIS_MODULE); > +} > + > +void irdma_put_dev_ref(struct irdma_sc_dev *dev) > +{ > + module_put(THIS_MODULE); > +} What are these used for? > + > +/** > + * irdma_find_iwdev - find a vsi device given a name > + * @name: name of iwdev > + */ Can't find uses of this function as well. > +struct irdma_device *irdma_find_iwdev(const char *name) > +{ > + struct irdma_handler *hdl; > + struct list_head *pos; > + struct list_head *tmp; > + struct irdma_device *iwdev; > + unsigned long flags; > + > + spin_lock_irqsave(&irdma_handler_lock, flags); > + list_for_each_entry(hdl, &irdma_handlers, list) { > + list_for_each_safe(pos, tmp, &hdl->rf.vsi_dev_list) { > + iwdev = container_of(pos, struct irdma_device, list); > + if (!strcmp(name, iwdev->iwibdev->ibdev.name)) { > + spin_unlock_irqrestore(&irdma_handler_lock, > + flags); > + return iwdev; > + } > + } > + } > + spin_unlock_irqrestore(&irdma_handler_lock, flags); > + > + return NULL; > +} > +
WARNING: multiple messages have this Message-ID (diff)
From: Gal Pressman <galpress@amazon.com> To: Shiraz Saleem <shiraz.saleem@intel.com>, <dledford@redhat.com>, <jgg@ziepe.ca>, <davem@davemloft.net> Cc: <linux-rdma@vger.kernel.org>, <netdev@vger.kernel.org>, <mustafa.ismail@intel.com>, <jeffrey.t.kirsher@intel.com> Subject: Re: [RFC v1 04/19] RDMA/irdma: Add driver framework definitions Date: Sun, 24 Feb 2019 17:02:10 +0200 [thread overview] Message-ID: <5215c158-2935-ef45-db49-44caa84e07d3@amazon.com> (raw) In-Reply-To: <20190215171107.6464-5-shiraz.saleem@intel.com> On 15-Feb-19 19:10, Shiraz Saleem wrote: > +/* client interface functions */ > +static const struct i40e_client_ops i40e_ops = { > + .open = i40iw_open, > + .close = i40iw_close, > + .l2_param_change = i40iw_l2param_change, > + .virtchnl_receive = NULL, > + .vf_reset = NULL, > + .vf_enable = NULL, > + .vf_capable = NULL NULL assignments are redundant. > +}; > + > diff --git a/drivers/infiniband/hw/irdma/irdma_if.c b/drivers/infiniband/hw/irdma/irdma_if.c > new file mode 100644 > index 0000000..f7b89e9 > --- /dev/null > +++ b/drivers/infiniband/hw/irdma/irdma_if.c > @@ -0,0 +1,430 @@ > +// SPDX-License-Identifier: GPL-2.0 or Linux-OpenIB > +/* Copyright (c) 2019, Intel Corporation. */ > + > +#include <linux/module.h> > +#include <linux/moduleparam.h> > +#include <ice_idc.h> > +#include "main.h" > +#include "ws.h" > +#include "icrdma_hw.h" > + > +void irdma_add_dev_ref(struct irdma_sc_dev *dev) > +{ > + try_module_get(THIS_MODULE); > +} > + > +void irdma_put_dev_ref(struct irdma_sc_dev *dev) > +{ > + module_put(THIS_MODULE); > +} What are these used for? > + > +/** > + * irdma_find_iwdev - find a vsi device given a name > + * @name: name of iwdev > + */ Can't find uses of this function as well. > +struct irdma_device *irdma_find_iwdev(const char *name) > +{ > + struct irdma_handler *hdl; > + struct list_head *pos; > + struct list_head *tmp; > + struct irdma_device *iwdev; > + unsigned long flags; > + > + spin_lock_irqsave(&irdma_handler_lock, flags); > + list_for_each_entry(hdl, &irdma_handlers, list) { > + list_for_each_safe(pos, tmp, &hdl->rf.vsi_dev_list) { > + iwdev = container_of(pos, struct irdma_device, list); > + if (!strcmp(name, iwdev->iwibdev->ibdev.name)) { > + spin_unlock_irqrestore(&irdma_handler_lock, > + flags); > + return iwdev; > + } > + } > + } > + spin_unlock_irqrestore(&irdma_handler_lock, flags); > + > + return NULL; > +} > +
next prev parent reply other threads:[~2019-02-24 15:02 UTC|newest] Thread overview: 59+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-02-15 17:10 [RFC v1 00/19] Add unified Intel Ethernet RDMA driver (irdma) Shiraz Saleem 2019-02-15 17:10 ` [RFC v1 01/19] net/i40e: Add peer register/unregister to struct i40e_netdev_priv Shiraz Saleem 2019-02-15 17:22 ` Jason Gunthorpe 2019-02-21 2:19 ` Saleem, Shiraz 2019-02-21 19:35 ` Jason Gunthorpe 2019-02-22 20:13 ` Ertman, David M 2019-02-22 20:23 ` Jason Gunthorpe 2019-03-13 2:11 ` Jeff Kirsher 2019-03-13 13:28 ` Jason Gunthorpe 2019-05-10 13:31 ` Shiraz Saleem 2019-05-10 18:17 ` Jason Gunthorpe 2019-02-15 17:10 ` [RFC v1 02/19] net/ice: Create framework for VSI queue context Shiraz Saleem 2019-02-15 17:10 ` [RFC v1 03/19] net/ice: Add support for ice peer devices and drivers Shiraz Saleem 2019-02-15 17:10 ` [RFC v1 04/19] RDMA/irdma: Add driver framework definitions Shiraz Saleem 2019-02-24 15:02 ` Gal Pressman [this message] 2019-02-24 15:02 ` Gal Pressman 2019-02-26 21:08 ` Saleem, Shiraz 2019-02-15 17:10 ` [RFC v1 05/19] RDMA/irdma: Implement device initialization definitions Shiraz Saleem 2019-02-15 17:10 ` [RFC v1 06/19] RDMA/irdma: Implement HW Admin Queue OPs Shiraz Saleem 2019-02-15 17:10 ` [RFC v1 07/19] RDMA/irdma: Add HMC backing store setup functions Shiraz Saleem 2019-02-15 17:10 ` [RFC v1 08/19] RDMA/irdma: Add privileged UDA queue implementation Shiraz Saleem 2019-02-24 11:42 ` Gal Pressman 2019-02-24 11:42 ` Gal Pressman 2019-02-15 17:10 ` [RFC v1 09/19] RDMA/irdma: Add QoS definitions Shiraz Saleem 2019-02-15 17:10 ` [RFC v1 10/19] RDMA/irdma: Add connection manager Shiraz Saleem 2019-02-24 11:21 ` Gal Pressman 2019-02-24 11:21 ` Gal Pressman 2019-02-25 18:46 ` Jason Gunthorpe 2019-02-26 21:07 ` Saleem, Shiraz 2019-02-15 17:10 ` [RFC v1 11/19] RDMA/irdma: Add PBLE resource manager Shiraz Saleem 2019-02-27 6:58 ` Leon Romanovsky 2019-02-15 17:10 ` [RFC v1 12/19] RDMA/irdma: Implement device supported verb APIs Shiraz Saleem 2019-02-15 17:35 ` Jason Gunthorpe 2019-02-15 22:19 ` Shiraz Saleem 2019-02-15 22:32 ` Jason Gunthorpe 2019-02-20 14:52 ` Saleem, Shiraz 2019-02-20 16:51 ` Jason Gunthorpe 2019-02-24 14:35 ` Gal Pressman 2019-02-24 14:35 ` Gal Pressman 2019-02-25 18:50 ` Jason Gunthorpe 2019-02-26 21:09 ` Saleem, Shiraz 2019-02-26 21:09 ` Saleem, Shiraz 2019-02-27 7:31 ` Gal Pressman 2019-02-15 17:11 ` [RFC v1 13/19] RDMA/irdma: Add RoCEv2 UD OP support Shiraz Saleem 2019-02-27 6:50 ` Leon Romanovsky 2019-02-15 17:11 ` [RFC v1 14/19] RDMA/irdma: Add user/kernel shared libraries Shiraz Saleem 2019-02-15 17:11 ` [RFC v1 15/19] RDMA/irdma: Add miscellaneous utility definitions Shiraz Saleem 2019-02-15 17:47 ` Jason Gunthorpe 2019-02-20 7:51 ` Leon Romanovsky 2019-02-20 14:53 ` Saleem, Shiraz 2019-02-20 16:53 ` Jason Gunthorpe 2019-02-15 17:11 ` [RFC v1 16/19] RDMA/irdma: Add dynamic tracing for CM Shiraz Saleem 2019-02-15 17:11 ` [RFC v1 17/19] RDMA/irdma: Add ABI definitions Shiraz Saleem 2019-02-15 17:16 ` Jason Gunthorpe 2019-02-20 14:52 ` Saleem, Shiraz 2019-02-20 16:50 ` Jason Gunthorpe 2019-02-15 17:11 ` [RFC v1 18/19] RDMA/irdma: Add Kconfig and Makefile Shiraz Saleem 2019-02-15 17:11 ` [RFC v1 19/19] RDMA/irdma: Update MAINTAINERS file Shiraz Saleem 2019-02-15 17:20 ` [RFC v1 00/19] Add unified Intel Ethernet RDMA driver (irdma) Jason Gunthorpe
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=5215c158-2935-ef45-db49-44caa84e07d3@amazon.com \ --to=galpress@amazon.com \ --cc=davem@davemloft.net \ --cc=dledford@redhat.com \ --cc=jeffrey.t.kirsher@intel.com \ --cc=jgg@ziepe.ca \ --cc=linux-rdma@vger.kernel.org \ --cc=mustafa.ismail@intel.com \ --cc=netdev@vger.kernel.org \ --cc=shiraz.saleem@intel.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: linkBe 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.