Re: [PATCH] sched: fix integer handling issue

2022-02-23 Thread Ferruh Yigit
On 2/23/2022 7:17 AM, Morten Brørup wrote: From: Stephen Hemminger [mailto:step...@networkplumber.org] Sent: Tuesday, 22 February 2022 17.03 On Tue, 22 Feb 2022 15:13:53 +0100 Morten Brørup wrote: From: Megha Ajmera [mailto:megha.ajm...@intel.com] Sent: Tuesday, 22 February 2022 14.19 Maskin

RE: [PATCH] sched: fix integer handling issue

2022-02-22 Thread Morten Brørup
> From: Stephen Hemminger [mailto:step...@networkplumber.org] > Sent: Tuesday, 22 February 2022 17.03 > > On Tue, 22 Feb 2022 15:13:53 +0100 > Morten Brørup wrote: > > > > From: Megha Ajmera [mailto:megha.ajm...@intel.com] > > > Sent: Tuesday, 22 February 2022 14.19 > > > > > > Masking of core m

RE: [PATCH] sched: fix integer handling issue

2022-02-22 Thread Ajmera, Megha
> From: Stephen Hemminger > Sent: Tuesday, February 22, 2022 9:33 PM > > On Tue, 22 Feb 2022 15:13:53 +0100 > Morten Brørup wrote: > > > > From: Megha Ajmera [mailto:megha.ajm...@intel.com] > > > Sent: Tuesday, 22 February 2022 14.19 > > > > > > Masking of core mask was incorrect. Instead of us

Re: [PATCH] sched: fix integer handling issue

2022-02-22 Thread Stephen Hemminger
On Tue, 22 Feb 2022 15:13:53 +0100 Morten Brørup wrote: > > From: Megha Ajmera [mailto:megha.ajm...@intel.com] > > Sent: Tuesday, 22 February 2022 14.19 > > > > Masking of core mask was incorrect. Instead of using 1U for shifting, > > it > > should be using 1LU as the result is assigned to uint6

RE: [PATCH] sched: fix integer handling issue

2022-02-22 Thread Morten Brørup
> From: Megha Ajmera [mailto:megha.ajm...@intel.com] > Sent: Tuesday, 22 February 2022 14.19 > > Masking of core mask was incorrect. Instead of using 1U for shifting, > it > should be using 1LU as the result is assigned to uint64. > > CID 375859: Potentially overflowing expression "1U << app_main

[PATCH] sched: fix integer handling issue

2022-02-22 Thread Megha Ajmera
Masking of core mask was incorrect. Instead of using 1U for shifting, it should be using 1LU as the result is assigned to uint64. CID 375859: Potentially overflowing expression "1U << app_main_core" with type "unsigned int" (32 bits, unsigned) is evaluated using 32-bit arithmetic, and then used in