Re: [for-next PATCH v5 00/11] RDMA/rxe: Add RDMA FLUSH operation

[Yanjun Zhu]

在 2022/11/11 13:10, lizhijian@xxxxxxxxxxx 写道:
> On 11/11/2022 10:49, Yanjun Zhu wrote:
> > 在 2022/10/29 1:57, Jason Gunthorpe 写道:
> > > On Tue, Sep 27, 2022 at 01:53:26PM +0800, Li Zhijian wrote:
> > > > Hey folks,
> > > >
> > > > Firstly i want to say thank you to all you guys, especially Bob, who
> > > > in the
> > > > past 1+ month, gave me a lots of idea and inspiration.
> > >
> > > I would like it if someone familiar with rxe could reviewed-by the
> > > protocol parts.
> >
> > Hi, Jason
> >
> > I reviewed these patches. I am fine with these patches.
> >
> > Hi, Zhijian
> >
> > I noticed the followings:
> > "
> > $ ./rdma_flush_server -s [server_address] -p [port_number]
> > client:
> > $ ./rdma_flush_client -s [server_address] -p [port_number]
> > "
> > Can you merge the server and the client to rdma-core?
>
> Yanjun,
>
> Yes, there was already a draft PR here
> https://github.com/linux-rdma/rdma-core/pull/1181, but it cannot go
> ahead until the kernel's patches are merged.
>
> and i will post a new version these days, would you mind if i add your
> "Reviewed-by" in next version ?

Reviewed-by: Zhu Yanjun <yanjun.zhu@xxxxxxxxx>
Thanks.

Another problem, normally rxe should connect to physical ib devices, 
such as mlx ib device. That is, one host is rxe, the other host is mlx 
ib device. The rdma connection should be created between the 2 hosts.

Do you connect to mlx ib device with this RDMA FLUSH operation?
And what is the test result?

Thanks a lot.
Zhu Yanjun

> > Thanks,
> > Zhu Yanjun
> >
> > > Jason