Re: Move the definitions for rxe_av.network_type to uAPI

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

 



On Fri, Oct 16, 2020 at 03:20:50PM -0500, Bob Pearson wrote:
> Jason,
> 
> Your recent commit:
> 
> 	commit e0d696d201dd5d31813787d9b61a42fc459eee89
> 	Author: Jason Gunthorpe <jgg@xxxxxxxx>
> 	Date:   Thu Oct 15 20:42:18 2020 -0300
> 
> 	RDMA/rxe: Move the definitions for rxe_av.network_type to uAPI
> 
> has some problems and so far I am having trouble making it work sensibly.
> 
> What you have done is to make the network_type field in rxe_av be private,
> (i.e. RXE_NETWORK_TYPE_XXX instead of RDMA_NETWORK_XXX). You then defined these
> private enums in rdma_uverbs_rxe.h. The problem is that there are more than one
> source of AVs those:

Bah, I missed rxe_av_fill_ip_info(), still it should just transcode
for now.

> There was also a confusion between V6 and V4. I will submit a small patch that
> fixes that and makes the enums the same again which is more efficient than transocding
> them.

Bah again. People need to check the stuff I write at 9pm :P

Jason



[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