Commit 249168ad authored by Thomas Falcon's avatar Thomas Falcon Committed by David S. Miller

ibmvnic: Make CRQ interrupt tasklet wait for all capabilities crqs

After sending device capability queries and requests to the vNIC Server,
an interrupt is triggered and the responses are written to the driver's
CRQ response buffer. Since the interrupt can be triggered before all
responses are written and visible to the partition, there is a danger
that the interrupt handler or tasklet can terminate before all responses
are read, resulting in a failure to initialize the device.

To avoid this scenario, when capability commands are sent, we set
a flag that will be checked in the following interrupt tasklet that
will handle the capability responses from the server. Once all
responses have been handled, the flag is disabled; and the tasklet
is allowed to terminate.
Signed-off-by: default avatarThomas Falcon <tlfalcon@linux.vnet.ibm.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 901e040a
...@@ -2418,6 +2418,7 @@ static void handle_request_cap_rsp(union ibmvnic_crq *crq, ...@@ -2418,6 +2418,7 @@ static void handle_request_cap_rsp(union ibmvnic_crq *crq,
struct ibmvnic_query_ip_offload_buffer *ip_offload_buf = struct ibmvnic_query_ip_offload_buffer *ip_offload_buf =
&adapter->ip_offload_buf; &adapter->ip_offload_buf;
adapter->wait_capability = false;
adapter->ip_offload_tok = dma_map_single(dev, ip_offload_buf, adapter->ip_offload_tok = dma_map_single(dev, ip_offload_buf,
buf_sz, buf_sz,
DMA_FROM_DEVICE); DMA_FROM_DEVICE);
...@@ -2713,9 +2714,11 @@ static void handle_query_cap_rsp(union ibmvnic_crq *crq, ...@@ -2713,9 +2714,11 @@ static void handle_query_cap_rsp(union ibmvnic_crq *crq,
} }
out: out:
if (atomic_read(&adapter->running_cap_crqs) == 0) if (atomic_read(&adapter->running_cap_crqs) == 0) {
adapter->wait_capability = false;
init_sub_crqs(adapter, 0); init_sub_crqs(adapter, 0);
/* We're done querying the capabilities, initialize sub-crqs */ /* We're done querying the capabilities, initialize sub-crqs */
}
} }
static void handle_control_ras_rsp(union ibmvnic_crq *crq, static void handle_control_ras_rsp(union ibmvnic_crq *crq,
...@@ -3458,9 +3461,18 @@ static void ibmvnic_tasklet(void *data) ...@@ -3458,9 +3461,18 @@ static void ibmvnic_tasklet(void *data)
ibmvnic_handle_crq(crq, adapter); ibmvnic_handle_crq(crq, adapter);
crq->generic.first = 0; crq->generic.first = 0;
} else { } else {
done = true; /* remain in tasklet until all
* capabilities responses are received
*/
if (!adapter->wait_capability)
done = true;
} }
} }
/* if capabilities CRQ's were sent in this tasklet, the following
* tasklet must wait until all responses are received
*/
if (atomic_read(&adapter->running_cap_crqs) != 0)
adapter->wait_capability = true;
spin_unlock_irqrestore(&queue->lock, flags); spin_unlock_irqrestore(&queue->lock, flags);
} }
......
...@@ -977,6 +977,7 @@ struct ibmvnic_adapter { ...@@ -977,6 +977,7 @@ struct ibmvnic_adapter {
int login_rsp_buf_sz; int login_rsp_buf_sz;
atomic_t running_cap_crqs; atomic_t running_cap_crqs;
bool wait_capability;
struct ibmvnic_sub_crq_queue **tx_scrq; struct ibmvnic_sub_crq_queue **tx_scrq;
struct ibmvnic_sub_crq_queue **rx_scrq; struct ibmvnic_sub_crq_queue **rx_scrq;
......
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