[Hyper-V] IB/mlx5: Respect new UMR capabilities

Bug #1762554 reported by Madhuri Kaniganti
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
linux-azure (Ubuntu)
Fix Released
Undecided
Marcelo Cerri
Bionic
Fix Released
Undecided
Marcelo Cerri

Bug Description

In some firmware configuration, UMR usage from Virtual Functions is restricted.
This information is published to the driver using new capability bits.

Avoid using UMRs in these cases and use the Firmware slow-path flow to create
mkeys and populate them with Virtual to Physical address translation.

Older drivers that do not have this patch, will end up using memory keys that
aren't populated with Virtual to Physical address translation that is done
part of the UMR work.

https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?h=next-20180409&id=c8d75a980fab886a9c716567e6b47cc414ad84ee

and pull this patch as well
IB/mlx5: Enable ECN capable bits for UD RoCE v2 QPs
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/drivers/infiniband/hw/mlx5?id=ea8af0d2f2b5b16da4553205ddaf225e0a057e03

Revision history for this message
Madhuri Kaniganti (madhukan) wrote :
description: updated
Joshua R. Poulson (jrp)
Changed in linux-azure (Ubuntu):
status: New → Confirmed
description: updated
Revision history for this message
Marcelo Cerri (mhcerri) wrote :

Just to confirm. Is that only required for 4.15 kernels?

Changed in linux-azure (Ubuntu Bionic):
assignee: nobody → Marcelo Cerri (mhcerri)
Revision history for this message
Joshua R. Poulson (jrp) wrote :

Yes, this is for DPDK and therefore applies only to 4.15 forward.

Revision history for this message
Marcelo Cerri (mhcerri) wrote :

Hi, Madhuri. Is this entire PR from comment #1 necessary?

I'm facing a lot of conflicts when trying to apply it to 4.15. The two commits from the initial description apply cleanly though.

Revision history for this message
David Coronel (davecore) wrote :

@Marcelo, I have a reply from Madhuri:

Created By: Madhuri Kaniganti (portal) (07/06/2018 11:40 AM)
Just these two fixes needed:

IB/mlx5: Respect new UMR capabilities
https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?h=next-20180409&id=c8d75a980fab886a9c716567e6b47cc414ad84ee

IB/mlx5: Enable ECN capable bits for UD RoCE v2 QPs
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/drivers/infiniband/hw/mlx5?id=ea8af0d2f2b5b16da4553205ddaf225e0a057e03

Marcelo Cerri (mhcerri)
Changed in linux-azure (Ubuntu Bionic):
status: Confirmed → In Progress
Marcelo Cerri (mhcerri)
Changed in linux-azure (Ubuntu Bionic):
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (61.4 KiB)

This bug was fixed in the package linux-azure - 4.15.0-1014.14~16.04.1

---------------
linux-azure (4.15.0-1014.14~16.04.1) xenial; urgency=medium

  * linux-azure: 4.15.0-1014.14~16.04.1 -proposed tracker (LP: #1776346)

  * linux-azure: 4.15.0-1014.14 -proposed tracker (LP: #1776342)

  * [Hyper-V] Disable CONFIG_HOTPLUG_CPU in linux-azure (LP: #1776293)
    - [Config] azure: CONFIG_HOTPLUG_CPU=n

  * [Hyper-V] IB/mlx5: Respect new UMR capabilities (LP: #1762554)
    - IB/mlx5: Enable ECN capable bits for UD RoCE v2 QPs
    - IB/mlx5: Respect new UMR capabilities

  [ Ubuntu: 4.15.0-24.26 ]

  * linux: 4.15.0-24.26 -proposed tracker (LP: #1776338)
  * Bionic update: upstream stable patchset 2018-06-06 (LP: #1775483)
    - drm: bridge: dw-hdmi: Fix overflow workaround for Amlogic Meson GX SoCs
    - i40e: Fix attach VF to VM issue
    - tpm: cmd_ready command can be issued only after granting locality
    - tpm: tpm-interface: fix tpm_transmit/_cmd kdoc
    - tpm: add retry logic
    - Revert "ath10k: send (re)assoc peer command when NSS changed"
    - bonding: do not set slave_dev npinfo before slave_enable_netpoll in
      bond_enslave
    - ipv6: add RTA_TABLE and RTA_PREFSRC to rtm_ipv6_policy
    - ipv6: sr: fix NULL pointer dereference in seg6_do_srh_encap()- v4 pkts
    - KEYS: DNS: limit the length of option strings
    - l2tp: check sockaddr length in pppol2tp_connect()
    - net: validate attribute sizes in neigh_dump_table()
    - llc: delete timers synchronously in llc_sk_free()
    - tcp: don't read out-of-bounds opsize
    - net: af_packet: fix race in PACKET_{R|T}X_RING
    - tcp: md5: reject TCP_MD5SIG or TCP_MD5SIG_EXT on established sockets
    - net: fix deadlock while clearing neighbor proxy table
    - team: avoid adding twice the same option to the event list
    - net/smc: fix shutdown in state SMC_LISTEN
    - team: fix netconsole setup over team
    - packet: fix bitfield update race
    - tipc: add policy for TIPC_NLA_NET_ADDR
    - pppoe: check sockaddr length in pppoe_connect()
    - vlan: Fix reading memory beyond skb->tail in skb_vlan_tagged_multi
    - amd-xgbe: Add pre/post auto-negotiation phy hooks
    - sctp: do not check port in sctp_inet6_cmp_addr
    - amd-xgbe: Improve KR auto-negotiation and training
    - strparser: Do not call mod_delayed_work with a timeout of LONG_MAX
    - amd-xgbe: Only use the SFP supported transceiver signals
    - strparser: Fix incorrect strp->need_bytes value.
    - net: sched: ife: signal not finding metaid
    - tcp: clear tp->packets_out when purging write queue
    - net: sched: ife: handle malformed tlv length
    - net: sched: ife: check on metadata length
    - llc: hold llc_sap before release_sock()
    - llc: fix NULL pointer deref for SOCK_ZAPPED
    - net: ethernet: ti: cpsw: fix tx vlan priority mapping
    - virtio_net: split out ctrl buffer
    - virtio_net: fix adding vids on big-endian
    - KVM: s390: force bp isolation for VSIE
    - s390: correct module section names for expoline code revert
    - microblaze: Setup dependencies for ASM optimized lib functions
    - commoncap: Handle memory allocation failure.
    - scsi: mptsas: Disable WRITE SAME
    - c...

Changed in linux-azure (Ubuntu):
status: Confirmed → Fix Released
status: Confirmed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (49.9 KiB)

This bug was fixed in the package linux-azure - 4.15.0-1014.14

---------------
linux-azure (4.15.0-1014.14) bionic; urgency=medium

  * linux-azure: 4.15.0-1014.14 -proposed tracker (LP: #1776342)

  * [Hyper-V] Disable CONFIG_HOTPLUG_CPU in linux-azure (LP: #1776293)
    - [Config] azure: CONFIG_HOTPLUG_CPU=n

  * [Hyper-V] IB/mlx5: Respect new UMR capabilities (LP: #1762554)
    - IB/mlx5: Enable ECN capable bits for UD RoCE v2 QPs
    - IB/mlx5: Respect new UMR capabilities

  [ Ubuntu: 4.15.0-24.26 ]

  * linux: 4.15.0-24.26 -proposed tracker (LP: #1776338)
  * Bionic update: upstream stable patchset 2018-06-06 (LP: #1775483)
    - drm: bridge: dw-hdmi: Fix overflow workaround for Amlogic Meson GX SoCs
    - i40e: Fix attach VF to VM issue
    - tpm: cmd_ready command can be issued only after granting locality
    - tpm: tpm-interface: fix tpm_transmit/_cmd kdoc
    - tpm: add retry logic
    - Revert "ath10k: send (re)assoc peer command when NSS changed"
    - bonding: do not set slave_dev npinfo before slave_enable_netpoll in
      bond_enslave
    - ipv6: add RTA_TABLE and RTA_PREFSRC to rtm_ipv6_policy
    - ipv6: sr: fix NULL pointer dereference in seg6_do_srh_encap()- v4 pkts
    - KEYS: DNS: limit the length of option strings
    - l2tp: check sockaddr length in pppol2tp_connect()
    - net: validate attribute sizes in neigh_dump_table()
    - llc: delete timers synchronously in llc_sk_free()
    - tcp: don't read out-of-bounds opsize
    - net: af_packet: fix race in PACKET_{R|T}X_RING
    - tcp: md5: reject TCP_MD5SIG or TCP_MD5SIG_EXT on established sockets
    - net: fix deadlock while clearing neighbor proxy table
    - team: avoid adding twice the same option to the event list
    - net/smc: fix shutdown in state SMC_LISTEN
    - team: fix netconsole setup over team
    - packet: fix bitfield update race
    - tipc: add policy for TIPC_NLA_NET_ADDR
    - pppoe: check sockaddr length in pppoe_connect()
    - vlan: Fix reading memory beyond skb->tail in skb_vlan_tagged_multi
    - amd-xgbe: Add pre/post auto-negotiation phy hooks
    - sctp: do not check port in sctp_inet6_cmp_addr
    - amd-xgbe: Improve KR auto-negotiation and training
    - strparser: Do not call mod_delayed_work with a timeout of LONG_MAX
    - amd-xgbe: Only use the SFP supported transceiver signals
    - strparser: Fix incorrect strp->need_bytes value.
    - net: sched: ife: signal not finding metaid
    - tcp: clear tp->packets_out when purging write queue
    - net: sched: ife: handle malformed tlv length
    - net: sched: ife: check on metadata length
    - llc: hold llc_sap before release_sock()
    - llc: fix NULL pointer deref for SOCK_ZAPPED
    - net: ethernet: ti: cpsw: fix tx vlan priority mapping
    - virtio_net: split out ctrl buffer
    - virtio_net: fix adding vids on big-endian
    - KVM: s390: force bp isolation for VSIE
    - s390: correct module section names for expoline code revert
    - microblaze: Setup dependencies for ASM optimized lib functions
    - commoncap: Handle memory allocation failure.
    - scsi: mptsas: Disable WRITE SAME
    - cdrom: information leak in cdrom_ioctl_media_changed()
    - m68k/mac: Don't remap SWIM MMI...

Changed in linux-azure (Ubuntu Bionic):
status: Fix Committed → Fix Released
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.