Re: [PATCH 1/3] drm/amd/display: drop redundant memset() in get_available_dsc_slices()

2023-05-18 Thread Marion & Christophe JAILLET
Le 17/05/2023 à 20:33, Hamza Mahfooz a écrit : get_available_dsc_slices() returns the number of indices set, and all of the users of get_available_dsc_slices() don't cross the returned bound when iterating over available_slices[]. So, the memset() in get_available_dsc_slices() is redundant and

Re: [PATCH] drm/amd/pm: avoid large variable on kernel stack

2022-12-15 Thread Marion & Christophe JAILLET
Le 15/12/2022 à 20:46, Christophe JAILLET a écrit : Le 15/12/2022 à 17:36, Arnd Bergmann a écrit : From: Arnd Bergmann The activity_monitor_external[] array is too big to fit on the kernel stack, resulting in this warning with clang:

Re: [PATCH] drm: amdgpu: Use the correct size when allocating memory

2020-08-10 Thread Marion & Christophe JAILLET
Le 10/08/2020 à 17:42, Dan Carpenter a écrit : On Sun, Aug 09, 2020 at 10:34:06PM +0200, Christophe JAILLET wrote: When '*sgt' is allocated, we must allocated 'sizeof(**sgt)' bytes instead of 'sizeof(*sg)'. 'sg' (i.e. struct scatterlist) is smaller than 'sgt' (i.e struct sg_table), so this