| Message ID | 1471355123-6227-5-git-send-email-leon@kernel.org (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
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 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? To be completely pedantic I think we need to maintain the old names at least for some time. Perhaps they are best put in an rdma_user_compat_ioctl.h which is included in the final rdma_user_ioctl.h? For this patch I think it would be sufficient to include rdma_user_ioctl.h in hfi1_user.h but I would have to double check that. Ira > > Signed-off-by: Matan Barak <matanb@mellanox.com> > Signed-off-by: Haggai Eran <haggaie@mellanox.com> > Signed-off-by: Leon Romanovsky <leonro@mellanox.com> > --- > include/uapi/rdma/hfi/hfi1_user.h | 55 ------------------------------------- > include/uapi/rdma/rdma_user_ioctl.h | 54 ++++++++++++++++++++++++++++++++++++ > 2 files changed, 54 insertions(+), 55 deletions(-) > > diff --git a/include/uapi/rdma/hfi/hfi1_user.h b/include/uapi/rdma/hfi/hfi1_user.h > index 8aa3867..8807f06 100644 > --- a/include/uapi/rdma/hfi/hfi1_user.h > +++ b/include/uapi/rdma/hfi/hfi1_user.h > @@ -58,7 +58,6 @@ > > #include <linux/types.h> > #include <rdma/rdma_user_ioctl.h> > -#include <rdma/hfi/hfi1_ioctl.h> > > /* > * This version number is given to the driver by the user code during > @@ -114,60 +113,6 @@ > #define HFI1_RCVHDR_ENTSIZE_16 (1UL << 1) > #define HFI1_RCVDHR_ENTSIZE_32 (1UL << 2) > > -/* User commands. */ > -#define HFI1_CMD_ASSIGN_CTXT 1 /* allocate HFI and context */ > -#define HFI1_CMD_CTXT_INFO 2 /* find out what resources we got */ > -#define HFI1_CMD_USER_INFO 3 /* set up userspace */ > -#define HFI1_CMD_TID_UPDATE 4 /* update expected TID entries */ > -#define HFI1_CMD_TID_FREE 5 /* free expected TID entries */ > -#define HFI1_CMD_CREDIT_UPD 6 /* force an update of PIO credit */ > - > -#define HFI1_CMD_RECV_CTRL 8 /* control receipt of packets */ > -#define HFI1_CMD_POLL_TYPE 9 /* set the kind of polling we want */ > -#define HFI1_CMD_ACK_EVENT 10 /* ack & clear user status bits */ > -#define HFI1_CMD_SET_PKEY 11 /* set context's pkey */ > -#define HFI1_CMD_CTXT_RESET 12 /* reset context's HW send context */ > -#define HFI1_CMD_TID_INVAL_READ 13 /* read TID cache invalidations */ > -#define HFI1_CMD_GET_VERS 14 /* get the version of the user cdev */ > - > -/* > - * User IOCTLs can not go above 128 if they do then see common.h and change the > - * base for the snoop ioctl > - */ > - > -/* > - * Make the ioctls occupy the last 0xf0-0xff portion of the IB range > - */ > -#define __NUM(cmd) (HFI1_CMD_##cmd + 0xe0) > - > -struct hfi1_cmd; > -#define HFI1_IOCTL_ASSIGN_CTXT \ > - _IOWR(IB_IOCTL_MAGIC, __NUM(ASSIGN_CTXT), struct hfi1_user_info) > -#define HFI1_IOCTL_CTXT_INFO \ > - _IOW(IB_IOCTL_MAGIC, __NUM(CTXT_INFO), struct hfi1_ctxt_info) > -#define HFI1_IOCTL_USER_INFO \ > - _IOW(IB_IOCTL_MAGIC, __NUM(USER_INFO), struct hfi1_base_info) > -#define HFI1_IOCTL_TID_UPDATE \ > - _IOWR(IB_IOCTL_MAGIC, __NUM(TID_UPDATE), struct hfi1_tid_info) > -#define HFI1_IOCTL_TID_FREE \ > - _IOWR(IB_IOCTL_MAGIC, __NUM(TID_FREE), struct hfi1_tid_info) > -#define HFI1_IOCTL_CREDIT_UPD \ > - _IO(IB_IOCTL_MAGIC, __NUM(CREDIT_UPD)) > -#define HFI1_IOCTL_RECV_CTRL \ > - _IOW(IB_IOCTL_MAGIC, __NUM(RECV_CTRL), int) > -#define HFI1_IOCTL_POLL_TYPE \ > - _IOW(IB_IOCTL_MAGIC, __NUM(POLL_TYPE), int) > -#define HFI1_IOCTL_ACK_EVENT \ > - _IOW(IB_IOCTL_MAGIC, __NUM(ACK_EVENT), unsigned long) > -#define HFI1_IOCTL_SET_PKEY \ > - _IOW(IB_IOCTL_MAGIC, __NUM(SET_PKEY), __u16) > -#define HFI1_IOCTL_CTXT_RESET \ > - _IO(IB_IOCTL_MAGIC, __NUM(CTXT_RESET)) > -#define HFI1_IOCTL_TID_INVAL_READ \ > - _IOWR(IB_IOCTL_MAGIC, __NUM(TID_INVAL_READ), struct hfi1_tid_info) > -#define HFI1_IOCTL_GET_VERS \ > - _IOR(IB_IOCTL_MAGIC, __NUM(GET_VERS), int) > - > #define _HFI1_EVENT_FROZEN_BIT 0 > #define _HFI1_EVENT_LINKDOWN_BIT 1 > #define _HFI1_EVENT_LID_CHANGE_BIT 2 > diff --git a/include/uapi/rdma/rdma_user_ioctl.h b/include/uapi/rdma/rdma_user_ioctl.h > index 820bf34..e9a69f0 100644 > --- a/include/uapi/rdma/rdma_user_ioctl.h > +++ b/include/uapi/rdma/rdma_user_ioctl.h > @@ -36,6 +36,7 @@ > #include <linux/types.h> > #include <linux/ioctl.h> > #include <rdma/ib_user_mad.h> > +#include <rdma/hfi/hfi1_ioctl.h> > > /* Documentation/ioctl/ioctl-number.txt */ > #define RDMA_IOCTL_MAGIC 0x1b > @@ -51,4 +52,57 @@ > #define IB_USER_MAD_REGISTER_AGENT2 _IOWR(IB_IOCTL_MAGIC, 4, \ > struct ib_user_mad_reg_req2) > > +/* User commands. */ > +#define HFI1_CMD_ASSIGN_CTXT 1 /* allocate HFI and context */ > +#define HFI1_CMD_CTXT_INFO 2 /* find out what resources we got */ > +#define HFI1_CMD_USER_INFO 3 /* set up userspace */ > +#define HFI1_CMD_TID_UPDATE 4 /* update expected TID entries */ > +#define HFI1_CMD_TID_FREE 5 /* free expected TID entries */ > +#define HFI1_CMD_CREDIT_UPD 6 /* force an update of PIO credit */ > + > +#define HFI1_CMD_RECV_CTRL 8 /* control receipt of packets */ > +#define HFI1_CMD_POLL_TYPE 9 /* set the kind of polling we want */ > +#define HFI1_CMD_ACK_EVENT 10 /* ack & clear user status bits */ > +#define HFI1_CMD_SET_PKEY 11 /* set context's pkey */ > +#define HFI1_CMD_CTXT_RESET 12 /* reset context's HW send context */ > +#define HFI1_CMD_TID_INVAL_READ 13 /* read TID cache invalidations */ > +#define HFI1_CMD_GET_VERS 14 /* get the version of the user cdev */ > + > +/* > + * User IOCTLs can not go above 128 if they do then see common.h and change the > + * base for the snoop ioctl > + */ > + > +/* > + * Make the ioctls occupy the last 0xf0-0xff portion of the IB range > + */ > +#define __NUM(cmd) (HFI1_CMD_##cmd + 0xe0) > + > +#define HFI1_IOCTL_ASSIGN_CTXT \ > + _IOWR(IB_IOCTL_MAGIC, __NUM(ASSIGN_CTXT), struct hfi1_user_info) > +#define HFI1_IOCTL_CTXT_INFO \ > + _IOW(IB_IOCTL_MAGIC, __NUM(CTXT_INFO), struct hfi1_ctxt_info) > +#define HFI1_IOCTL_USER_INFO \ > + _IOW(IB_IOCTL_MAGIC, __NUM(USER_INFO), struct hfi1_base_info) > +#define HFI1_IOCTL_TID_UPDATE \ > + _IOWR(IB_IOCTL_MAGIC, __NUM(TID_UPDATE), struct hfi1_tid_info) > +#define HFI1_IOCTL_TID_FREE \ > + _IOWR(IB_IOCTL_MAGIC, __NUM(TID_FREE), struct hfi1_tid_info) > +#define HFI1_IOCTL_CREDIT_UPD \ > + _IO(IB_IOCTL_MAGIC, __NUM(CREDIT_UPD)) > +#define HFI1_IOCTL_RECV_CTRL \ > + _IOW(IB_IOCTL_MAGIC, __NUM(RECV_CTRL), int) > +#define HFI1_IOCTL_POLL_TYPE \ > + _IOW(IB_IOCTL_MAGIC, __NUM(POLL_TYPE), int) > +#define HFI1_IOCTL_ACK_EVENT \ > + _IOW(IB_IOCTL_MAGIC, __NUM(ACK_EVENT), unsigned long) > +#define HFI1_IOCTL_SET_PKEY \ > + _IOW(IB_IOCTL_MAGIC, __NUM(SET_PKEY), __u16) > +#define HFI1_IOCTL_CTXT_RESET \ > + _IO(IB_IOCTL_MAGIC, __NUM(CTXT_RESET)) > +#define HFI1_IOCTL_TID_INVAL_READ \ > + _IOWR(IB_IOCTL_MAGIC, __NUM(TID_INVAL_READ), struct hfi1_tid_info) > +#define HFI1_IOCTL_GET_VERS \ > + _IOR(IB_IOCTL_MAGIC, __NUM(GET_VERS), int) > + > #endif /* RDMA_USER_IOCTL_H */ > -- > 2.7.4 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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. > > 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. > > To be completely pedantic I think we need to maintain the old names at least > for some time. Perhaps they are best put in an rdma_user_compat_ioctl.h which > is included in the final rdma_user_ioctl.h? Except __NUM() macro, there is no change in names. > > For this patch I think it would be sufficient to include rdma_user_ioctl.h in > hfi1_user.h but I would have to double check that. It is already done in patches before. > > Ira > > > > > Signed-off-by: Matan Barak <matanb@mellanox.com> > > Signed-off-by: Haggai Eran <haggaie@mellanox.com> > > Signed-off-by: Leon Romanovsky <leonro@mellanox.com> > > --- > > include/uapi/rdma/hfi/hfi1_user.h | 55 ------------------------------------- > > include/uapi/rdma/rdma_user_ioctl.h | 54 ++++++++++++++++++++++++++++++++++++ > > 2 files changed, 54 insertions(+), 55 deletions(-) > > > > diff --git a/include/uapi/rdma/hfi/hfi1_user.h b/include/uapi/rdma/hfi/hfi1_user.h > > index 8aa3867..8807f06 100644 > > --- a/include/uapi/rdma/hfi/hfi1_user.h > > +++ b/include/uapi/rdma/hfi/hfi1_user.h > > @@ -58,7 +58,6 @@ > > > > #include <linux/types.h> > > #include <rdma/rdma_user_ioctl.h> > > -#include <rdma/hfi/hfi1_ioctl.h> > > > > /* > > * This version number is given to the driver by the user code during > > @@ -114,60 +113,6 @@ > > #define HFI1_RCVHDR_ENTSIZE_16 (1UL << 1) > > #define HFI1_RCVDHR_ENTSIZE_32 (1UL << 2) > > > > -/* User commands. */ > > -#define HFI1_CMD_ASSIGN_CTXT 1 /* allocate HFI and context */ > > -#define HFI1_CMD_CTXT_INFO 2 /* find out what resources we got */ > > -#define HFI1_CMD_USER_INFO 3 /* set up userspace */ > > -#define HFI1_CMD_TID_UPDATE 4 /* update expected TID entries */ > > -#define HFI1_CMD_TID_FREE 5 /* free expected TID entries */ > > -#define HFI1_CMD_CREDIT_UPD 6 /* force an update of PIO credit */ > > - > > -#define HFI1_CMD_RECV_CTRL 8 /* control receipt of packets */ > > -#define HFI1_CMD_POLL_TYPE 9 /* set the kind of polling we want */ > > -#define HFI1_CMD_ACK_EVENT 10 /* ack & clear user status bits */ > > -#define HFI1_CMD_SET_PKEY 11 /* set context's pkey */ > > -#define HFI1_CMD_CTXT_RESET 12 /* reset context's HW send context */ > > -#define HFI1_CMD_TID_INVAL_READ 13 /* read TID cache invalidations */ > > -#define HFI1_CMD_GET_VERS 14 /* get the version of the user cdev */ > > - > > -/* > > - * User IOCTLs can not go above 128 if they do then see common.h and change the > > - * base for the snoop ioctl > > - */ > > - > > -/* > > - * Make the ioctls occupy the last 0xf0-0xff portion of the IB range > > - */ > > -#define __NUM(cmd) (HFI1_CMD_##cmd + 0xe0) > > - > > -struct hfi1_cmd; > > -#define HFI1_IOCTL_ASSIGN_CTXT \ > > - _IOWR(IB_IOCTL_MAGIC, __NUM(ASSIGN_CTXT), struct hfi1_user_info) > > -#define HFI1_IOCTL_CTXT_INFO \ > > - _IOW(IB_IOCTL_MAGIC, __NUM(CTXT_INFO), struct hfi1_ctxt_info) > > -#define HFI1_IOCTL_USER_INFO \ > > - _IOW(IB_IOCTL_MAGIC, __NUM(USER_INFO), struct hfi1_base_info) > > -#define HFI1_IOCTL_TID_UPDATE \ > > - _IOWR(IB_IOCTL_MAGIC, __NUM(TID_UPDATE), struct hfi1_tid_info) > > -#define HFI1_IOCTL_TID_FREE \ > > - _IOWR(IB_IOCTL_MAGIC, __NUM(TID_FREE), struct hfi1_tid_info) > > -#define HFI1_IOCTL_CREDIT_UPD \ > > - _IO(IB_IOCTL_MAGIC, __NUM(CREDIT_UPD)) > > -#define HFI1_IOCTL_RECV_CTRL \ > > - _IOW(IB_IOCTL_MAGIC, __NUM(RECV_CTRL), int) > > -#define HFI1_IOCTL_POLL_TYPE \ > > - _IOW(IB_IOCTL_MAGIC, __NUM(POLL_TYPE), int) > > -#define HFI1_IOCTL_ACK_EVENT \ > > - _IOW(IB_IOCTL_MAGIC, __NUM(ACK_EVENT), unsigned long) > > -#define HFI1_IOCTL_SET_PKEY \ > > - _IOW(IB_IOCTL_MAGIC, __NUM(SET_PKEY), __u16) > > -#define HFI1_IOCTL_CTXT_RESET \ > > - _IO(IB_IOCTL_MAGIC, __NUM(CTXT_RESET)) > > -#define HFI1_IOCTL_TID_INVAL_READ \ > > - _IOWR(IB_IOCTL_MAGIC, __NUM(TID_INVAL_READ), struct hfi1_tid_info) > > -#define HFI1_IOCTL_GET_VERS \ > > - _IOR(IB_IOCTL_MAGIC, __NUM(GET_VERS), int) > > - > > #define _HFI1_EVENT_FROZEN_BIT 0 > > #define _HFI1_EVENT_LINKDOWN_BIT 1 > > #define _HFI1_EVENT_LID_CHANGE_BIT 2 > > diff --git a/include/uapi/rdma/rdma_user_ioctl.h b/include/uapi/rdma/rdma_user_ioctl.h > > index 820bf34..e9a69f0 100644 > > --- a/include/uapi/rdma/rdma_user_ioctl.h > > +++ b/include/uapi/rdma/rdma_user_ioctl.h > > @@ -36,6 +36,7 @@ > > #include <linux/types.h> > > #include <linux/ioctl.h> > > #include <rdma/ib_user_mad.h> > > +#include <rdma/hfi/hfi1_ioctl.h> > > > > /* Documentation/ioctl/ioctl-number.txt */ > > #define RDMA_IOCTL_MAGIC 0x1b > > @@ -51,4 +52,57 @@ > > #define IB_USER_MAD_REGISTER_AGENT2 _IOWR(IB_IOCTL_MAGIC, 4, \ > > struct ib_user_mad_reg_req2) > > > > +/* User commands. */ > > +#define HFI1_CMD_ASSIGN_CTXT 1 /* allocate HFI and context */ > > +#define HFI1_CMD_CTXT_INFO 2 /* find out what resources we got */ > > +#define HFI1_CMD_USER_INFO 3 /* set up userspace */ > > +#define HFI1_CMD_TID_UPDATE 4 /* update expected TID entries */ > > +#define HFI1_CMD_TID_FREE 5 /* free expected TID entries */ > > +#define HFI1_CMD_CREDIT_UPD 6 /* force an update of PIO credit */ > > + > > +#define HFI1_CMD_RECV_CTRL 8 /* control receipt of packets */ > > +#define HFI1_CMD_POLL_TYPE 9 /* set the kind of polling we want */ > > +#define HFI1_CMD_ACK_EVENT 10 /* ack & clear user status bits */ > > +#define HFI1_CMD_SET_PKEY 11 /* set context's pkey */ > > +#define HFI1_CMD_CTXT_RESET 12 /* reset context's HW send context */ > > +#define HFI1_CMD_TID_INVAL_READ 13 /* read TID cache invalidations */ > > +#define HFI1_CMD_GET_VERS 14 /* get the version of the user cdev */ > > + > > +/* > > + * User IOCTLs can not go above 128 if they do then see common.h and change the > > + * base for the snoop ioctl > > + */ > > + > > +/* > > + * Make the ioctls occupy the last 0xf0-0xff portion of the IB range > > + */ > > +#define __NUM(cmd) (HFI1_CMD_##cmd + 0xe0) > > + > > +#define HFI1_IOCTL_ASSIGN_CTXT \ > > + _IOWR(IB_IOCTL_MAGIC, __NUM(ASSIGN_CTXT), struct hfi1_user_info) > > +#define HFI1_IOCTL_CTXT_INFO \ > > + _IOW(IB_IOCTL_MAGIC, __NUM(CTXT_INFO), struct hfi1_ctxt_info) > > +#define HFI1_IOCTL_USER_INFO \ > > + _IOW(IB_IOCTL_MAGIC, __NUM(USER_INFO), struct hfi1_base_info) > > +#define HFI1_IOCTL_TID_UPDATE \ > > + _IOWR(IB_IOCTL_MAGIC, __NUM(TID_UPDATE), struct hfi1_tid_info) > > +#define HFI1_IOCTL_TID_FREE \ > > + _IOWR(IB_IOCTL_MAGIC, __NUM(TID_FREE), struct hfi1_tid_info) > > +#define HFI1_IOCTL_CREDIT_UPD \ > > + _IO(IB_IOCTL_MAGIC, __NUM(CREDIT_UPD)) > > +#define HFI1_IOCTL_RECV_CTRL \ > > + _IOW(IB_IOCTL_MAGIC, __NUM(RECV_CTRL), int) > > +#define HFI1_IOCTL_POLL_TYPE \ > > + _IOW(IB_IOCTL_MAGIC, __NUM(POLL_TYPE), int) > > +#define HFI1_IOCTL_ACK_EVENT \ > > + _IOW(IB_IOCTL_MAGIC, __NUM(ACK_EVENT), unsigned long) > > +#define HFI1_IOCTL_SET_PKEY \ > > + _IOW(IB_IOCTL_MAGIC, __NUM(SET_PKEY), __u16) > > +#define HFI1_IOCTL_CTXT_RESET \ > > + _IO(IB_IOCTL_MAGIC, __NUM(CTXT_RESET)) > > +#define HFI1_IOCTL_TID_INVAL_READ \ > > + _IOWR(IB_IOCTL_MAGIC, __NUM(TID_INVAL_READ), struct hfi1_tid_info) > > +#define HFI1_IOCTL_GET_VERS \ > > + _IOR(IB_IOCTL_MAGIC, __NUM(GET_VERS), int) > > + > > #endif /* RDMA_USER_IOCTL_H */ > > -- > > 2.7.4 > > > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > > the body of a message to majordomo@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html
T24gV2VkLCAyMDE2LTA4LTE3IGF0IDA5OjU1ICswMzAwLCBMZW9uIFJvbWFub3Zza3kgd3JvdGU6 DQo+IE9uIFdlZCwgQXVnIDE3LCAyMDE2IGF0IDAyOjE2OjMxQU0gLTA0MDAsIGlyYS53ZWlueSB3 cm90ZToNCj4gPiBPbiBUdWUsIEF1ZyAxNiwgMjAxNiBhdCAwNDo0NToyMVBNICswMzAwLCBMZW9u IFJvbWFub3Zza3kgd3JvdGU6DQo+ID4gPiBGcm9tOiBMZW9uIFJvbWFub3Zza3kgPGxlb25yb0Bt ZWxsYW5veC5jb20+DQo+ID4gPiANCj4gPiA+IE1vdmUgSEZJMSBJT0NUTCBkZWNsYXJhdGlvbnMg dG8gcmRtYV91c2VyX2lvY3RsLmggZmlsZS4NCj4gPiANCj4gPiBJIGhhdmUgbm90IHRyaWVkIHdp dGggdGhlIHBhdGNoIGJ1dCBJJ20gOTklIHN1cmUgdGhpcyB3aWxsIGJyZWFrDQo+ID4gdGhlIFBT TTINCj4gPiBsaWJyYXJ5IGJ1aWxkIHdoaWNoIGluY2x1ZGVzIGhmaTFfdXNlci5oLg0KPiA+IA0K PiA+IFRoaXMgaXMgb25lIG9mIHRob3NlIHRoaW5ncyBJIGhhdmUgcG9uZGVyZWQgaW4gdGhlIHBh c3QuwqDCoE1vc3Qgb2YNCj4gPiB0aGUgcmRtYQ0KPiA+IGxpYnJhcmllcyBkb24ndCBhY3R1YWxs eSB1c2UgdGhlc2UgZGVmaW5pdGlvbnMgZGlyZWN0bHkuwqDCoFBTTTINCj4gPiBkb2VzLg0KPiAN Cj4gSSdtIG5vdCBzbyBjb252aW5jZWQgYWJvdXQgaXQuDQo+ICIjaW5jbHVkZSA8cmRtYS9yZG1h X3VzZXJfaW9jdGwuaD4iIHdhcyBhZGRlZCB0byBoZmkxX3VzZXIuaCB0byBzaGFyZQ0KPiBhbGwg ZGVmaW5pdGlvbnMuIFBTTTIgbGlicmFyeSB3aWxsIHNlZSBpdC4NCg0KT24gZmFjZSB2YWx1ZSB0 aGF0IHNob3VsZCBiZSB0cnVlLiBJIHdvdWxkIHdhbnQgdG8gcnVuIGl0IGJ5IHRoZSBQU00NCmZv bGtzIGFuZCB0ZXN0IGl0IG91dCB0aG91Z2ggdG8gYmUgc3VyZS4NCg0KDQoNCj4gPiBJJ20gbm90 IHN1cmUgd2hhdCBvdGhlciBsaWJyYXJpZXMgZG8uDQo+ID4gDQo+ID4gSW4gdGhlIGZpbmFsIHBh dGNoIG9mIHRoaXMgc2VyaWVzIHlvdSBhZG1pdCB0aGF0IHRoZSBuYW1lIGNoYW5nZXMNCj4gPiBp biB0aGF0IHBhdGNoDQo+ID4gd2lsbCBicmVhayB1c2Vyc3BhY2Ugd2hpY2ggdXNlcyB0aGUgZGVm aW5lcyBkaXJlY3RseS7CoMKgQ2FuIHdlLA0KPiA+IHNob3VsZCB3ZSwgZG8NCj4gPiB0aGF0Pw0K PiANCj4gSSdtIHRhbGtpbmcgYWJvdXQgX19OVU0oKSBtYWNyb3MuDQo+IA0KPiBEbyB5b3UgcmVh bGx5IHVzZSBfX05VTShBU1NJR05fQ1RYVCkgaW4gdXNlciBhcHBsaWNhdGlvbj8gV2h5IGRpZCB5 b3UNCj4gZG8NCj4gaXQ/IFlvdSBzdXBwb3NlZCB0byB1c2UgSEZJMV9JT0NUTF9BU1NJR05fQ1RY VCBpbnN0ZWFkLg0KDQpZZWFoIEkgdGhpbmsgd2UgY2FuIGdldCByaWQgb2YgdGhlIF9fTlVNKCkg bWFjcm8uIEl0IHNob3VsZCBub3QNCnN0cmljdGx5IGJlIG5lZWRlZCBieSB1c2VyIHNwYWNlLiBV c2VyIHNob3VsZCBiZSB1c2luZyB0aGUgdGhpbmcgdGhhdA0KX19OVU0oKSB3YXMgdXNlZCBpbiwg dGhlIElPQ1RMIG51bWJlci4NCg0KLURlbm55 -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Wed, Aug 17, 2016 at 01:35:53PM +0000, Dalessandro, Dennis wrote: > 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 agree, Thanks.
On Wed, Aug 17, 2016 at 09:55:29AM +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. Ok I see it now. Sorry it was late. > > > > > 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. Your commit message says "... and MAD indexes were renamed. It has the potential to break application which use these defines directly." I took that at face value. I've taken a closer look ... I see now that neither the numbers nor the define names changed so ok, my mistake. Ira > > > > > To be completely pedantic I think we need to maintain the old names at least > > for some time. Perhaps they are best put in an rdma_user_compat_ioctl.h which > > is included in the final rdma_user_ioctl.h? > > Except __NUM() macro, there is no change in names. > > > > > For this patch I think it would be sufficient to include rdma_user_ioctl.h in > > hfi1_user.h but I would have to double check that. > > It is already done in patches before. > > > > > Ira > > > > > > > > Signed-off-by: Matan Barak <matanb@mellanox.com> > > > Signed-off-by: Haggai Eran <haggaie@mellanox.com> > > > Signed-off-by: Leon Romanovsky <leonro@mellanox.com> > > > --- > > > include/uapi/rdma/hfi/hfi1_user.h | 55 ------------------------------------- > > > include/uapi/rdma/rdma_user_ioctl.h | 54 ++++++++++++++++++++++++++++++++++++ > > > 2 files changed, 54 insertions(+), 55 deletions(-) > > > > > > diff --git a/include/uapi/rdma/hfi/hfi1_user.h b/include/uapi/rdma/hfi/hfi1_user.h > > > index 8aa3867..8807f06 100644 > > > --- a/include/uapi/rdma/hfi/hfi1_user.h > > > +++ b/include/uapi/rdma/hfi/hfi1_user.h > > > @@ -58,7 +58,6 @@ > > > > > > #include <linux/types.h> > > > #include <rdma/rdma_user_ioctl.h> > > > -#include <rdma/hfi/hfi1_ioctl.h> > > > > > > /* > > > * This version number is given to the driver by the user code during > > > @@ -114,60 +113,6 @@ > > > #define HFI1_RCVHDR_ENTSIZE_16 (1UL << 1) > > > #define HFI1_RCVDHR_ENTSIZE_32 (1UL << 2) > > > > > > -/* User commands. */ > > > -#define HFI1_CMD_ASSIGN_CTXT 1 /* allocate HFI and context */ > > > -#define HFI1_CMD_CTXT_INFO 2 /* find out what resources we got */ > > > -#define HFI1_CMD_USER_INFO 3 /* set up userspace */ > > > -#define HFI1_CMD_TID_UPDATE 4 /* update expected TID entries */ > > > -#define HFI1_CMD_TID_FREE 5 /* free expected TID entries */ > > > -#define HFI1_CMD_CREDIT_UPD 6 /* force an update of PIO credit */ > > > - > > > -#define HFI1_CMD_RECV_CTRL 8 /* control receipt of packets */ > > > -#define HFI1_CMD_POLL_TYPE 9 /* set the kind of polling we want */ > > > -#define HFI1_CMD_ACK_EVENT 10 /* ack & clear user status bits */ > > > -#define HFI1_CMD_SET_PKEY 11 /* set context's pkey */ > > > -#define HFI1_CMD_CTXT_RESET 12 /* reset context's HW send context */ > > > -#define HFI1_CMD_TID_INVAL_READ 13 /* read TID cache invalidations */ > > > -#define HFI1_CMD_GET_VERS 14 /* get the version of the user cdev */ > > > - > > > -/* > > > - * User IOCTLs can not go above 128 if they do then see common.h and change the > > > - * base for the snoop ioctl > > > - */ > > > - > > > -/* > > > - * Make the ioctls occupy the last 0xf0-0xff portion of the IB range > > > - */ > > > -#define __NUM(cmd) (HFI1_CMD_##cmd + 0xe0) > > > - > > > -struct hfi1_cmd; > > > -#define HFI1_IOCTL_ASSIGN_CTXT \ > > > - _IOWR(IB_IOCTL_MAGIC, __NUM(ASSIGN_CTXT), struct hfi1_user_info) > > > -#define HFI1_IOCTL_CTXT_INFO \ > > > - _IOW(IB_IOCTL_MAGIC, __NUM(CTXT_INFO), struct hfi1_ctxt_info) > > > -#define HFI1_IOCTL_USER_INFO \ > > > - _IOW(IB_IOCTL_MAGIC, __NUM(USER_INFO), struct hfi1_base_info) > > > -#define HFI1_IOCTL_TID_UPDATE \ > > > - _IOWR(IB_IOCTL_MAGIC, __NUM(TID_UPDATE), struct hfi1_tid_info) > > > -#define HFI1_IOCTL_TID_FREE \ > > > - _IOWR(IB_IOCTL_MAGIC, __NUM(TID_FREE), struct hfi1_tid_info) > > > -#define HFI1_IOCTL_CREDIT_UPD \ > > > - _IO(IB_IOCTL_MAGIC, __NUM(CREDIT_UPD)) > > > -#define HFI1_IOCTL_RECV_CTRL \ > > > - _IOW(IB_IOCTL_MAGIC, __NUM(RECV_CTRL), int) > > > -#define HFI1_IOCTL_POLL_TYPE \ > > > - _IOW(IB_IOCTL_MAGIC, __NUM(POLL_TYPE), int) > > > -#define HFI1_IOCTL_ACK_EVENT \ > > > - _IOW(IB_IOCTL_MAGIC, __NUM(ACK_EVENT), unsigned long) > > > -#define HFI1_IOCTL_SET_PKEY \ > > > - _IOW(IB_IOCTL_MAGIC, __NUM(SET_PKEY), __u16) > > > -#define HFI1_IOCTL_CTXT_RESET \ > > > - _IO(IB_IOCTL_MAGIC, __NUM(CTXT_RESET)) > > > -#define HFI1_IOCTL_TID_INVAL_READ \ > > > - _IOWR(IB_IOCTL_MAGIC, __NUM(TID_INVAL_READ), struct hfi1_tid_info) > > > -#define HFI1_IOCTL_GET_VERS \ > > > - _IOR(IB_IOCTL_MAGIC, __NUM(GET_VERS), int) > > > - > > > #define _HFI1_EVENT_FROZEN_BIT 0 > > > #define _HFI1_EVENT_LINKDOWN_BIT 1 > > > #define _HFI1_EVENT_LID_CHANGE_BIT 2 > > > diff --git a/include/uapi/rdma/rdma_user_ioctl.h b/include/uapi/rdma/rdma_user_ioctl.h > > > index 820bf34..e9a69f0 100644 > > > --- a/include/uapi/rdma/rdma_user_ioctl.h > > > +++ b/include/uapi/rdma/rdma_user_ioctl.h > > > @@ -36,6 +36,7 @@ > > > #include <linux/types.h> > > > #include <linux/ioctl.h> > > > #include <rdma/ib_user_mad.h> > > > +#include <rdma/hfi/hfi1_ioctl.h> > > > > > > /* Documentation/ioctl/ioctl-number.txt */ > > > #define RDMA_IOCTL_MAGIC 0x1b > > > @@ -51,4 +52,57 @@ > > > #define IB_USER_MAD_REGISTER_AGENT2 _IOWR(IB_IOCTL_MAGIC, 4, \ > > > struct ib_user_mad_reg_req2) > > > > > > +/* User commands. */ > > > +#define HFI1_CMD_ASSIGN_CTXT 1 /* allocate HFI and context */ > > > +#define HFI1_CMD_CTXT_INFO 2 /* find out what resources we got */ > > > +#define HFI1_CMD_USER_INFO 3 /* set up userspace */ > > > +#define HFI1_CMD_TID_UPDATE 4 /* update expected TID entries */ > > > +#define HFI1_CMD_TID_FREE 5 /* free expected TID entries */ > > > +#define HFI1_CMD_CREDIT_UPD 6 /* force an update of PIO credit */ > > > + > > > +#define HFI1_CMD_RECV_CTRL 8 /* control receipt of packets */ > > > +#define HFI1_CMD_POLL_TYPE 9 /* set the kind of polling we want */ > > > +#define HFI1_CMD_ACK_EVENT 10 /* ack & clear user status bits */ > > > +#define HFI1_CMD_SET_PKEY 11 /* set context's pkey */ > > > +#define HFI1_CMD_CTXT_RESET 12 /* reset context's HW send context */ > > > +#define HFI1_CMD_TID_INVAL_READ 13 /* read TID cache invalidations */ > > > +#define HFI1_CMD_GET_VERS 14 /* get the version of the user cdev */ > > > + > > > +/* > > > + * User IOCTLs can not go above 128 if they do then see common.h and change the > > > + * base for the snoop ioctl > > > + */ > > > + > > > +/* > > > + * Make the ioctls occupy the last 0xf0-0xff portion of the IB range > > > + */ > > > +#define __NUM(cmd) (HFI1_CMD_##cmd + 0xe0) > > > + > > > +#define HFI1_IOCTL_ASSIGN_CTXT \ > > > + _IOWR(IB_IOCTL_MAGIC, __NUM(ASSIGN_CTXT), struct hfi1_user_info) > > > +#define HFI1_IOCTL_CTXT_INFO \ > > > + _IOW(IB_IOCTL_MAGIC, __NUM(CTXT_INFO), struct hfi1_ctxt_info) > > > +#define HFI1_IOCTL_USER_INFO \ > > > + _IOW(IB_IOCTL_MAGIC, __NUM(USER_INFO), struct hfi1_base_info) > > > +#define HFI1_IOCTL_TID_UPDATE \ > > > + _IOWR(IB_IOCTL_MAGIC, __NUM(TID_UPDATE), struct hfi1_tid_info) > > > +#define HFI1_IOCTL_TID_FREE \ > > > + _IOWR(IB_IOCTL_MAGIC, __NUM(TID_FREE), struct hfi1_tid_info) > > > +#define HFI1_IOCTL_CREDIT_UPD \ > > > + _IO(IB_IOCTL_MAGIC, __NUM(CREDIT_UPD)) > > > +#define HFI1_IOCTL_RECV_CTRL \ > > > + _IOW(IB_IOCTL_MAGIC, __NUM(RECV_CTRL), int) > > > +#define HFI1_IOCTL_POLL_TYPE \ > > > + _IOW(IB_IOCTL_MAGIC, __NUM(POLL_TYPE), int) > > > +#define HFI1_IOCTL_ACK_EVENT \ > > > + _IOW(IB_IOCTL_MAGIC, __NUM(ACK_EVENT), unsigned long) > > > +#define HFI1_IOCTL_SET_PKEY \ > > > + _IOW(IB_IOCTL_MAGIC, __NUM(SET_PKEY), __u16) > > > +#define HFI1_IOCTL_CTXT_RESET \ > > > + _IO(IB_IOCTL_MAGIC, __NUM(CTXT_RESET)) > > > +#define HFI1_IOCTL_TID_INVAL_READ \ > > > + _IOWR(IB_IOCTL_MAGIC, __NUM(TID_INVAL_READ), struct hfi1_tid_info) > > > +#define HFI1_IOCTL_GET_VERS \ > > > + _IOR(IB_IOCTL_MAGIC, __NUM(GET_VERS), int) > > > + > > > #endif /* RDMA_USER_IOCTL_H */ > > > -- > > > 2.7.4 > > > > > > -- > > > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > > > the body of a message to majordomo@vger.kernel.org > > > More majordomo info at http://vger.kernel.org/majordomo-info.html > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > > the body of a message to majordomo@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Wed, Aug 17, 2016 at 02:10:51PM -0400, ira.weiny wrote: > On Wed, Aug 17, 2016 at 09:55:29AM +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. > > Ok I see it now. Sorry it was late. > > > > > > > > > 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. > > Your commit message says "... and MAD indexes were renamed. It has the > potential to break application which use these defines directly." > > I took that at face value. I've taken a closer look ... I see now that neither > the numbers nor the define names changed so ok, my mistake. Sorry for inconvenience, I'll update commit message, so it will better reflect reality. Thanks
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 sure what other libraries do. They should all be fixed to use the new uapi headers instead of cloning them privately. If we go ahead with the rdma-plumbing I might even do it.. > 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? This happens from time to time, userspace should gain a build-time test to deal with it, if it is a big enough deal, IMHO. We don't guarentee 100% source compatibility.. Jason -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/include/uapi/rdma/hfi/hfi1_user.h b/include/uapi/rdma/hfi/hfi1_user.h index 8aa3867..8807f06 100644 --- a/include/uapi/rdma/hfi/hfi1_user.h +++ b/include/uapi/rdma/hfi/hfi1_user.h @@ -58,7 +58,6 @@ #include <linux/types.h> #include <rdma/rdma_user_ioctl.h> -#include <rdma/hfi/hfi1_ioctl.h> /* * This version number is given to the driver by the user code during @@ -114,60 +113,6 @@ #define HFI1_RCVHDR_ENTSIZE_16 (1UL << 1) #define HFI1_RCVDHR_ENTSIZE_32 (1UL << 2) -/* User commands. */ -#define HFI1_CMD_ASSIGN_CTXT 1 /* allocate HFI and context */ -#define HFI1_CMD_CTXT_INFO 2 /* find out what resources we got */ -#define HFI1_CMD_USER_INFO 3 /* set up userspace */ -#define HFI1_CMD_TID_UPDATE 4 /* update expected TID entries */ -#define HFI1_CMD_TID_FREE 5 /* free expected TID entries */ -#define HFI1_CMD_CREDIT_UPD 6 /* force an update of PIO credit */ - -#define HFI1_CMD_RECV_CTRL 8 /* control receipt of packets */ -#define HFI1_CMD_POLL_TYPE 9 /* set the kind of polling we want */ -#define HFI1_CMD_ACK_EVENT 10 /* ack & clear user status bits */ -#define HFI1_CMD_SET_PKEY 11 /* set context's pkey */ -#define HFI1_CMD_CTXT_RESET 12 /* reset context's HW send context */ -#define HFI1_CMD_TID_INVAL_READ 13 /* read TID cache invalidations */ -#define HFI1_CMD_GET_VERS 14 /* get the version of the user cdev */ - -/* - * User IOCTLs can not go above 128 if they do then see common.h and change the - * base for the snoop ioctl - */ - -/* - * Make the ioctls occupy the last 0xf0-0xff portion of the IB range - */ -#define __NUM(cmd) (HFI1_CMD_##cmd + 0xe0) - -struct hfi1_cmd; -#define HFI1_IOCTL_ASSIGN_CTXT \ - _IOWR(IB_IOCTL_MAGIC, __NUM(ASSIGN_CTXT), struct hfi1_user_info) -#define HFI1_IOCTL_CTXT_INFO \ - _IOW(IB_IOCTL_MAGIC, __NUM(CTXT_INFO), struct hfi1_ctxt_info) -#define HFI1_IOCTL_USER_INFO \ - _IOW(IB_IOCTL_MAGIC, __NUM(USER_INFO), struct hfi1_base_info) -#define HFI1_IOCTL_TID_UPDATE \ - _IOWR(IB_IOCTL_MAGIC, __NUM(TID_UPDATE), struct hfi1_tid_info) -#define HFI1_IOCTL_TID_FREE \ - _IOWR(IB_IOCTL_MAGIC, __NUM(TID_FREE), struct hfi1_tid_info) -#define HFI1_IOCTL_CREDIT_UPD \ - _IO(IB_IOCTL_MAGIC, __NUM(CREDIT_UPD)) -#define HFI1_IOCTL_RECV_CTRL \ - _IOW(IB_IOCTL_MAGIC, __NUM(RECV_CTRL), int) -#define HFI1_IOCTL_POLL_TYPE \ - _IOW(IB_IOCTL_MAGIC, __NUM(POLL_TYPE), int) -#define HFI1_IOCTL_ACK_EVENT \ - _IOW(IB_IOCTL_MAGIC, __NUM(ACK_EVENT), unsigned long) -#define HFI1_IOCTL_SET_PKEY \ - _IOW(IB_IOCTL_MAGIC, __NUM(SET_PKEY), __u16) -#define HFI1_IOCTL_CTXT_RESET \ - _IO(IB_IOCTL_MAGIC, __NUM(CTXT_RESET)) -#define HFI1_IOCTL_TID_INVAL_READ \ - _IOWR(IB_IOCTL_MAGIC, __NUM(TID_INVAL_READ), struct hfi1_tid_info) -#define HFI1_IOCTL_GET_VERS \ - _IOR(IB_IOCTL_MAGIC, __NUM(GET_VERS), int) - #define _HFI1_EVENT_FROZEN_BIT 0 #define _HFI1_EVENT_LINKDOWN_BIT 1 #define _HFI1_EVENT_LID_CHANGE_BIT 2 diff --git a/include/uapi/rdma/rdma_user_ioctl.h b/include/uapi/rdma/rdma_user_ioctl.h index 820bf34..e9a69f0 100644 --- a/include/uapi/rdma/rdma_user_ioctl.h +++ b/include/uapi/rdma/rdma_user_ioctl.h @@ -36,6 +36,7 @@ #include <linux/types.h> #include <linux/ioctl.h> #include <rdma/ib_user_mad.h> +#include <rdma/hfi/hfi1_ioctl.h> /* Documentation/ioctl/ioctl-number.txt */ #define RDMA_IOCTL_MAGIC 0x1b @@ -51,4 +52,57 @@ #define IB_USER_MAD_REGISTER_AGENT2 _IOWR(IB_IOCTL_MAGIC, 4, \ struct ib_user_mad_reg_req2) +/* User commands. */ +#define HFI1_CMD_ASSIGN_CTXT 1 /* allocate HFI and context */ +#define HFI1_CMD_CTXT_INFO 2 /* find out what resources we got */ +#define HFI1_CMD_USER_INFO 3 /* set up userspace */ +#define HFI1_CMD_TID_UPDATE 4 /* update expected TID entries */ +#define HFI1_CMD_TID_FREE 5 /* free expected TID entries */ +#define HFI1_CMD_CREDIT_UPD 6 /* force an update of PIO credit */ + +#define HFI1_CMD_RECV_CTRL 8 /* control receipt of packets */ +#define HFI1_CMD_POLL_TYPE 9 /* set the kind of polling we want */ +#define HFI1_CMD_ACK_EVENT 10 /* ack & clear user status bits */ +#define HFI1_CMD_SET_PKEY 11 /* set context's pkey */ +#define HFI1_CMD_CTXT_RESET 12 /* reset context's HW send context */ +#define HFI1_CMD_TID_INVAL_READ 13 /* read TID cache invalidations */ +#define HFI1_CMD_GET_VERS 14 /* get the version of the user cdev */ + +/* + * User IOCTLs can not go above 128 if they do then see common.h and change the + * base for the snoop ioctl + */ + +/* + * Make the ioctls occupy the last 0xf0-0xff portion of the IB range + */ +#define __NUM(cmd) (HFI1_CMD_##cmd + 0xe0) + +#define HFI1_IOCTL_ASSIGN_CTXT \ + _IOWR(IB_IOCTL_MAGIC, __NUM(ASSIGN_CTXT), struct hfi1_user_info) +#define HFI1_IOCTL_CTXT_INFO \ + _IOW(IB_IOCTL_MAGIC, __NUM(CTXT_INFO), struct hfi1_ctxt_info) +#define HFI1_IOCTL_USER_INFO \ + _IOW(IB_IOCTL_MAGIC, __NUM(USER_INFO), struct hfi1_base_info) +#define HFI1_IOCTL_TID_UPDATE \ + _IOWR(IB_IOCTL_MAGIC, __NUM(TID_UPDATE), struct hfi1_tid_info) +#define HFI1_IOCTL_TID_FREE \ + _IOWR(IB_IOCTL_MAGIC, __NUM(TID_FREE), struct hfi1_tid_info) +#define HFI1_IOCTL_CREDIT_UPD \ + _IO(IB_IOCTL_MAGIC, __NUM(CREDIT_UPD)) +#define HFI1_IOCTL_RECV_CTRL \ + _IOW(IB_IOCTL_MAGIC, __NUM(RECV_CTRL), int) +#define HFI1_IOCTL_POLL_TYPE \ + _IOW(IB_IOCTL_MAGIC, __NUM(POLL_TYPE), int) +#define HFI1_IOCTL_ACK_EVENT \ + _IOW(IB_IOCTL_MAGIC, __NUM(ACK_EVENT), unsigned long) +#define HFI1_IOCTL_SET_PKEY \ + _IOW(IB_IOCTL_MAGIC, __NUM(SET_PKEY), __u16) +#define HFI1_IOCTL_CTXT_RESET \ + _IO(IB_IOCTL_MAGIC, __NUM(CTXT_RESET)) +#define HFI1_IOCTL_TID_INVAL_READ \ + _IOWR(IB_IOCTL_MAGIC, __NUM(TID_INVAL_READ), struct hfi1_tid_info) +#define HFI1_IOCTL_GET_VERS \ + _IOR(IB_IOCTL_MAGIC, __NUM(GET_VERS), int) + #endif /* RDMA_USER_IOCTL_H */