diff options
| author | Jon Mason <jon.mason@intel.com> | 2013-01-17 15:28:45 -0700 |
|---|---|---|
| committer | Jon Mason <jon.mason@intel.com> | 2013-05-15 10:58:06 -0700 |
| commit | c336acd3331dcc191a97dbc66a557d47741657c7 (patch) | |
| tree | 8dfb29b36fe7f143588877f205df7b7cb2008024 /drivers/ntb | |
| parent | c9d534c8cbaedbb522a1d2cb037c6c394f610317 (diff) | |
| download | lwn-c336acd3331dcc191a97dbc66a557d47741657c7.tar.gz lwn-c336acd3331dcc191a97dbc66a557d47741657c7.zip | |
NTB: memcpy lockup workaround
The system will appear to lockup for long periods of time due to the NTB
driver spending too much time in memcpy. Avoid this by reducing the
number of packets that can be serviced on a given interrupt.
Signed-off-by: Jon Mason <jon.mason@intel.com>
Diffstat (limited to 'drivers/ntb')
| -rw-r--r-- | drivers/ntb/ntb_transport.c | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/drivers/ntb/ntb_transport.c b/drivers/ntb/ntb_transport.c index cd9745b062e4..583a7d3f0ce1 100644 --- a/drivers/ntb/ntb_transport.c +++ b/drivers/ntb/ntb_transport.c @@ -1034,11 +1034,16 @@ out: static void ntb_transport_rx(unsigned long data) { struct ntb_transport_qp *qp = (struct ntb_transport_qp *)data; - int rc; + int rc, i; - do { + /* Limit the number of packets processed in a single interrupt to + * provide fairness to others + */ + for (i = 0; i < qp->rx_max_entry; i++) { rc = ntb_process_rxc(qp); - } while (!rc); + if (rc) + break; + } } static void ntb_transport_rxc_db(void *data, int db_num) |