RDMA/hfi1: Avoid redeclaration error
authorLeon Romanovsky <leonro@mellanox.com>
Sun, 4 Sep 2016 11:30:32 +0000 (14:30 +0300)
committerDoug Ledford <dledford@redhat.com>
Tue, 10 Jan 2017 17:41:40 +0000 (12:41 -0500)
Move hfi1 ioctl definitions to a new header which can be included by
both the hfi1 and qib drivers to avoid a duplicate enum definition
as shown in this build error for qib:

  CC [M] drivers/infiniband/hw/qib/qib_sysfs.o
In file included from ./include/uapi/rdma/rdma_user_ioctl.h:39:0,
 from include/uapi/rdma/ib_user_mad.h:38,
 from include/rdma/ib_mad.h:43,
 from include/rdma/ib_pma.h:38,
 from drivers/infiniband/hw/qib/qib_mad.h:37,
 from drivers/infiniband/hw/qib/qib_init.c:49:
./include/uapi/rdma/hfi/hfi1_user.h:370:2: error: redeclaration of
enumerator ‘ur_rcvhdrtail’
  ur_rcvhdrtail = 0,

Move hfi1 structures to separate file to avoid this failure.

The actual move of the ioctl definitions comes in a follow on patch.

Signed-off-by: Matan Barak <matanb@mellanox.com>
Signed-off-by: Haggai Eran <haggaie@mellanox.com>
Signed-off-by: Leon Romanovsky <leonro@mellanox.com>
Reviewed-by: Jason Gunthorpe <jgunthorpe@obsidianresearch.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
include/uapi/rdma/hfi/Kbuild
include/uapi/rdma/hfi/hfi1_ioctl.h [new file with mode: 0644]
include/uapi/rdma/hfi/hfi1_user.h

index ef23c294fc71f96fcc432fcfcbdfc60b2f2673aa..b65b0b3a5f632ae27f7749dc20934e3bb61f4926 100644 (file)
@@ -1,2 +1,3 @@
 # UAPI Header export list
 header-y += hfi1_user.h
+header-y += hfi1_ioctl.h
diff --git a/include/uapi/rdma/hfi/hfi1_ioctl.h b/include/uapi/rdma/hfi/hfi1_ioctl.h
new file mode 100644 (file)
index 0000000..4791cc8
--- /dev/null
@@ -0,0 +1,173 @@
+/*
+ *
+ * This file is provided under a dual BSD/GPLv2 license.  When using or
+ * redistributing this file, you may do so under either license.
+ *
+ * GPL LICENSE SUMMARY
+ *
+ * Copyright(c) 2015 Intel Corporation.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of version 2 of the GNU General Public License as
+ * published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * General Public License for more details.
+ *
+ * BSD LICENSE
+ *
+ * Copyright(c) 2015 Intel Corporation.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ *
+ *  - Redistributions of source code must retain the above copyright
+ *    notice, this list of conditions and the following disclaimer.
+ *  - Redistributions in binary form must reproduce the above copyright
+ *    notice, this list of conditions and the following disclaimer in
+ *    the documentation and/or other materials provided with the
+ *    distribution.
+ *  - Neither the name of Intel Corporation nor the names of its
+ *    contributors may be used to endorse or promote products derived
+ *    from this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+ * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+ * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+ * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+ * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+ * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+ * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ *
+ */
+
+#ifndef _LINUX__HFI1_IOCTL_H
+#define _LINUX__HFI1_IOCTL_H
+#include <linux/types.h>
+
+/*
+ * This structure is passed to the driver to tell it where
+ * user code buffers are, sizes, etc.   The offsets and sizes of the
+ * fields must remain unchanged, for binary compatibility.  It can
+ * be extended, if userversion is changed so user code can tell, if needed
+ */
+struct hfi1_user_info {
+       /*
+        * version of user software, to detect compatibility issues.
+        * Should be set to HFI1_USER_SWVERSION.
+        */
+       __u32 userversion;
+       __u32 pad;
+       /*
+        * If two or more processes wish to share a context, each process
+        * must set the subcontext_cnt and subcontext_id to the same
+        * values.  The only restriction on the subcontext_id is that
+        * it be unique for a given node.
+        */
+       __u16 subctxt_cnt;
+       __u16 subctxt_id;
+       /* 128bit UUID passed in by PSM. */
+       __u8 uuid[16];
+};
+
+struct hfi1_ctxt_info {
+       __u64 runtime_flags;    /* chip/drv runtime flags (HFI1_CAP_*) */
+       __u32 rcvegr_size;      /* size of each eager buffer */
+       __u16 num_active;       /* number of active units */
+       __u16 unit;             /* unit (chip) assigned to caller */
+       __u16 ctxt;             /* ctxt on unit assigned to caller */
+       __u16 subctxt;          /* subctxt on unit assigned to caller */
+       __u16 rcvtids;          /* number of Rcv TIDs for this context */
+       __u16 credits;          /* number of PIO credits for this context */
+       __u16 numa_node;        /* NUMA node of the assigned device */
+       __u16 rec_cpu;          /* cpu # for affinity (0xffff if none) */
+       __u16 send_ctxt;        /* send context in use by this user context */
+       __u16 egrtids;          /* number of RcvArray entries for Eager Rcvs */
+       __u16 rcvhdrq_cnt;      /* number of RcvHdrQ entries */
+       __u16 rcvhdrq_entsize;  /* size (in bytes) for each RcvHdrQ entry */
+       __u16 sdma_ring_size;   /* number of entries in SDMA request ring */
+};
+
+struct hfi1_tid_info {
+       /* virtual address of first page in transfer */
+       __u64 vaddr;
+       /* pointer to tid array. this array is big enough */
+       __u64 tidlist;
+       /* number of tids programmed by this request */
+       __u32 tidcnt;
+       /* length of transfer buffer programmed by this request */
+       __u32 length;
+};
+
+/*
+ * This structure is returned by the driver immediately after
+ * open to get implementation-specific info, and info specific to this
+ * instance.
+ *
+ * This struct must have explicit pad fields where type sizes
+ * may result in different alignments between 32 and 64 bit
+ * programs, since the 64 bit * bit kernel requires the user code
+ * to have matching offsets
+ */
+struct hfi1_base_info {
+       /* version of hardware, for feature checking. */
+       __u32 hw_version;
+       /* version of software, for feature checking. */
+       __u32 sw_version;
+       /* Job key */
+       __u16 jkey;
+       __u16 padding1;
+       /*
+        * The special QP (queue pair) value that identifies PSM
+        * protocol packet from standard IB packets.
+        */
+       __u32 bthqp;
+       /* PIO credit return address, */
+       __u64 sc_credits_addr;
+       /*
+        * Base address of write-only pio buffers for this process.
+        * Each buffer has sendpio_credits*64 bytes.
+        */
+       __u64 pio_bufbase_sop;
+       /*
+        * Base address of write-only pio buffers for this process.
+        * Each buffer has sendpio_credits*64 bytes.
+        */
+       __u64 pio_bufbase;
+       /* address where receive buffer queue is mapped into */
+       __u64 rcvhdr_bufbase;
+       /* base address of Eager receive buffers. */
+       __u64 rcvegr_bufbase;
+       /* base address of SDMA completion ring */
+       __u64 sdma_comp_bufbase;
+       /*
+        * User register base for init code, not to be used directly by
+        * protocol or applications.  Always maps real chip register space.
+        * the register addresses are:
+        * ur_rcvhdrhead, ur_rcvhdrtail, ur_rcvegrhead, ur_rcvegrtail,
+        * ur_rcvtidflow
+        */
+       __u64 user_regbase;
+       /* notification events */
+       __u64 events_bufbase;
+       /* status page */
+       __u64 status_bufbase;
+       /* rcvhdrtail update */
+       __u64 rcvhdrtail_base;
+       /*
+        * shared memory pages for subctxts if ctxt is shared; these cover
+        * all the processes in the group sharing a single context.
+        * all have enough space for the num_subcontexts value on this job.
+        */
+       __u64 subctxt_uregbase;
+       __u64 subctxt_rcvegrbuf;
+       __u64 subctxt_rcvhdrbuf;
+};
+#endif /* _LINIUX__HFI1_IOCTL_H */
index 57d3613f2ff8517e0ad9bef20c5242c1298d3548..220a020d606d5749da69f8dcd91714704a9fa186 100644 (file)
@@ -58,6 +58,7 @@
 
 #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
@@ -211,60 +212,6 @@ struct hfi1_cmd;
 #define HFI1_POLL_TYPE_ANYRCV     0x0
 #define HFI1_POLL_TYPE_URGENT     0x1
 
-/*
- * This structure is passed to the driver to tell it where
- * user code buffers are, sizes, etc.   The offsets and sizes of the
- * fields must remain unchanged, for binary compatibility.  It can
- * be extended, if userversion is changed so user code can tell, if needed
- */
-struct hfi1_user_info {
-       /*
-        * version of user software, to detect compatibility issues.
-        * Should be set to HFI1_USER_SWVERSION.
-        */
-       __u32 userversion;
-       __u32 pad;
-       /*
-        * If two or more processes wish to share a context, each process
-        * must set the subcontext_cnt and subcontext_id to the same
-        * values.  The only restriction on the subcontext_id is that
-        * it be unique for a given node.
-        */
-       __u16 subctxt_cnt;
-       __u16 subctxt_id;
-       /* 128bit UUID passed in by PSM. */
-       __u8 uuid[16];
-};
-
-struct hfi1_ctxt_info {
-       __u64 runtime_flags;    /* chip/drv runtime flags (HFI1_CAP_*) */
-       __u32 rcvegr_size;      /* size of each eager buffer */
-       __u16 num_active;       /* number of active units */
-       __u16 unit;             /* unit (chip) assigned to caller */
-       __u16 ctxt;             /* ctxt on unit assigned to caller */
-       __u16 subctxt;          /* subctxt on unit assigned to caller */
-       __u16 rcvtids;          /* number of Rcv TIDs for this context */
-       __u16 credits;          /* number of PIO credits for this context */
-       __u16 numa_node;        /* NUMA node of the assigned device */
-       __u16 rec_cpu;          /* cpu # for affinity (0xffff if none) */
-       __u16 send_ctxt;        /* send context in use by this user context */
-       __u16 egrtids;          /* number of RcvArray entries for Eager Rcvs */
-       __u16 rcvhdrq_cnt;      /* number of RcvHdrQ entries */
-       __u16 rcvhdrq_entsize;  /* size (in bytes) for each RcvHdrQ entry */
-       __u16 sdma_ring_size;   /* number of entries in SDMA request ring */
-};
-
-struct hfi1_tid_info {
-       /* virtual address of first page in transfer */
-       __u64 vaddr;
-       /* pointer to tid array. this array is big enough */
-       __u64 tidlist;
-       /* number of tids programmed by this request */
-       __u32 tidcnt;
-       /* length of transfer buffer programmed by this request */
-       __u32 length;
-};
-
 enum hfi1_sdma_comp_state {
        FREE = 0,
        QUEUED,
@@ -289,71 +236,6 @@ struct hfi1_status {
        char freezemsg[0];
 };
 
-/*
- * This structure is returned by the driver immediately after
- * open to get implementation-specific info, and info specific to this
- * instance.
- *
- * This struct must have explicit pad fields where type sizes
- * may result in different alignments between 32 and 64 bit
- * programs, since the 64 bit * bit kernel requires the user code
- * to have matching offsets
- */
-struct hfi1_base_info {
-       /* version of hardware, for feature checking. */
-       __u32 hw_version;
-       /* version of software, for feature checking. */
-       __u32 sw_version;
-       /* Job key */
-       __u16 jkey;
-       __u16 padding1;
-       /*
-        * The special QP (queue pair) value that identifies PSM
-        * protocol packet from standard IB packets.
-        */
-       __u32 bthqp;
-       /* PIO credit return address, */
-       __u64 sc_credits_addr;
-       /*
-        * Base address of write-only pio buffers for this process.
-        * Each buffer has sendpio_credits*64 bytes.
-        */
-       __u64 pio_bufbase_sop;
-       /*
-        * Base address of write-only pio buffers for this process.
-        * Each buffer has sendpio_credits*64 bytes.
-        */
-       __u64 pio_bufbase;
-       /* address where receive buffer queue is mapped into */
-       __u64 rcvhdr_bufbase;
-       /* base address of Eager receive buffers. */
-       __u64 rcvegr_bufbase;
-       /* base address of SDMA completion ring */
-       __u64 sdma_comp_bufbase;
-       /*
-        * User register base for init code, not to be used directly by
-        * protocol or applications.  Always maps real chip register space.
-        * the register addresses are:
-        * ur_rcvhdrhead, ur_rcvhdrtail, ur_rcvegrhead, ur_rcvegrtail,
-        * ur_rcvtidflow
-        */
-       __u64 user_regbase;
-       /* notification events */
-       __u64 events_bufbase;
-       /* status page */
-       __u64 status_bufbase;
-       /* rcvhdrtail update */
-       __u64 rcvhdrtail_base;
-       /*
-        * shared memory pages for subctxts if ctxt is shared; these cover
-        * all the processes in the group sharing a single context.
-        * all have enough space for the num_subcontexts value on this job.
-        */
-       __u64 subctxt_uregbase;
-       __u64 subctxt_rcvegrbuf;
-       __u64 subctxt_rcvhdrbuf;
-};
-
 enum sdma_req_opcode {
        EXPECTED = 0,
        EAGER