* [PATCH net 1/1] qed: Fix mask for physical address in ILT entry
@ 2018-05-21 19:31 Shahed Shaikh
2018-05-22 19:33 ` David Miller
0 siblings, 1 reply; 4+ messages in thread
From: Shahed Shaikh @ 2018-05-21 19:31 UTC (permalink / raw)
To: davem; +Cc: netdev, Ariel.Elior, Dept-EngEverestLinuxL2
ILT entry requires 12 bit right shifted physical address.
Existing mask for ILT entry of physical address i.e.
ILT_ENTRY_PHY_ADDR_MASK is not sufficient to handle 64bit
address because upper 8 bits of 64 bit address were getting
masked which resulted in completer abort error on
PCIe bus due to invalid address.
Fix that mask to handle 64bit physical address.
Fixes: fe56b9e6a8d9 ("qed: Add module with basic common support")
Signed-off-by: Shahed Shaikh <shahed.shaikh@cavium.com>
Signed-off-by: Ariel Elior <ariel.elior@cavium.com>
---
drivers/net/ethernet/qlogic/qed/qed_cxt.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/net/ethernet/qlogic/qed/qed_cxt.c b/drivers/net/ethernet/qlogic/qed/qed_cxt.c
index 00f41c1..820b226 100644
--- a/drivers/net/ethernet/qlogic/qed/qed_cxt.c
+++ b/drivers/net/ethernet/qlogic/qed/qed_cxt.c
@@ -77,7 +77,7 @@
#define ILT_CFG_REG(cli, reg) PSWRQ2_REG_ ## cli ## _ ## reg ## _RT_OFFSET
/* ILT entry structure */
-#define ILT_ENTRY_PHY_ADDR_MASK 0x000FFFFFFFFFFFULL
+#define ILT_ENTRY_PHY_ADDR_MASK (~0ULL >> 12)
#define ILT_ENTRY_PHY_ADDR_SHIFT 0
#define ILT_ENTRY_VALID_MASK 0x1ULL
#define ILT_ENTRY_VALID_SHIFT 52
--
1.7.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH net 1/1] qed: Fix mask for physical address in ILT entry
2018-05-21 19:31 [PATCH net 1/1] qed: Fix mask for physical address in ILT entry Shahed Shaikh
@ 2018-05-22 19:33 ` David Miller
2018-05-22 19:42 ` Shaikh, Shahed
0 siblings, 1 reply; 4+ messages in thread
From: David Miller @ 2018-05-22 19:33 UTC (permalink / raw)
To: shahed.shaikh; +Cc: netdev, Ariel.Elior, Dept-EngEverestLinuxL2
From: Shahed Shaikh <shahed.shaikh@cavium.com>
Date: Mon, 21 May 2018 12:31:47 -0700
> ILT entry requires 12 bit right shifted physical address.
> Existing mask for ILT entry of physical address i.e.
> ILT_ENTRY_PHY_ADDR_MASK is not sufficient to handle 64bit
> address because upper 8 bits of 64 bit address were getting
> masked which resulted in completer abort error on
> PCIe bus due to invalid address.
>
> Fix that mask to handle 64bit physical address.
>
> Fixes: fe56b9e6a8d9 ("qed: Add module with basic common support")
>
> Signed-off-by: Shahed Shaikh <shahed.shaikh@cavium.com>
> Signed-off-by: Ariel Elior <ariel.elior@cavium.com>
Please do not put an empty line between Fixes: and other tags in the
future. I fixed it up this time.
Applied, thanks.
^ permalink raw reply [flat|nested] 4+ messages in thread
* RE: [PATCH net 1/1] qed: Fix mask for physical address in ILT entry
2018-05-22 19:33 ` David Miller
@ 2018-05-22 19:42 ` Shaikh, Shahed
2018-05-22 19:48 ` David Miller
0 siblings, 1 reply; 4+ messages in thread
From: Shaikh, Shahed @ 2018-05-22 19:42 UTC (permalink / raw)
To: David Miller; +Cc: netdev, Elior, Ariel, Dept-Eng Everest Linux L2
> -----Original Message-----
> From: David Miller <davem@davemloft.net>
> Sent: Tuesday, May 22, 2018 12:34 PM
> To: Shaikh, Shahed <Shahed.Shaikh@cavium.com>
> Cc: netdev@vger.kernel.org; Elior, Ariel <Ariel.Elior@cavium.com>; Dept-Eng
> Everest Linux L2 <Dept-EngEverestLinuxL2@cavium.com>
> Subject: Re: [PATCH net 1/1] qed: Fix mask for physical address in ILT entry
>
> From: Shahed Shaikh <shahed.shaikh@cavium.com>
> Date: Mon, 21 May 2018 12:31:47 -0700
>
> > ILT entry requires 12 bit right shifted physical address.
> > Existing mask for ILT entry of physical address i.e.
> > ILT_ENTRY_PHY_ADDR_MASK is not sufficient to handle 64bit address
> > because upper 8 bits of 64 bit address were getting masked which
> > resulted in completer abort error on PCIe bus due to invalid address.
> >
> > Fix that mask to handle 64bit physical address.
> >
> > Fixes: fe56b9e6a8d9 ("qed: Add module with basic common support")
> >
> > Signed-off-by: Shahed Shaikh <shahed.shaikh@cavium.com>
> > Signed-off-by: Ariel Elior <ariel.elior@cavium.com>
>
> Please do not put an empty line between Fixes: and other tags in the future. I
> fixed it up this time.
Sorry about this. Thanks for fixing.
>
> Applied, thanks.
Can you please queues this fix for -stable?
Thanks,
Shahed
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH net 1/1] qed: Fix mask for physical address in ILT entry
2018-05-22 19:42 ` Shaikh, Shahed
@ 2018-05-22 19:48 ` David Miller
0 siblings, 0 replies; 4+ messages in thread
From: David Miller @ 2018-05-22 19:48 UTC (permalink / raw)
To: Shahed.Shaikh; +Cc: netdev, Ariel.Elior, Dept-EngEverestLinuxL2
From: "Shaikh, Shahed" <Shahed.Shaikh@cavium.com>
Date: Tue, 22 May 2018 19:42:46 +0000
> Can you please queues this fix for -stable?
I did, see:
http://patchwork.ozlabs.org/bundle/davem/stable/?series=&submitter=&state=*&q=&archive=
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-05-22 19:48 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-05-21 19:31 [PATCH net 1/1] qed: Fix mask for physical address in ILT entry Shahed Shaikh
2018-05-22 19:33 ` David Miller
2018-05-22 19:42 ` Shaikh, Shahed
2018-05-22 19:48 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).