Re: Revert "drm/i915: Enable GMBUS for post-gen2 chipsets"

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Jun 11, 2011 at 10:58 PM, Jean Delvare <khali@xxxxxxxxxxxx> wrote:
> Hi Florian,
>
> On Sat, 11 Jun 2011 13:28:15 +0200, Florian Mickler wrote:
>> On Sat, 04 Jun 2011 19:34:56 -0000
>> Jean Delvare <khali@xxxxxxxxxxxx> wrote:
>>
>> > Revert commit 8f9a3f9b63b8cd3f03be9dc53533f90bd4120e5f. This fixes a
>> > hang when loading the eeprom driver (see bug #35572.) GMBUS will be
>> > re-enabled later, differently.
>> >
>> > Signed-off-by: Jean Delvare <khali@xxxxxxxxxxxx>
>> > Reported-by: Marek Otahal <markotahal@xxxxxxxxx>
>> > Tested-by: Yermandu Patapitafious <yermandu.dev@xxxxxxxxx>
>> > Tested-by: Andrew Lutomirski <luto@xxxxxxx>
>> > Acked-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
>> > Cc: David Airlie <airlied@xxxxxxxx>
>>
>> is this[1] resolved some other way in the meantime?
>>
>> Regards,
>> Flo
>>
>> [1]: https://bugzilla.kernel.org/show_bug.cgi?id=35572
>
> Not that I know of (and I don't see any other way at least for 2.6.39.)
> This is a shame, really, my revert patch should have been applied
> several days ago already.
>
> Keith, Chris, David, can you please get it rolling? This is a
> regression presumably affecting a lot of users, we should really fix it
> quickly, both in 2.6.39.x and 3.0-rc.

This patch really had no info other than the bug link to tell me wtf its doing,

I actually don't think reverting this is the correct fix, since it looks like
the code path thats screws with the mutex still happens on GEN2 machines
which unlucky for them.

Chris, also I don't see an ack anywhere on the list, only some
discussion in the bug,
I suspect the correct fix is to remove all the offending code instead
of just putting back the piece
of plaster that fell out of the wall.

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


[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux