From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754442Ab1CYHnt (ORCPT ); Fri, 25 Mar 2011 03:43:49 -0400 Received: from darkcity.gna.ch ([195.226.6.51]:33109 "EHLO mail.gna.ch" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752952Ab1CYHnr convert rfc822-to-8bit (ORCPT ); Fri, 25 Mar 2011 03:43:47 -0400 X-Amavis-Alert: BAD HEADER SECTION, Improper folded header field made up entirely of whitespace (char 09 hex): Face: ...MWASAkVVViQjzP\n jycPrvgA\n\t\n R1goSzOnkp14Y[...] Subject: Re: [git pull] drm fixes From: Michel =?ISO-8859-1?Q?D=E4nzer?= To: Dave Airlie Cc: Linus Torvalds , linux-kernel@vger.kernel.org, DRI mailing list In-Reply-To: References: <1300864998.3522.71.camel@thor.local> <1300868532.3522.81.camel@thor.local> <1300880747.16522.13.camel@thor.local> Face: iVBORw0KGgoAAAANSUhEUgAAADAAAAAwBAMAAAClLOS0AAAAAXNSR0IArs4c6QAAADBQTFRFDg4OHh4eLCwsOzs7S0tLWlpaa2treXl5hISEjY2NmJiYqKiotLS0xsbG1dXV/Pz81CO0SQAAArtJREFUOMtd1M9P01AcAHCI/4AtGq/QDfDHRfraEX8eaNeJFw1rO/DCYet7mxc1ZG0x3sStHQkmZpqtHDwAi+tMiFEzbZdwNWEJR48cjPG4g5HhELUbrHvjpYe2n7zvt++977cD/7rjsCry8uNG93Gge9OKUyAAgLB1AlpTZICmAzR15QTEiQAPAKADYLMPfhNnEJR4HvD0tT5YI2KGUcyqihQN7mDwZ3hMN4q2N4ol+gEGTSLWhorrjYXrGPwc0jTDOoKP4xi8G0W6adl2Gz6zGDwag5p5PMON7vZgJuSB976+3U6y2QdeKNet1+uum9/qwVQHvEjtKesY0EIb7CNYe+7DIRXCID/vQ4tksVAY7JFBD7yvqrWTL93xoUmOQsPIddbnuk8v+bBPsigB2KRlFxS4nL/owwEpKBSg2MU3UcDf+nATyyHEQwrHzJZFNpXeuOHDC0qW4sMhEHESFGOUrvgQpWUYFVNQdjQxca8abnSB55CmehdcLSxa1ifoQ4JBpmGYWbhsly3X0fxQ7xmkW3Y5CztLcXI+fAu2oWho3nbV6s5rH35xSC/aBR2tOpVa/Utv25tcTDPL6aT21kG17WrvaFtMBJmFhJCsVF4uu9VG76DWBaRnEiNs7pU659pYlfwtQSRy9GCYlwR7C6/dPQgBw3MsTPNWA4d9SeMDDC9JYdnqq/amdF+diGnVhXFztQ/2lJSWjulOxjRX+uC7EkOqhLRk2ejrqHVBEqCqJLO5cmEXgx8TrBiWVQh1u2DhzQlPsyIveU2YLGorGBxODoR5notlpcUieoLB1/NEmGc4AalGJpLe8WF/8txMWASAkVVViQjzP jycPrvgA R1goSzOnkp14YCYHsp7QJHAS5QcXDqG1jBxdSITVgBNkBTFloj88Q/gMkFcuItYiQPUCBGc2xh5drsD/wGZrgsgDOE4ZAAAAABJRU5ErkJggg== Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 8BIT Date: Fri, 25 Mar 2011 08:43:30 +0100 Message-ID: <1301039010.12159.56.camel@thor.local> Mime-Version: 1.0 X-Mailer: Evolution 2.32.2 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fre, 2011-03-25 at 17:21 +1000, Dave Airlie wrote: > On Fri, Mar 25, 2011 at 10:17 AM, Linus Torvalds > wrote: > > On Thu, Mar 24, 2011 at 5:07 PM, Dave Airlie wrote: > >> > >> Like seriously you really think VFS locking rework wasn't under > >> development or discussion when you merged it? I'm sure Al would have > >> something to say about it considering the number of times he cursed in > >> irc about that code after you merged it. > > > > Umm. That code was basically over a year old by the time it was merged. > > > > How old was the code we're talking about now? Seriously? > > It was 30 lines of clean code, that really was fine to be merged in > its first form it was merely a future maintaince issue to clean up the > interface before it was released as stable. >>From my POV the real failure here was that the change made it to *any* tree while there were outstanding review issues from when it was initially discussed a few weeks earlier. Then when the change was submitted — more or less unchanged — I was on my birthday weekend enjoying some time away from computers, and when I had caught up with things, it was already in drm-next. > In this case, if you had a >2 monitor setup connected to an evergreen > card, and you tried to do 3D on the 3rd monitor it would just hang the > app in a loop forever, the fix needs 3 pieces, one in the kernel, and > two userspace fixes. Actually, the hangs could be fixed in the X driver alone, but the author seems uninterested in contemplating that. Maybe because he seems to think it's easier to get the kernel fix to users, but I'm with you on that it's quite clearly the opposite. That said, I agree with your analysis in general, but not in this particular case. -- Earthling Michel Dänzer | http://www.vmware.com Libre software enthusiast | Debian, X and DRI developer