This is a note to let you know that I've just added the patch titled qlcnic: use the correct ring in qlcnic_83xx_process_rcv_ring_diag() to the 4.4-stable tree which can be found at: http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary The filename of the patch is: qlcnic-use-the-correct-ring-in-qlcnic_83xx_process_rcv_ring_diag.patch and it can be found in the queue-4.4 subdirectory. If you, or anyone else, feels it should not be added to the stable tree, please let <stable@xxxxxxxxxxxxxxx> know about it. >From 5b4d10f5e0369ed79434593b7cd8e85eebbe473f Mon Sep 17 00:00:00 2001 From: Dan Carpenter <dan.carpenter@xxxxxxxxxx> Date: Mon, 27 Jun 2016 23:50:29 +0300 Subject: qlcnic: use the correct ring in qlcnic_83xx_process_rcv_ring_diag() From: Dan Carpenter <dan.carpenter@xxxxxxxxxx> commit 5b4d10f5e0369ed79434593b7cd8e85eebbe473f upstream. There is a static checker warning here "warn: mask and shift to zero" and the code sets "ring" to zero every time. From looking at how QLCNIC_FETCH_RING_ID() is used in qlcnic_83xx_process_rcv_ring() the qlcnic_83xx_hndl() should be removed. Fixes: 4be41e92f7c6 ('qlcnic: 83xx data path routines') Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> Signed-off-by: David S. Miller <davem@xxxxxxxxxxxxx> Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> --- drivers/net/ethernet/qlogic/qlcnic/qlcnic_io.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- a/drivers/net/ethernet/qlogic/qlcnic/qlcnic_io.c +++ b/drivers/net/ethernet/qlogic/qlcnic/qlcnic_io.c @@ -2220,7 +2220,7 @@ void qlcnic_83xx_process_rcv_ring_diag(s if (!opcode) return; - ring = QLCNIC_FETCH_RING_ID(qlcnic_83xx_hndl(sts_data[0])); + ring = QLCNIC_FETCH_RING_ID(sts_data[0]); qlcnic_83xx_process_rcv_diag(adapter, ring, sts_data); desc = &sds_ring->desc_head[consumer]; desc->status_desc_data[0] = cpu_to_le64(STATUS_OWNER_PHANTOM); Patches currently in stable-queue which might be from dan.carpenter@xxxxxxxxxx are queue-4.4/net-moxa-fix-an-error-code.patch queue-4.4/cx23885-uninitialized-variable-in-cx23885_av_work_handler.patch queue-4.4/power-ipaq-micro-battery-freeing-the-wrong-variable.patch queue-4.4/i40e-fix-an-uninitialized-variable-bug.patch queue-4.4/mips-rm7000-double-locking-bug-in-rm7k_tc_disable.patch queue-4.4/btrfs-clean-up-an-error-code-in-btrfs_init_space_info.patch queue-4.4/xprtrdma-checking-for-null-instead-of-is_err.patch queue-4.4/asoc-intel-pass-correct-parameter-in-sst_alloc_stream_mrfld.patch queue-4.4/ath9k_htc-check-for-underflow-in-ath9k_htc_rx_msg.patch queue-4.4/qlcnic-use-the-correct-ring-in-qlcnic_83xx_process_rcv_ring_diag.patch queue-4.4/mips-octeon-off-by-one-in-octeon_irq_gpio_map.patch queue-4.4/qede-uninitialized-variable-in-qede_start_xmit.patch queue-4.4/mfd-lp8788-irq-uninitialized-variable-in-irq-handler.patch queue-4.4/nfc-nci-memory-leak-in-nci_core_conn_create.patch queue-4.4/ethernet-micrel-fix-some-error-codes.patch queue-4.4/am437x-vpfe-fix-an-uninitialized-variable-bug.patch queue-4.4/qlcnic-potential-null-dereference-in-qlcnic_83xx_get_minidump_template.patch queue-4.4/bpf-mips-fix-off-by-one-in-ctx-offset-allocation.patch queue-4.4/mdio-sun4i-oops-in-error-handling-in-probe.patch queue-4.4/vfio-platform-reset-fix-a-warning-message-condition.patch queue-4.4/x86-apic-uv-silence-a-shift-wrapping-warning.patch