[openib-general] [PATCH] (resend) mthca_eq improvements
Roland Dreier
roland at topspin.com
Mon Jan 17 08:04:39 PST 2005
Michael> Hello, Roland, all! Here are some mthca eq improvements,
Michael> rolled into one patch. These are all touching the same
Michael> code and so are related, but if its too big to be
Michael> readable, let me know and I'll try to split it up.
It looks like you concatenated more than one patch (there are out of
order chunks, which at least my version of patch complains about).
Anyway, easy to split up...
Michael> 1. general interrupt handling speedup I noticed that
Michael> several operations where performed multiple times, for
Michael> example get_eqe was called multiple times, as another
Michael> example, consumer index was truncated to eq size upon
Michael> each eqe found.
Michael> 2. non-MSI interrupt handling simplification This is the
Michael> 3'd chunk from the end.
Looks great, I'll apply these two pieces. With MSI-X I get about a 3%
increase in throughput with netperf over IPoIB with these changes.
Michael> 3. Added memfree support in the eq initialisation
Michael> command. This is in the last 2 chunks.
This looks wrong to me:
+ if (dev->hca_type == ARBEL_NATIVE)
+ page = MTHCA_KAR_PAGE;
+ else
+ page = 0;
the test is reversed, right?
I'm going to hold off applying this change until I have the rest of
mem-free EQ support ready.
Thanks,
Roland
More information about the general
mailing list