All of lore.kernel.org
 help / color / mirror / Atom feed
From: bugzilla-daemon@freedesktop.org
To: dri-devel@lists.freedesktop.org
Subject: [Bug 22576] [KMS] mesa demo spectex broken on rv280
Date: Mon, 10 Dec 2012 16:37:52 +0000	[thread overview]
Message-ID: <bug-22576-502-cwuSxdDG3a@http.bugs.freedesktop.org/> (raw)
In-Reply-To: <bug-22576-502@http.bugs.freedesktop.org/>


[-- Attachment #1.1: Type: text/plain, Size: 1417 bytes --]

https://bugs.freedesktop.org/show_bug.cgi?id=22576

Roland Scheidegger <sroland@vmware.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #21 from Roland Scheidegger <sroland@vmware.com> ---
(In reply to comment #18)
>  I think i've found source of these lighting problems on r200 in this and
> other bugs. It is just a typo it seems in r200_state_init.c lit_emit()
> 
>  OUT_VEC(atom->cmd[LIT_CMD_1], atom->cmd+LIT_CMD_1+1);
> 
>  instead of OUT_VEC it needs to be OUT_SCL:
> 
>  OUT_SCL(atom->cmd[LIT_CMD_1], atom->cmd+LIT_CMD_1+1);

Should be fixed by 320d531373e7b0873f5de42f6173b986290f593f, thanks!
FWIW the command emit mechanism looks a bit too complicated it could profit
from some refactoring. For instance it is still based on the fake
drm_radeon_cmd_header_t structure, the scl vs. scl2 and vec vs. veclinear emits
are nothing but crude hacks around limitations in that structure, even though
since ums is gone there is absolutely no point in using that struct...
Section size calcs are also somewhat confusing, the sanity code (which was
really great back then) is totally unused etc.
Maybe another day...

-- 
You are receiving this mail because:
You are the assignee for the bug.

[-- Attachment #1.2: Type: text/html, Size: 2968 bytes --]

[-- Attachment #2: Type: text/plain, Size: 159 bytes --]

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

  parent reply	other threads:[~2012-12-10 16:37 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <bug-22576-502@http.bugs.freedesktop.org/>
2010-04-11 17:05 ` [Bug 22576] [KMS] mesa demo spectex broken on rv280 bugzilla-daemon
2010-04-20  5:51 ` bugzilla-daemon
2010-04-20 14:11 ` bugzilla-daemon
2010-04-22  0:48 ` bugzilla-daemon
2010-04-22  1:29 ` bugzilla-daemon
2010-04-22  1:38 ` bugzilla-daemon
2010-08-08 17:29 ` bugzilla-daemon
2010-09-13 18:45 ` bugzilla-daemon
2010-09-13 18:50 ` bugzilla-daemon
2010-10-12  8:20 ` bugzilla-daemon
2010-10-20  5:42 ` bugzilla-daemon
2010-10-20  5:43 ` bugzilla-daemon
2012-01-13 14:01 ` bugzilla-daemon
2012-12-09 23:11 ` bugzilla-daemon
2012-12-10 14:24 ` bugzilla-daemon
2012-12-10 15:28 ` bugzilla-daemon
2012-12-10 16:37 ` bugzilla-daemon [this message]
2012-12-18 17:42 ` bugzilla-daemon
2012-12-18 20:43 ` bugzilla-daemon

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=bug-22576-502-cwuSxdDG3a@http.bugs.freedesktop.org/ \
    --to=bugzilla-daemon@freedesktop.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.