提交 5919ce29 编写于 作者: B Brian King 提交者: James Bottomley

[SCSI] ibmvfc: Fix command timeouts due to cached CRQ access

The CRQs used by the ibmvfc driver are read and written by both
the client and the server. Therefore, we need to mark them volatile
so that we do not cache their contents when handling an interrupt.
This fixes a problem which can surface as occasional command timeouts.
No commands were actually timing out, but due to accessing cached data
for the CRQ in the interrupt handler, the interrupt was not processing
all command completions as it should.
Signed-off-by: NBrian King <brking@linux.vnet.ibm.com>
Signed-off-by: NJames Bottomley <James.Bottomley@HansenPartnership.com>
上级 4081b77c
...@@ -523,10 +523,10 @@ enum ibmvfc_async_event { ...@@ -523,10 +523,10 @@ enum ibmvfc_async_event {
}; };
struct ibmvfc_crq { struct ibmvfc_crq {
u8 valid; volatile u8 valid;
u8 format; volatile u8 format;
u8 reserved[6]; u8 reserved[6];
u64 ioba; volatile u64 ioba;
}__attribute__((packed, aligned (8))); }__attribute__((packed, aligned (8)));
struct ibmvfc_crq_queue { struct ibmvfc_crq_queue {
...@@ -536,13 +536,13 @@ struct ibmvfc_crq_queue { ...@@ -536,13 +536,13 @@ struct ibmvfc_crq_queue {
}; };
struct ibmvfc_async_crq { struct ibmvfc_async_crq {
u8 valid; volatile u8 valid;
u8 pad[3]; u8 pad[3];
u32 pad2; u32 pad2;
u64 event; volatile u64 event;
u64 scsi_id; volatile u64 scsi_id;
u64 wwpn; volatile u64 wwpn;
u64 node_name; volatile u64 node_name;
u64 reserved; u64 reserved;
}__attribute__((packed, aligned (8))); }__attribute__((packed, aligned (8)));
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册