Hi Carl,

I have a few things.

1) Please apply the first attached patch, which fixes is a build
failure caused by one of the recent fixes in the branch. (that driver
is not built by default)

2) Please cherry-pick this. It's a fix that should have been nominated
but wasn't.

commit c8011c1885003b79c9f0c6530e46ae6cb0e69575
Author: Christian König <christian.koe...@amd.com>
Date:   Wed Jul 9 14:38:06 2014 +0200

    radeonsi: fix order of r600_need_dma_space and r600_context_bo_reloc

3) Please cherry-pick this too. It's required for and fixes R9 290X
(Hawaii) despite what the commit message says.

commit 315f3c171d423e13069beb99a6b772726a141865
Author: Marek Olšák <marek.ol...@amd.com>
Date:   Wed Apr 30 01:03:40 2014 +0200

    radeonsi: use DRAW_PREAMBLE on CIK


I've only been testing some apps to see if R9 290X isn't totally
broken (and it's not, which is great).

Marek




On Thu, Jul 31, 2014 at 7:39 AM, Carl Worth <cwo...@cworth.org> wrote:
> I've just pushed some updates to the 10.2 branch in preparation for an
> upcoming stable release. The patches that are queued (as well as those
> that have been nominated but aren't yet on master) can be seen here:
>
>         http://cworth.org/~cworth/mesa-stable-queue/
>
> As always, I would really appreciate any testing or feedback that people
> can give on this branch. Before pushing the branch, I tested that piglit
> shows no regressions with the i965 driver on Haswell. I'm still testing
> swrast (both DRI and gallium versions).
>
> I was a couple of days late pushing out this branch, so I'll delay the
> actual release by one day. I'll plan to push that out on Saturday, not
> Friday like I normally would.
>
> The list of patches nominated-awaiting-review is getting a bit large at
> this point, (most of it in 3 large patch series from Anuj, Ian, and
> Emil). Roughly half of those patches were first sent to the list over a
> month ago, (and almost all of these long outstanding patches are in Anuj
> and Ian's patch series).
>
> Additional review for outstanding patches would be most appreciated.[*]
>
> -Carl
>
> [*] Of course, I appreciate review for mediocre patched as well.
>
> --
> carl.d.wo...@intel.com
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
From 8920597c49bab8595e79abc81ee26ffb106540b2 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marek=20Ol=C5=A1=C3=A1k?= <marek.ol...@amd.com>
Date: Thu, 31 Jul 2014 20:04:42 +0200
Subject: [PATCH] radeonsi: fix build because of lack of draw_indirect
 infrastructure in 10.2

---
 src/gallium/drivers/radeonsi/si_state_draw.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/gallium/drivers/radeonsi/si_state_draw.c b/src/gallium/drivers/radeonsi/si_state_draw.c
index 33e8e2a..ec144a5 100644
--- a/src/gallium/drivers/radeonsi/si_state_draw.c
+++ b/src/gallium/drivers/radeonsi/si_state_draw.c
@@ -433,8 +433,7 @@ static bool si_update_draw_info_state(struct si_context *sctx,
 		 * We don't know that for indirect drawing, so treat it as
 		 * always problematic. */
 		if (sctx->b.family == CHIP_HAWAII &&
-		    (info->indirect ||
-		     (info->instance_count > 1 &&
+		    ((info->instance_count > 1 &&
 		      u_prims_for_vertices(info->mode, info->count) < primgroup_size))) {
 			wd_switch_on_eop = true;
 			ia_switch_on_eop = true;
-- 
1.9.1

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to