Prev: [PATCH 0/3] Removing dead code
Next: [PATCH 1/3] sound/oss: Remove dead CONFIG_SOFTOSS*
From: Jiri Slaby on 22 Jul 2010 10:00
On 07/22/2010 11:19 AM, Jin, Gordon wrote:
> Chris Wilson wrote on Wednesday, July 21, 2010 9:37 PM:
>> On Wed, 21 Jul 2010 14:35:22 +0200, Tim Gardner <tcanonical(a)tpi.com>
>> wrote:
>>> What's up with this patch? I haven't received any comment, nor has it
>>> appeared in your git repository.
>>
>> Dave hasn't taken it because it is a regression fix for i915 and the
>> current procedure is for those to go through Eric's tree. Eric has
>> been busy working on a new compiler for GLSL the last few weeks and
>> has been applying patches in batches.
>>
>> The patch I am happy to ack, but since I introduced the regression
>> [https://bugzilla.kernel.org/show_bug.cgi?id=16294], I'd rather have
>> someone more familiar with the intricacies of G33/G35/Q35 review it.
>>
>> For what is worth:
>> Acked-by: Chris Wilson <chris(a)chris-wilson.co.uk>
>
> This patch fixes the bug on our G33 (0x29b2).

FWIW works for me too.

And as this is present as of 35-rc1 upstream, this means almost 2 months
lag since the break till now. Could you guys sort this up ASAP, please?

thanks,
--
js
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
From: Jin, Gordon on 22 Jul 2010 20:10
Jin, Gordon wrote on Thursday, July 22, 2010 5:19 PM:
> Chris Wilson wrote on Wednesday, July 21, 2010 9:37 PM:
>> On Wed, 21 Jul 2010 14:35:22 +0200, Tim Gardner <tcanonical(a)tpi.com>
>> wrote:
>>> What's up with this patch? I haven't received any comment, nor has
>>> it appeared in your git repository.
>>
>> Dave hasn't taken it because it is a regression fix for i915 and the
>> current procedure is for those to go through Eric's tree. Eric has
>> been busy working on a new compiler for GLSL the last few weeks and
>> has been applying patches in batches.
>>
>> The patch I am happy to ack, but since I introduced the regression
>> [https://bugzilla.kernel.org/show_bug.cgi?id=16294], I'd rather have
>> someone more familiar with the intricacies of G33/G35/Q35 review it.
>>
>> For what is worth:
>> Acked-by: Chris Wilson <chris(a)chris-wilson.co.uk>
>
> This patch fixes the bug on our G33 (0x29b2).
> Tested-by: Xudong Hao <xudong.hao(a)intel.com>

Sorry, the pci id for above G33 is actually 0x29c2.

And we also validated the patch on Q35 (0x29b2):
Tested-by: Xun Fang <xunx.fang(a)intel.com>

-Gordon
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
First  |  Prev  | 
Pages: 1 2
Prev: [PATCH 0/3] Removing dead code
Next: [PATCH 1/3] sound/oss: Remove dead CONFIG_SOFTOSS*