Re: [PATCH] drm/amd/display: Use pre-allocated temp struct for bounding box update

2022-06-08 Thread Aurabindo Pillai
:* Wentland, Harry ; Pillai, Aurabindo ; Siqueira, Rodrigo ; Li, Sun peng (Leo) *Subject:* [PATCH] drm/amd/display: Use pre-allocated temp struct for bounding box update From: Leo Li [Why] There is a theoretical problem in prior patches for reducing the stack size of *update_bw_bounding_box

Re: [PATCH] drm/amd/display: Use pre-allocated temp struct for bounding box update

2022-06-08 Thread Harry Wentland
On 2022-06-08 12:48, sunpeng...@amd.com wrote: > From: Leo Li > > [Why] > > There is a theoretical problem in prior patches for reducing the stack > size of *update_bw_bounding_box() functions. > > By modifying the soc.clock_limits[n] struct directly, this can cause > unintended behavior as

Re: [PATCH] drm/amd/display: Use pre-allocated temp struct for bounding box update

2022-06-08 Thread Rodrigo Siqueira Jordao
, 2022 12:48 PM *To:* amd-gfx@lists.freedesktop.org *Cc:* Wentland, Harry ; Pillai, Aurabindo ; Siqueira, Rodrigo ; Li, Sun peng (Leo) *Subject:* [PATCH] drm/amd/display: Use pre-allocated temp struct for bounding box update From: Leo Li [Why] There is a theoretical problem in prior

Re: [PATCH] drm/amd/display: Use pre-allocated temp struct for bounding box update

2022-06-08 Thread Leo
-- > *From:* Li, Sun peng (Leo) > *Sent:* Wednesday, June 8, 2022 12:48 PM > *To:* amd-gfx@lists.freedesktop.org > *Cc:* Wentland, Harry ; Pillai, Aurabindo > ; Siqueira, Rodrigo ; Li, > Sun peng (Leo) > *Subject:* [PATCH] drm/amd/display: Use pre-allocated

Re: [PATCH] drm/amd/display: Use pre-allocated temp struct for bounding box update

2022-06-08 Thread Pillai, Aurabindo
) Sent: Wednesday, June 8, 2022 12:48 PM To: amd-gfx@lists.freedesktop.org Cc: Wentland, Harry ; Pillai, Aurabindo ; Siqueira, Rodrigo ; Li, Sun peng (Leo) Subject: [PATCH] drm/amd/display: Use pre-allocated temp struct for bounding box update From: Leo Li [Why] There is a theoretical problem

[PATCH] drm/amd/display: Use pre-allocated temp struct for bounding box update

2022-06-08 Thread sunpeng.li
From: Leo Li [Why] There is a theoretical problem in prior patches for reducing the stack size of *update_bw_bounding_box() functions. By modifying the soc.clock_limits[n] struct directly, this can cause unintended behavior as the for loop attempts to swap rows in clock_limits[n]. A temporary