Re: [Intel-gfx] [PATCH] drm/modes: Switch to 64bit maths to avoid integer overflow

2020-11-25 Thread Chris Wilson
Quoting Ville Syrjälä (2020-10-30 14:43:46) > On Fri, Oct 30, 2020 at 02:19:45PM +, Chris Wilson wrote: > > Quoting Ville Syrjala (2020-10-22 20:42:56) > > > From: Ville Syrjälä > > > > > > The new >8k CEA modes have dotclocks reaching 5.94 GHz, which > > > means our clock*1000 will now overf

Re: [Intel-gfx] [PATCH] drm/modes: Switch to 64bit maths to avoid integer overflow

2020-10-30 Thread Ville Syrjälä
On Fri, Oct 30, 2020 at 02:19:45PM +, Chris Wilson wrote: > Quoting Ville Syrjala (2020-10-22 20:42:56) > > From: Ville Syrjälä > > > > The new >8k CEA modes have dotclocks reaching 5.94 GHz, which > > means our clock*1000 will now overflow the 32bit unsigned > > integer. Switch to 64bit math

Re: [Intel-gfx] [PATCH] drm/modes: Switch to 64bit maths to avoid integer overflow

2020-10-30 Thread Chris Wilson
Quoting Ville Syrjala (2020-10-22 20:42:56) > From: Ville Syrjälä > > The new >8k CEA modes have dotclocks reaching 5.94 GHz, which > means our clock*1000 will now overflow the 32bit unsigned > integer. Switch to 64bit maths to avoid it. > > Cc: sta...@vger.kernel.org > Reported-by: Randy Dunlap

Re: [Intel-gfx] [PATCH] drm/modes: Switch to 64bit maths to avoid integer overflow

2020-10-23 Thread Randy Dunlap
On 10/22/20 12:42 PM, Ville Syrjala wrote: > From: Ville Syrjälä > > The new >8k CEA modes have dotclocks reaching 5.94 GHz, which > means our clock*1000 will now overflow the 32bit unsigned > integer. Switch to 64bit maths to avoid it. > > Cc: sta...@vger.kernel.org > Reported-by: Randy Dunlap

[Intel-gfx] [PATCH] drm/modes: Switch to 64bit maths to avoid integer overflow

2020-10-22 Thread Ville Syrjala
From: Ville Syrjälä The new >8k CEA modes have dotclocks reaching 5.94 GHz, which means our clock*1000 will now overflow the 32bit unsigned integer. Switch to 64bit maths to avoid it. Cc: sta...@vger.kernel.org Reported-by: Randy Dunlap Signed-off-by: Ville Syrjälä --- An interesting question