mlx4_core: Don't read reserved fields in mlx4_QUERY_ADAPTER()

Previous thread: IB/mthca: Fix and simplify page size calculation in mthca_reg_phys_mr() by Linux Kernel Mailing List on Tuesday, February 5, 2008 - 1:06 pm. (1 message)

Next thread: IB/srp: Retry stale connections by Linux Kernel Mailing List on Tuesday, February 5, 2008 - 1:06 pm. (1 message)
From: Linux Kernel Mailing List
Date: Tuesday, February 5, 2008 - 1:06 pm

Gitweb:     http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=893da7...
Commit:     893da75956ab48545e8732b46e1cf4350bd25f9c
Parent:     6ccef1de2c1718729dd1c7ee8bd98473519eb3b3
Author:     Jack Morgenstein <jackm@dev.mellanox.co.il>
AuthorDate: Sun Jan 27 18:13:25 2008 +0200
Committer:  Roland Dreier <rolandd@cisco.com>
CommitDate: Mon Feb 4 20:20:43 2008 -0800

    mlx4_core: Don't read reserved fields in mlx4_QUERY_ADAPTER()
    
    The firmware QUERY_ADAPTER command does not return vendor_id,
    device_id, and revision_id; eliminate these fields from the query.
    
    Initialize the rev_id field of the mlx4 device via init_node_data (MAD
    IFC query), as is done in the query_device verb implementation.
    
    Signed-off-by: Jack Morgenstein <jackm@dev.mellanox.co.il>
    Signed-off-by: Roland Dreier <rolandd@cisco.com>
---
 drivers/infiniband/hw/mlx4/main.c |    1 +
 drivers/net/mlx4/fw.c             |    6 ------
 drivers/net/mlx4/fw.h             |    3 ---
 drivers/net/mlx4/main.c           |    1 -
 4 files changed, 1 insertions(+), 10 deletions(-)

diff --git a/drivers/infiniband/hw/mlx4/main.c b/drivers/infiniband/hw/mlx4/main.c
index d8287d9..d2f50b6 100644
--- a/drivers/infiniband/hw/mlx4/main.c
+++ b/drivers/infiniband/hw/mlx4/main.c
@@ -468,6 +468,7 @@ static int init_node_data(struct mlx4_ib_dev *dev)
 	if (err)
 		goto out;
 
+	dev->dev->rev_id = be32_to_cpup((__be32 *) (out_mad->data + 32));
 	memcpy(&dev->ib_dev.node_guid, out_mad->data + 12, 8);
 
 out:
diff --git a/drivers/net/mlx4/fw.c b/drivers/net/mlx4/fw.c
index 535a446..61dc495 100644
--- a/drivers/net/mlx4/fw.c
+++ b/drivers/net/mlx4/fw.c
@@ -617,9 +617,6 @@ int mlx4_QUERY_ADAPTER(struct mlx4_dev *dev, struct mlx4_adapter *adapter)
 	int err;
 
 #define QUERY_ADAPTER_OUT_SIZE             0x100
-#define QUERY_ADAPTER_VENDOR_ID_OFFSET     0x00
-#define QUERY_ADAPTER_DEVICE_ID_OFFSET     0x04
-#define ...
Previous thread: IB/mthca: Fix and simplify page size calculation in mthca_reg_phys_mr() by Linux Kernel Mailing List on Tuesday, February 5, 2008 - 1:06 pm. (1 message)

Next thread: IB/srp: Retry stale connections by Linux Kernel Mailing List on Tuesday, February 5, 2008 - 1:06 pm. (1 message)