Re: [PATCH v1] drm/i915/selftest: Change throttle criteria for rps

2025-01-11 Thread Raag Jadav
On Sat, Jan 11, 2025 at 09:41:47PM +0100, Andi Shyti wrote: > Hi Raag, > > On Thu, Jan 02, 2025 at 04:36:18PM +0530, Raag Jadav wrote: > > Current live_rps_control() implementation errors out on throttling. > > This was done with the assumption that throttling to minimum frequency > > is a catastr

Re: [PATCH v1] drm/i915/selftest: Change throttle criteria for rps

2025-01-11 Thread Andi Shyti
Hi Raag, On Thu, Jan 02, 2025 at 04:36:18PM +0530, Raag Jadav wrote: > Current live_rps_control() implementation errors out on throttling. > This was done with the assumption that throttling to minimum frequency > is a catastrophic failure, which is incorrect. Throttling can happen > due to variet

Re: [PATCH v1] drm/i915/selftest: Change throttle criteria for rps

2025-01-09 Thread Rodrigo Vivi
On Wed, Jan 08, 2025 at 11:28:47AM +0200, Raag Jadav wrote: > On Thu, Jan 02, 2025 at 04:36:18PM +0530, Raag Jadav wrote: > > Current live_rps_control() implementation errors out on throttling. > > This was done with the assumption that throttling to minimum frequency > > is a catastrophic failure,

Re: [PATCH v1] drm/i915/selftest: Change throttle criteria for rps

2025-01-09 Thread Belgaumkar, Vinay
On 1/2/2025 3:06 AM, Raag Jadav wrote: Current live_rps_control() implementation errors out on throttling. This was done with the assumption that throttling to minimum frequency is a catastrophic failure, which is incorrect. Throttling can happen due to variety of reasons and often times out of

Re: [PATCH v1] drm/i915/selftest: Change throttle criteria for rps

2025-01-08 Thread Raag Jadav
On Thu, Jan 02, 2025 at 04:36:18PM +0530, Raag Jadav wrote: > Current live_rps_control() implementation errors out on throttling. > This was done with the assumption that throttling to minimum frequency > is a catastrophic failure, which is incorrect. Throttling can happen > due to variety of reaso

[PATCH v1] drm/i915/selftest: Change throttle criteria for rps

2025-01-02 Thread Raag Jadav
Current live_rps_control() implementation errors out on throttling. This was done with the assumption that throttling to minimum frequency is a catastrophic failure, which is incorrect. Throttling can happen due to variety of reasons and often times out of our control. Also, the resulting frequency