Commit 06f2597f authored by Michael J. Ruhl's avatar Michael J. Ruhl Committed by Doug Ledford

IB/{rdmavt, hfi1, qib}: Remove get_card_name() downcall

rdmavt has a down call to client drivers to retrieve a crafted card
name.

This name should be the IB defined name.

Rather than craft the name each time it is needed, simply retrieve
the IB allocated name from the IB device.

Update the function name to reflect its application.

Clean up driver code to match this change.
Reviewed-by: default avatarMike Marciniszyn <mike.marciniszyn@intel.com>
Signed-off-by: default avatarMichael J. Ruhl <michael.j.ruhl@intel.com>
Signed-off-by: default avatarDennis Dalessandro <dennis.dalessandro@intel.com>
Signed-off-by: default avatarDoug Ledford <dledford@redhat.com>
parent 5084c8ff
...@@ -167,14 +167,6 @@ const char *get_unit_name(int unit) ...@@ -167,14 +167,6 @@ const char *get_unit_name(int unit)
return iname; return iname;
} }
const char *get_card_name(struct rvt_dev_info *rdi)
{
struct hfi1_ibdev *ibdev = container_of(rdi, struct hfi1_ibdev, rdi);
struct hfi1_devdata *dd = container_of(ibdev,
struct hfi1_devdata, verbs_dev);
return get_unit_name(dd->unit);
}
struct pci_dev *get_pci_dev(struct rvt_dev_info *rdi) struct pci_dev *get_pci_dev(struct rvt_dev_info *rdi)
{ {
struct hfi1_ibdev *ibdev = container_of(rdi, struct hfi1_ibdev, rdi); struct hfi1_ibdev *ibdev = container_of(rdi, struct hfi1_ibdev, rdi);
......
...@@ -1972,7 +1972,6 @@ int get_platform_config_field(struct hfi1_devdata *dd, ...@@ -1972,7 +1972,6 @@ int get_platform_config_field(struct hfi1_devdata *dd,
u32 *data, u32 len); u32 *data, u32 len);
const char *get_unit_name(int unit); const char *get_unit_name(int unit);
const char *get_card_name(struct rvt_dev_info *rdi);
struct pci_dev *get_pci_dev(struct rvt_dev_info *rdi); struct pci_dev *get_pci_dev(struct rvt_dev_info *rdi);
/* /*
......
...@@ -1890,7 +1890,6 @@ int hfi1_register_ib_device(struct hfi1_devdata *dd) ...@@ -1890,7 +1890,6 @@ int hfi1_register_ib_device(struct hfi1_devdata *dd)
* Fill in rvt info object. * Fill in rvt info object.
*/ */
dd->verbs_dev.rdi.driver_f.port_callback = hfi1_create_port_files; dd->verbs_dev.rdi.driver_f.port_callback = hfi1_create_port_files;
dd->verbs_dev.rdi.driver_f.get_card_name = get_card_name;
dd->verbs_dev.rdi.driver_f.get_pci_dev = get_pci_dev; dd->verbs_dev.rdi.driver_f.get_pci_dev = get_pci_dev;
dd->verbs_dev.rdi.driver_f.check_ah = hfi1_check_ah; dd->verbs_dev.rdi.driver_f.check_ah = hfi1_check_ah;
dd->verbs_dev.rdi.driver_f.notify_new_ah = hfi1_notify_new_ah; dd->verbs_dev.rdi.driver_f.notify_new_ah = hfi1_notify_new_ah;
......
...@@ -1429,7 +1429,6 @@ u64 qib_sps_ints(void); ...@@ -1429,7 +1429,6 @@ u64 qib_sps_ints(void);
dma_addr_t qib_map_page(struct pci_dev *, struct page *, unsigned long, dma_addr_t qib_map_page(struct pci_dev *, struct page *, unsigned long,
size_t, int); size_t, int);
const char *qib_get_unit_name(int unit); const char *qib_get_unit_name(int unit);
const char *qib_get_card_name(struct rvt_dev_info *rdi);
struct pci_dev *qib_get_pci_dev(struct rvt_dev_info *rdi); struct pci_dev *qib_get_pci_dev(struct rvt_dev_info *rdi);
/* /*
......
...@@ -89,14 +89,6 @@ const char *qib_get_unit_name(int unit) ...@@ -89,14 +89,6 @@ const char *qib_get_unit_name(int unit)
return iname; return iname;
} }
const char *qib_get_card_name(struct rvt_dev_info *rdi)
{
struct qib_ibdev *ibdev = container_of(rdi, struct qib_ibdev, rdi);
struct qib_devdata *dd = container_of(ibdev,
struct qib_devdata, verbs_dev);
return qib_get_unit_name(dd->unit);
}
struct pci_dev *qib_get_pci_dev(struct rvt_dev_info *rdi) struct pci_dev *qib_get_pci_dev(struct rvt_dev_info *rdi)
{ {
struct qib_ibdev *ibdev = container_of(rdi, struct qib_ibdev, rdi); struct qib_ibdev *ibdev = container_of(rdi, struct qib_ibdev, rdi);
......
...@@ -1585,7 +1585,6 @@ int qib_register_ib_device(struct qib_devdata *dd) ...@@ -1585,7 +1585,6 @@ int qib_register_ib_device(struct qib_devdata *dd)
* Fill in rvt info object. * Fill in rvt info object.
*/ */
dd->verbs_dev.rdi.driver_f.port_callback = qib_create_port_files; dd->verbs_dev.rdi.driver_f.port_callback = qib_create_port_files;
dd->verbs_dev.rdi.driver_f.get_card_name = qib_get_card_name;
dd->verbs_dev.rdi.driver_f.get_pci_dev = qib_get_pci_dev; dd->verbs_dev.rdi.driver_f.get_pci_dev = qib_get_pci_dev;
dd->verbs_dev.rdi.driver_f.check_ah = qib_check_ah; dd->verbs_dev.rdi.driver_f.check_ah = qib_check_ah;
dd->verbs_dev.rdi.driver_f.check_send_wqe = qib_check_send_wqe; dd->verbs_dev.rdi.driver_f.check_send_wqe = qib_check_send_wqe;
......
...@@ -45,8 +45,8 @@ ...@@ -45,8 +45,8 @@
* *
*/ */
#define RDI_DEV_ENTRY(rdi) __string(dev, rdi->driver_f.get_card_name(rdi)) #define RDI_DEV_ENTRY(rdi) __string(dev, rvt_get_ibdev_name(rdi))
#define RDI_DEV_ASSIGN(rdi) __assign_str(dev, rdi->driver_f.get_card_name(rdi)) #define RDI_DEV_ASSIGN(rdi) __assign_str(dev, rvt_get_ibdev_name(rdi))
#include "trace_rvt.h" #include "trace_rvt.h"
#include "trace_qp.h" #include "trace_qp.h"
......
...@@ -413,7 +413,6 @@ static noinline int check_support(struct rvt_dev_info *rdi, int verb) ...@@ -413,7 +413,6 @@ static noinline int check_support(struct rvt_dev_info *rdi, int verb)
* required for rdmavt to function. * required for rdmavt to function.
*/ */
if ((!rdi->driver_f.port_callback) || if ((!rdi->driver_f.port_callback) ||
(!rdi->driver_f.get_card_name) ||
(!rdi->driver_f.get_pci_dev)) (!rdi->driver_f.get_pci_dev))
return -EINVAL; return -EINVAL;
break; break;
......
...@@ -63,19 +63,19 @@ ...@@ -63,19 +63,19 @@
#define rvt_pr_info(rdi, fmt, ...) \ #define rvt_pr_info(rdi, fmt, ...) \
__rvt_pr_info(rdi->driver_f.get_pci_dev(rdi), \ __rvt_pr_info(rdi->driver_f.get_pci_dev(rdi), \
rdi->driver_f.get_card_name(rdi), \ rvt_get_ibdev_name(rdi), \
fmt, \ fmt, \
##__VA_ARGS__) ##__VA_ARGS__)
#define rvt_pr_warn(rdi, fmt, ...) \ #define rvt_pr_warn(rdi, fmt, ...) \
__rvt_pr_warn(rdi->driver_f.get_pci_dev(rdi), \ __rvt_pr_warn(rdi->driver_f.get_pci_dev(rdi), \
rdi->driver_f.get_card_name(rdi), \ rvt_get_ibdev_name(rdi), \
fmt, \ fmt, \
##__VA_ARGS__) ##__VA_ARGS__)
#define rvt_pr_err(rdi, fmt, ...) \ #define rvt_pr_err(rdi, fmt, ...) \
__rvt_pr_err(rdi->driver_f.get_pci_dev(rdi), \ __rvt_pr_err(rdi->driver_f.get_pci_dev(rdi), \
rdi->driver_f.get_card_name(rdi), \ rvt_get_ibdev_name(rdi), \
fmt, \ fmt, \
##__VA_ARGS__) ##__VA_ARGS__)
......
...@@ -227,13 +227,6 @@ struct rvt_driver_provided { ...@@ -227,13 +227,6 @@ struct rvt_driver_provided {
/* Passed to ib core registration. Callback to create syfs files */ /* Passed to ib core registration. Callback to create syfs files */
int (*port_callback)(struct ib_device *, u8, struct kobject *); int (*port_callback)(struct ib_device *, u8, struct kobject *);
/*
* Returns a string to represent the device for which is being
* registered. This is primarily used for error and debug messages on
* the console.
*/
const char * (*get_card_name)(struct rvt_dev_info *rdi);
/* /*
* Returns a pointer to the undelying hardware's PCI device. This is * Returns a pointer to the undelying hardware's PCI device. This is
* used to display information as to what hardware is being referenced * used to display information as to what hardware is being referenced
...@@ -432,6 +425,17 @@ static inline void rvt_set_ibdev_name(struct rvt_dev_info *rdi, ...@@ -432,6 +425,17 @@ static inline void rvt_set_ibdev_name(struct rvt_dev_info *rdi,
snprintf(rdi->ibdev.name, sizeof(rdi->ibdev.name), fmt, name, unit); snprintf(rdi->ibdev.name, sizeof(rdi->ibdev.name), fmt, name, unit);
} }
/**
* rvt_get_ibdev_name - return the IB name
* @rdi: rdmavt device
*
* Return the registered name of the device.
*/
static inline const char *rvt_get_ibdev_name(const struct rvt_dev_info *rdi)
{
return rdi->ibdev.name;
}
static inline struct rvt_pd *ibpd_to_rvtpd(struct ib_pd *ibpd) static inline struct rvt_pd *ibpd_to_rvtpd(struct ib_pd *ibpd)
{ {
return container_of(ibpd, struct rvt_pd, ibpd); return container_of(ibpd, struct rvt_pd, ibpd);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment