[PATCH] drm/nouveau: force alignment to 0x1000 for gpu objects

Ben Skeggs skeggsb at gmail.com
Tue Sep 3 20:34:27 PDT 2013


On Tue, Sep 3, 2013 at 12:31 AM, Maarten Lankhorst
<maarten.lankhorst at canonical.com> wrote:
> There are a lot of places that allocate multiples of 1000,
> but do not set alignment correctly and still require this
> alignment implicitly or explicitly.
This is wrong.  Where are the places you think you need this?

Ben.

> ---
>  drivers/gpu/drm/nouveau/core/core/gpuobj.c | 8 ++++++++
>  1 file changed, 8 insertions(+)
>
> diff --git a/drivers/gpu/drm/nouveau/core/core/gpuobj.c b/drivers/gpu/drm/nouveau/core/core/gpuobj.c
> index 7595506..7bcae1d 100644
> --- a/drivers/gpu/drm/nouveau/core/core/gpuobj.c
> +++ b/drivers/gpu/drm/nouveau/core/core/gpuobj.c
> @@ -65,6 +65,14 @@ nouveau_gpuobj_create_(struct nouveau_object *parent,
>         int ret, i;
>         u64 addr;
>
> +       /*
> +        * There are a lot of places that allocate multiples of 1000,
> +        * but do not set alignment correctly and still require this
> +        * alignment implicitly or explicitly.
> +        */
> +       if (size >= 0x1000 && align < 0x1000)
> +               align = 0x1000;
> +
>         *pobject = NULL;
>
>         if (pargpu) {
> --
> 1.8.3.4
>
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel


More information about the dri-devel mailing list