On 7/10/2025 1:33 PM, Loktionov, Aleksandr wrote:
-----Original Message-----
From: Intel-wired-lan <[email protected]> On Behalf
Of Simon Horman
Sent: Thursday, July 10, 2025 12:25 PM
To: Markus Blöchl <[email protected]>
Cc: Nguyen, Anthony L <[email protected]>; Kitszel,
Przemyslaw <[email protected]>; Richard Cochran
<[email protected]>; Thomas Gleixner <[email protected]>;
Lakshmi Sowjanya D <[email protected]>; Andrew Lunn
<[email protected]>; David S. Miller <[email protected]>; Eric
Dumazet <[email protected]>; Jakub Kicinski <[email protected]>; Paolo
Abeni <[email protected]>; Bloechl, Markus
<[email protected]>; John Stultz <[email protected]>;
[email protected]; [email protected]; linux-
[email protected]
Subject: Re: [Intel-wired-lan] [PATCH v2] e1000e: Populate entire
system_counterval_t in get_time_fn() callback
On Wed, Jul 09, 2025 at 07:28:07PM +0200, Markus Blöchl wrote:
get_time_fn() callback implementations are expected to fill out the
entire system_counterval_t struct as it may be initially
uninitialized.
This broke with the removal of convert_art_to_tsc() helper functions
which left use_nsecs uninitialized.
Assign the entire struct again.
Fixes: bd48b50be50a ("e1000e: Replace convert_art_to_tsc()")
Cc: [email protected]
Signed-off-by: Markus Blöchl <[email protected]>
---
Notes:
Related-To:
<https://lore.kernel.org/lkml/txyrr26hxe3xpq3ebqb5ewkgvhvp7xalotaouwlu
djtjifnah2@7tmgczln4aoo/>
Changes in v2:
- Add Lakshmi in Cc:
- Add Signed-off-by: trailer which was lost in b4 workflow
- Link to v1:
https://lore.kernel.org/r/20250709-e1000e_crossts-v1-1-
f8a80c792e4f@bl
ochl.de
Reviewed-by: Simon Horman <[email protected]>
Reviewed-by: Aleksandr Loktionov <[email protected]>
Reviewed-by: Vitaly Lifshits <[email protected]>
