On 2015/11/27 5:15, Thomas Monjalon wrote: > 2015-11-26 09:35, Michael Qiu: >> In i686 platform, long is 32bit, so XXX_CYCLECOUNTER_MASK >> need define as 'ULL' >> >> Signed-off-by: Michael Qiu <michael.qiu at intel.com> > This patch is correct but the description is not exact: > I have no issue with my i686 compiler. > For future reference, please could you be more precise > about the reproduction environment? Is it related to a specific compiler?
OK, I will be for careful about the compile and os next time. > We also need to add these lines: > Fixes: 9c857bf6be87 ("igb: support ieee1588 functions for device time") > Fixes: 1c4445e1f28e ("ixgbe: support ieee1588 functions for device time") > Fixes: f3a4e40eca0c ("i40e: support ieee1588 functions for device time") So I will repost the patch set. Thanks, Michael