diff options
author | Jan Glauber <jang@linux.vnet.ibm.com> | 2008-12-25 07:38:47 -0500 |
---|---|---|
committer | Martin Schwidefsky <schwidefsky@de.ibm.com> | 2008-12-25 07:38:59 -0500 |
commit | 50f769df1c4bea766c4eb927eae35728fb93e305 (patch) | |
tree | c86228e6965de0ea84bddc2104d7dfa339e2a187 /drivers/s390/cio/qdio_setup.c | |
parent | 22f9934767f49012ffbae753b28b8055bd28348f (diff) |
[S390] qdio: improve inbound buffer acknowledgement
- Use automatic acknowledgement of incoming buffers in QEBSM mode
- Move ACK for non-QEBSM mode always to the newest buffer to prevent
a race with qdio_stop_polling
- Remove the polling spinlock, the upper layer drivers return new buffers
in the same code path and could not run in parallel
- Don't flood the error log in case of no-target-buffer-empty
- In handle_inbound we check if we would overwrite an ACK'ed buffer, if so
advance the pointer to the oldest ACK'ed buffer so we don't overwrite an
empty buffer in qdio_stop_polling
Signed-off-by: Jan Glauber <jang@linux.vnet.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
Diffstat (limited to 'drivers/s390/cio/qdio_setup.c')
-rw-r--r-- | drivers/s390/cio/qdio_setup.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/s390/cio/qdio_setup.c b/drivers/s390/cio/qdio_setup.c index 7323799b2ce3..18d54fc21ce9 100644 --- a/drivers/s390/cio/qdio_setup.c +++ b/drivers/s390/cio/qdio_setup.c | |||
@@ -167,7 +167,6 @@ static void setup_queues(struct qdio_irq *irq_ptr, | |||
167 | setup_queues_misc(q, irq_ptr, qdio_init->input_handler, i); | 167 | setup_queues_misc(q, irq_ptr, qdio_init->input_handler, i); |
168 | 168 | ||
169 | q->is_input_q = 1; | 169 | q->is_input_q = 1; |
170 | spin_lock_init(&q->u.in.lock); | ||
171 | setup_storage_lists(q, irq_ptr, input_sbal_array, i); | 170 | setup_storage_lists(q, irq_ptr, input_sbal_array, i); |
172 | input_sbal_array += QDIO_MAX_BUFFERS_PER_Q; | 171 | input_sbal_array += QDIO_MAX_BUFFERS_PER_Q; |
173 | 172 | ||