Re: linux-next: manual merge of the selinux tree with Linus' tree

From: Daniel Jurgens
Date: Mon May 22 2017 - 17:13:16 EST


On 5/22/2017 4:08 PM, Paul Moore wrote:
> On Sun, May 21, 2017 at 10:38 PM, Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx> wrote:
>> Hi Paul,
>>
>> Today's linux-next merge of the selinux tree got a conflict in:
>>
>> include/rdma/ib_verbs.h
>>
>> between commit:
>>
>> 2fc775726491 ("IB/opa-vnic: RDMA NETDEV interface")
>>
>> from Linus' tree and commit:
>>
>> 89b54b4d09bd ("IB/core: Enforce PKey security on QPs")
>>
>> from the selinux tree.
>>
>> I fixed it up (see below) and can carry the fix as necessary. This
>> is now fixed as far as linux-next is concerned, but any non trivial
>> conflicts should be mentioned to your upstream maintainer when your tree
>> is submitted for merging. You may also want to consider cooperating
>> with the maintainer of the conflicting tree to minimise any particularly
>> complex conflicts.
> Thanks Stephen.
>
> Daniel and Doug, does Stephen's patch look right to you?

Looks good to me.

>
>> diff --cc include/rdma/ib_verbs.h
>> index ba8314ec5768,c9e903fc824b..000000000000
>> --- a/include/rdma/ib_verbs.h
>> +++ b/include/rdma/ib_verbs.h
>> @@@ -1890,7 -1878,10 +1930,8 @@@ enum ib_mad_result
>> IB_MAD_RESULT_CONSUMED = 1 << 2 /* Packet consumed: stop processing */
>> };
>>
>> -#define IB_DEVICE_NAME_MAX 64
>> -
>> struct ib_port_cache {
>> + u64 subnet_prefix;
>> struct ib_pkey_cache *pkey;
>> struct ib_gid_table *gid;
>> u8 lmc;
>> @@@ -1912,34 -1903,12 +1953,40 @@@ struct ib_port_immutable
>> u32 max_mad_size;
>> };
>>
>> +/* rdma netdev type - specifies protocol type */
>> +enum rdma_netdev_t {
>> + RDMA_NETDEV_OPA_VNIC,
>> + RDMA_NETDEV_IPOIB,
>> +};
>> +
>> +/**
>> + * struct rdma_netdev - rdma netdev
>> + * For cases where netstack interfacing is required.
>> + */
>> +struct rdma_netdev {
>> + void *clnt_priv;
>> + struct ib_device *hca;
>> + u8 port_num;
>> +
>> + /* control functions */
>> + void (*set_id)(struct net_device *netdev, int id);
>> + /* send packet */
>> + int (*send)(struct net_device *dev, struct sk_buff *skb,
>> + struct ib_ah *address, u32 dqpn);
>> + /* multicast */
>> + int (*attach_mcast)(struct net_device *dev, struct ib_device *hca,
>> + union ib_gid *gid, u16 mlid,
>> + int set_qkey, u32 qkey);
>> + int (*detach_mcast)(struct net_device *dev, struct ib_device *hca,
>> + union ib_gid *gid, u16 mlid);
>> +};
>> +
>> + struct ib_port_pkey_list {
>> + /* Lock to hold while modifying the list. */
>> + spinlock_t list_lock;
>> + struct list_head pkey_list;
>> + };
>> +
>> struct ib_device {
>> /* Do not access @dma_device directly from ULP nor from HW drivers. */
>> struct device *dma_device;
>
>