diff options
| author | Julian Wiedmann <jwi@linux.ibm.com> | 2020-06-17 15:30:14 +0200 |
|---|---|---|
| committer | Heiko Carstens <hca@linux.ibm.com> | 2020-07-20 10:55:22 +0200 |
| commit | 2bbf282a5e8e7e6b36586718b484a36117b6b8a0 (patch) | |
| tree | 6cad0b850638a7f616fffeca2eb54a5fbdcd21ec | |
| parent | s390/qdio: fix statistics for 128 SBALs (diff) | |
| download | linux-2bbf282a5e8e7e6b36586718b484a36117b6b8a0.tar.xz linux-2bbf282a5e8e7e6b36586718b484a36117b6b8a0.zip | |
s390/qdio: allow to scan all 128 Input SBALs
The comment is inaccurate, qdio_inbound_q_moved() and/or its callers no
longer get confused by a count of 128 completed SBALs.
Scanning all 128 SBALs at once can improve IRQ reduction (as we now
place the ACK at the right spot), and reduce the amount of processing
needed to handle all completed SBALs.
Signed-off-by: Julian Wiedmann <jwi@linux.ibm.com>
Signed-off-by: Heiko Carstens <hca@linux.ibm.com>
| -rw-r--r-- | drivers/s390/cio/qdio_main.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/drivers/s390/cio/qdio_main.c b/drivers/s390/cio/qdio_main.c index d4c699773070..0c1f186c6291 100644 --- a/drivers/s390/cio/qdio_main.c +++ b/drivers/s390/cio/qdio_main.c @@ -457,11 +457,7 @@ static int get_inbound_buffer_frontier(struct qdio_q *q, unsigned int start) q->timestamp = get_tod_clock_fast(); - /* - * Don't check 128 buffers, as otherwise qdio_inbound_q_moved - * would return 0. - */ - count = min(atomic_read(&q->nr_buf_used), QDIO_MAX_BUFFERS_MASK); + count = atomic_read(&q->nr_buf_used); if (!count) return 0; |