All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alex Deucher <alexdeucher@gmail.com>
To: Andy Furniss <andyqos@ukfsn.org>
Cc: dri-devel@lists.freedesktop.org
Subject: Re: [PATCH] drm/radeon/kms: r600 CS parser fixes
Date: Wed, 8 Sep 2010 18:37:47 -0400	[thread overview]
Message-ID: <AANLkTimLJdoSW5Ev0GHriZrT_f2HYmMw6sL8ALZ__XD7@mail.gmail.com> (raw)
In-Reply-To: <4C652096.7020705@ukfsn.org>

On Fri, Aug 13, 2010 at 6:38 AM, Andy Furniss <andyqos@ukfsn.org> wrote:
> Jon Sturm wrote:
>
>> ut2004 has been having issues for a while so I wouldn't blame this
>> patch 100%, Then again my issues seem to be similar to
>> https://bugs.freedesktop.org/show_bug.cgi?id=27443 which may or may
>> not be related.
>
> Only having the demo and not seriously playing all levels (or much at all) I
> haven't hit that one.
>
> This one does seem to be down to the patch. If you have the full game it
> could be things are different, as it's actually the nvidia ad that the demo
> starts with the provokes this. Doom3 demo I can get into a saved game, but
> it dies with the same error after about 10 secs.
>
> Both work with the same d-r-t without the patch.

Does this patch help?

diff --git a/drivers/gpu/drm/radeon/r600_cs.c b/drivers/gpu/drm/radeon/r600_cs.c
index d886494..e83fc88 100644
--- a/drivers/gpu/drm/radeon/r600_cs.c
+++ b/drivers/gpu/drm/radeon/r600_cs.c
@@ -1046,7 +1046,6 @@ static void r600_texture_size(unsigned nfaces,
unsigned blevel, unsigned nlevels
                        rowstride = ALIGN((width * bpe), pitch_align);
                        size = height * rowstride * depth;
                        offset += size;
-                       offset = (offset + 0x1f) & ~0x1f;
                }
        }
        *l0_size = ALIGN((w0 * bpe), pitch_align) * h0 * d0;


Alex

> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
>

  parent reply	other threads:[~2010-09-08 22:37 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-08-06  6:54 [PATCH] drm/radeon/kms: r600 CS parser fixes Alex Deucher
2010-08-10 15:14 ` Andy Furniss
2010-08-13  2:56   ` Jon Sturm
2010-08-13 10:38     ` Andy Furniss
2010-08-17 21:24       ` Alex Deucher
2010-08-18 11:39         ` Andy Furniss
2010-08-18 11:45           ` Andy Furniss
2010-08-18 14:07             ` Alex Deucher
2010-08-18 16:05               ` Andy Furniss
2010-09-08 22:37       ` Alex Deucher [this message]
2010-09-09 10:27         ` Andy Furniss

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=AANLkTimLJdoSW5Ev0GHriZrT_f2HYmMw6sL8ALZ__XD7@mail.gmail.com \
    --to=alexdeucher@gmail.com \
    --cc=andyqos@ukfsn.org \
    --cc=dri-devel@lists.freedesktop.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.