Re: [PATCH 37/37] IB/rdmavt: Add support for new memory registration API

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




This patch exists to provide parity for what is in qib. Should we not
have it?  If not, why do we have:

commit 38071a461f0a ("IB/qib: Support the new memory registration API")

That was done by me because I saw this in qib and assumed that it was
supported. Now that I found out that it isn't, I'd say it should be
removed altogether shouldn't it?


That doesn't mean it can't be added to rdmavt as a future enhancement
though if there is a need.

Well, given that we're trying to consolidate on post send registration
interface it's kind of a must I'd say.

Are you asking because soft-roce will need it?

I was asking in general, but in specific soft-roce as a consumer will
need to support that yes.

I think it makes sense to revisit when soft-roce comes in,

I agree.

since qib/hfi do not need IB_WR_LOCAL_INV.

Can you explain? Does qib/hfi have a magic way to invalidate memory
regions?
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux