Re: [PATCH 1/8] drm/radeon: fix VA range check

From: Jerome Glisse <j.glisse@gmail.com>
To: "Christian König" <deathsimple@vodafone.de>
Cc: dri-devel@lists.freedesktop.org
Subject: Re: [PATCH 1/8] drm/radeon: fix VA range check
Date: Tue, 11 Sep 2012 12:15:30 -0400	[thread overview]
Message-ID: <CAH3drwZNS8C7yzaR_nvq8-J_bPGxj=3hFaN=w0-atNKMundoDg@mail.gmail.com> (raw)
In-Reply-To: <1347372604-26557-1-git-send-email-deathsimple@vodafone.de>

On Tue, Sep 11, 2012 at 10:09 AM, Christian König
<deathsimple@vodafone.de> wrote:
> The end offset is exclusive not inclusive.
>
> Signed-off-by: Christian König <deathsimple@vodafone.de>

Reviewed-by: Jerome Glisse <jglisse@redhat.com>

> ---
>  drivers/gpu/drm/radeon/radeon_gart.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/radeon/radeon_gart.c b/drivers/gpu/drm/radeon/radeon_gart.c
> index d7bd46b..614e31a 100644
> --- a/drivers/gpu/drm/radeon/radeon_gart.c
> +++ b/drivers/gpu/drm/radeon/radeon_gart.c
> @@ -732,7 +732,7 @@ int radeon_vm_bo_add(struct radeon_device *rdev,
>         head = &vm->va;
>         last_offset = 0;
>         list_for_each_entry(tmp, &vm->va, vm_list) {
> -               if (bo_va->soffset >= last_offset && bo_va->eoffset < tmp->soffset) {
> +               if (bo_va->soffset >= last_offset && bo_va->eoffset <= tmp->soffset) {
>                         /* bo can be added before this one */
>                         break;
>                 }
> --
> 1.7.9.5
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel