registry team mailing list archive

[Uwe Geuder <ubuntuLp-ugeuder@xxxxxxxxxxxxxx>]

As already mentioned in
https://bugs.freedesktop.org/show_bug.cgi?id=29367#c5 the patch in
https://bugs.freedesktop.org/show_bug.cgi?id=21374#c5 does not apply to
current mesa. I searched a bit in the git history and it applies cleanly
e. g. to commit 57f55a. However, this commit is 150 commits apart from
today (and that is only counting files named r200 and radeon_common,
there might be others less obvious ones) So I assume that that old patch
is the wrong approach for us today.

So let's start from a different angle.

The assertion does not occur on Maverick. Maverick seems to based on
Mesa 7.7.1 (according to the source naming, I haven't checked the
contents). And 7.7.1 is much newer than the old patch. So the issue was
probably corrected one way or the other.

If we look at the version history of the r200 source in git, we see that
7.7.1 was made in a side branch. The attached screen dump was produced
with the command

$ gitk --all src/mesa/drivers/dri/r200/*
src/mesa/drivers/dri/radeon/radeon_common.?

The side branch ends in gitk row 70 (the currently selected one), which
precedes mesa_7_7_1_rc2, which in turn precedes mesa_7_7_1

So I could guess that one of the changes in the sidebranch actually
fixes the assertion but it has not been applied (or applied incorrectly)
to the "trunk" which lead towards Maverick. At least nothing has been
visibly merged as we can easily see from gitk.

Now we are "only" some 70-80 commits away from today. Still that will
require reading some source, and from what I've seen during my
investigations so far I have some doubts whether anybody not familiar
with the code can do it in reasonable time. I'm not sure when I have
time to investigate the source further. I might be very busy the coming
weeks.

Maybe you Tormod could report my findings so far upstream. (I don't have
an account in freedesktop and now it's not the right time for me to get
involved there.)


** Bug watch added: freedesktop.org Bugzilla #21374
   http://bugs.freedesktop.org/show_bug.cgi?id=21374

** Attachment added: "gitk screenshot showing the history of r200 related source"
   https://bugs.launchpad.net/ubuntu/+source/mesa/+bug/656100/+attachment/1680757/+files/mesa-r200-history.png

-- 
[R200] ../../radeon/radeon_cs_gem.c:181: cs_gem_write_reloc: Assertion `boi->space_accounted' failed.
https://bugs.launchpad.net/bugs/656100
You received this bug notification because you are a member of Registry
Administrators, which is the registrant for Mesa.