Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1760753AbYGQTLm (ORCPT ); Thu, 17 Jul 2008 15:11:42 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1757839AbYGQTLd (ORCPT ); Thu, 17 Jul 2008 15:11:33 -0400 Received: from rv-out-0506.google.com ([209.85.198.237]:62407 "EHLO rv-out-0506.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754916AbYGQTLc (ORCPT ); Thu, 17 Jul 2008 15:11:32 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:sender:to:subject:cc:in-reply-to:mime-version :content-type:content-transfer-encoding:content-disposition :references:x-google-sender-auth; b=WdA9G2LNv+8Cuz6Uo8LU5Da++FWDxkzjYBK+9k4k6n7SkJF/xMVEsPmsYReso9Jvys 11BV/Hcc/VwMDUfp4qcHznpf//s0gj8c0rdKpP5mslgQkcHw4mPCwaPZrohAtTVtB38K s7TUXitrPlY46YIPP6d2Ki4DLJFFYM9ruiQ/k= Message-ID: <2c0942db0807171211s39e2653fwdc50f92d9c3020d2@mail.gmail.com> Date: Thu, 17 Jul 2008 12:11:32 -0700 From: "Ray Lee" To: "Andi Kleen" Subject: Re: Please pull ACPI updates Cc: "Linus Torvalds" , "Jesse Barnes" , "Rafael J. Wysocki" , torvalds@linuxfoundation.org, linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org In-Reply-To: <487F71E8.9050705@firstfloor.org> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <20080716214516.GA10777@basil.nowhere.org> <200807170011.12184.rjw@sisk.pl> <200807161633.01375.jbarnes@virtuousgeek.org> <487EEAEB.4050009@firstfloor.org> <487F71E8.9050705@firstfloor.org> X-Google-Sender-Auth: 9548a20c9a680ccf Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1945 Lines: 37 On Thu, Jul 17, 2008 at 9:23 AM, Andi Kleen wrote: > Linus Torvalds wrote: >> But I *really* hate pulling from somebody, and seeing commit dates that >> are from five minutes ago, and based on something that I had just pushed >> out (which was essentially the case for this round of ACPI changes). >> >> That literally shows that the code was hardly tested _at_all_ in that >> exact configuration. It may have gotten testing based on some earlier >> kernel version, but then it very clearly got rebased (or just quilt >> imported) on top of a totally new kernel base, and was not tested in that >> version very much if at all. > > Hmm, perhaps I'm thick, but for me the only difference seems to be > that the submitter does the merge that you would do (and safes > himself a yelling at if it doesn't build or crashes at boot or similar...). > > In both cases there's a "untested in that configuration" end configuration > in the end, but there's nothing much that can be done against it, > can't it? It matters to us end-testers when we do a git bisect. If you leave the history intact and let the merging happen at Linus' end (or, at least at merge time), then there is a point in history of your tree that someone (or git bisect) can check out and try, validating the patch, and therefore fingering a failed merge. It's the difference between having tested patches and an untested merge, or untested new patches and an untested merge. Your point is the end result is untested either way. The other way to look at it is *how much* untested history ends up in the tree. In Linus' version, just the point from the merge onward is untested. In your version, everything is new. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/