On Wed, 2016-08-17 at 09:55 +0300, Leon Romanovsky wrote:
> On Wed, Aug 17, 2016 at 02:16:31AM -0400, ira.weiny wrote:
> > On Tue, Aug 16, 2016 at 04:45:21PM +0300, Leon Romanovsky wrote:
> > > From: Leon Romanovsky <leonro@mellanox.com>
> > > 
> > > Move HFI1 IOCTL declarations to rdma_user_ioctl.h file.
> > 
> > I have not tried with the patch but I'm 99% sure this will break
> > the PSM2
> > library build which includes hfi1_user.h.
> > 
> > This is one of those things I have pondered in the past.  Most of
> > the rdma
> > libraries don't actually use these definitions directly.  PSM2
> > does.
> 
> I'm not so convinced about it.
> "#include <rdma/rdma_user_ioctl.h>" was added to hfi1_user.h to share
> all definitions. PSM2 library will see it.

On face value that should be true. I would want to run it by the PSM
folks and test it out though to be sure.



> > I'm not sure what other libraries do.
> > 
> > In the final patch of this series you admit that the name changes
> > in that patch
> > will break userspace which uses the defines directly.  Can we,
> > should we, do
> > that?
> 
> I'm talking about __NUM() macros.
> 
> Do you really use __NUM(ASSIGN_CTXT) in user application? Why did you
> do
> it? You supposed to use HFI1_IOCTL_ASSIGN_CTXT instead.

Yeah I think we can get rid of the __NUM() macro. It should not
strictly be needed by user space. User should be using the thing that
__NUM() was used in, the IOCTL number.

-DennyN�����r��y���b�X��ǧv�^�)޺{.n�+����{��ٚ�{ay�ʇڙ�,j��f���h�����/oSc��ڳ9�u�����&jw��(�階�ݢj"���m�����z�ޖ���f���h���~�m�