OSDN Git Service

net: lan966x: Fix lan966x_ifh_get
authorHoratiu Vultur <horatiu.vultur@microchip.com>
Mon, 17 Apr 2023 07:26:41 +0000 (09:26 +0200)
committerDavid S. Miller <davem@davemloft.net>
Mon, 17 Apr 2023 08:56:49 +0000 (09:56 +0100)
commit99676a5766412f3936c55b9d18565d248e5463ee
tree5386bf2aed82821e0fec679f2692d16d736bd2d7
parent0af03871b6af83622b652218039f3ac8864a5a3b
net: lan966x: Fix lan966x_ifh_get

From time to time, it was observed that the nanosecond part of the
received timestamp, which is extracted from the IFH, it was actually
bigger than 1 second. So then when actually calculating the full
received timestamp, based on the nanosecond part from IFH and the second
part which is read from HW, it was actually wrong.

The issue seems to be inside the function lan966x_ifh_get, which
extracts information from an IFH(which is an byte array) and returns the
value in a u64. When extracting the timestamp value from the IFH, which
starts at bit 192 and have the size of 32 bits, then if the most
significant bit was set in the timestamp, then this bit was extended
then the return value became 0xffffffff... . And the reason of this is
because constants without any postfix are treated as signed longs and
that is the reason why '1 << 31' becomes 0xffffffff80000000.
This is fixed by adding the postfix 'ULL' to 1.

Fixes: fd7627833ddf ("net: lan966x: Stop using packing library")
Signed-off-by: Horatiu Vultur <horatiu.vultur@microchip.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/microchip/lan966x/lan966x_main.c