[Nouveau] [RFC mesa] clover: Fix pipe_grid_info.indirect not being initialized
Hans de Goede
hdegoede at redhat.com
Mon Mar 14 14:00:57 UTC 2016
Messed up the subject prefix, sorry. Resending with proper prefix.
On 14-03-16 15:00, Hans de Goede wrote:
> After pipe_grid_info.indirect was introduced, clover was not modified
> to set it causing it to pass uninitialized memory for it to launch_grid.
>
> This commit fixes this by zero-ing the entire pipe_grid_info struct when
> declaring it, to avoid similar problems popping-up in the future.
>
> Cc: "11.2" <mesa-stable at lists.freedesktop.org>
> Signed-off-by: Hans de Goede <hdegoede at redhat.com>
> ---
> Changes in v2:
> -Drop trailing "," from struct initializer
> -Add Cc: "11.2" <mesa-stable at lists.freedesktop.org>
> ---
> src/gallium/state_trackers/clover/core/kernel.cpp | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gallium/state_trackers/clover/core/kernel.cpp b/src/gallium/state_trackers/clover/core/kernel.cpp
> index 8396be9..1ab87ec 100644
> --- a/src/gallium/state_trackers/clover/core/kernel.cpp
> +++ b/src/gallium/state_trackers/clover/core/kernel.cpp
> @@ -55,7 +55,7 @@ kernel::launch(command_queue &q,
> const auto reduced_grid_size =
> map(divides(), grid_size, block_size);
> void *st = exec.bind(&q, grid_offset);
> - struct pipe_grid_info info;
> + struct pipe_grid_info info = { 0 };
>
> // The handles are created during exec_context::bind(), so we need make
> // sure to call exec_context::bind() before retrieving them.
>
More information about the Nouveau
mailing list