RE: [PATCH 1/1] qlcnic: add wmb() call in transmit data path.

2016-06-30 Thread Sony Chacko
Subject: Re: [PATCH 1/1] qlcnic: add wmb() call in transmit data path. > >> +/* Ensure writes are complete before HW fetches Tx descriptors */ >> +wmb(); >> qlcnic_update_cmd_producer(tx_ring); >> >> return NETDEV_TX_OK; >> > > Would not an mmiowb be more appropriate in this ca

[PATCH 0/1] qlcnic: add wmb() call in transmit data path.

2016-06-29 Thread Sony Chacko
0001-qlcnic-add-wmb-call-in-transmit-data-path.patch This patch adds a wmb() call in the Tx data path to ensure writes are completed before hardware fetches updated Tx descriptors. Please apply to net. Thanks, Sony

[PATCH 1/1] qlcnic: add wmb() call in transmit data path.

2016-06-29 Thread Sony Chacko
Call wmb() to ensure writes are complete before hardware fetches updated Tx descriptors. Signed-off-by: Sony Chacko --- drivers/net/ethernet/qlogic/qlcnic/qlcnic_io.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/drivers/net/ethernet/qlogic/qlcnic/qlcnic_io.c b

RE: [PATCH] qlcnic: constify qlcnic_mbx_ops structure

2015-10-12 Thread Sony Chacko
struct workqueue_struct *work_q; > struct qlcnic_adapter *adapter; > - struct qlcnic_mbx_ops *ops; > + const struct qlcnic_mbx_ops *ops; > struct work_struct work; > struct completion completion; > struct list_headcmd_q; > Acked-by: Sony Chacko Thanks, Sony -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

RE: [PATCH] bnx2:Make various functions to have a return type of void in the file bnx2.c

2015-07-13 Thread Sony Chacko
> Sony, > I also sent this patch and was wondering if I can get a reply on it. > From 4a607447562bec161fd947caae5eb02c2365c58a Mon Sep > 17 00:00:00 2001 > From: Nicholas Krause > Date: Wed, 8 Jul 2015 08:29:07 -0400 > Subject: [PATCH] bnx2i:Fix backwards locking scenario in the > function bnx2i_

RE: [PATCH] bnx2:Make various functions to have a return type of void in the file bnx2.c

2015-07-09 Thread Sony Chacko
> -Original Message- > From: Nicholas Krause [mailto:xerofo...@gmail.com] > Sent: Wednesday, July 08, 2015 11:31 AM > To: Sony Chacko > Cc: Dept-GE Linux NIC Dev; netdev; linux-kernel > Subject: [PATCH] bnx2:Make various functions to have a return > type of vo

RE: [PATCH net] bnx2x: fix lockdep splat

2015-06-26 Thread Sony Chacko
s/net/ethernet/broadcom/bnx2x/bnx2x_main.c > @@ -9331,7 +9331,8 @@ unload_error: > * function stop ramrod is sent, since as part of this ramrod > FW access >* PTP registers. >*/ > - bnx2x_stop_ptp(bp); > + if (bp->flags & PTP_SUPPORTED) > + bnx2x_stop_ptp(bp); > > /* Disable HW interrupts, NAPI */ > bnx2x_netif_stop(bp, 1); > Acked-by: Sony Chacko Thanks.