On 11/18/2015 4:16 PM, Sagi Grimberg wrote:
Sagi, it works in TGT and AFAIR with the initiator too.
Looking on this paper of Pete Wyckoff [1] I see that he says that
few changes to the initiator were needed, not sure which.
I see. I wasn't aware that TGT supports bidi. However, AFAICT the
initiator support was never fully introduced upstream nor in our mlnx
backports (perhaps in an off-tree implementation). As I see it, bidi
functionality, is broken for a long time (if it was ever supported).
Sounds like we weren't communicating enough while reviewing the patches
since you joined as maintainer... lets improve.
So I'd suggest we (you and me Or) look at bidi separately and try to
find out if someone wants it (not for academic research).
I didn't follow on the comment, what's wrong with having the upstream
kernel serve
for academics? features used for academics today might turn to
production tomorrow.
It's not that we're writing a whole new driver for that, there's one
piece in our
design/code which is good for that purpose, this is perfectly fine.
Would you mind if we don't include bidi considerations in this patchset?
You should not further break it, whatever is still there should remain.
As for breakages
that were introduced over the last few cycles, we should think that to do.
Or.
--
To unsubscribe from this list: send the line "unsubscribe target-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
