Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752643AbZAKLqQ (ORCPT ); Sun, 11 Jan 2009 06:46:16 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751308AbZAKLp5 (ORCPT ); Sun, 11 Jan 2009 06:45:57 -0500 Received: from fg-out-1718.google.com ([72.14.220.153]:34138 "EHLO fg-out-1718.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750938AbZAKLp4 (ORCPT ); Sun, 11 Jan 2009 06:45:56 -0500 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=date:from:to:cc:subject:message-id:references:mime-version :content-type:content-disposition:in-reply-to:user-agent; b=bHQSP2vXzUeR1cwtWegiigZSrY8Xbzud/WCUsTLD+pSVQ4lJEMU+CiJhRbZZUZJzNu GM5KRDfHU1/zrcIzZiZcti1KgU1+jR1mgapF8P5qZ3lCEVqNUGodihZiG1Z1XiaReCpE 2LdrynZ8xjDsCso7qQvPLIEMnH9JWnie0e2V8= Date: Sun, 11 Jan 2009 14:45:53 +0300 From: Cyrill Gorcunov To: Paul Mackerras Cc: David Miller , akpm@linux-foundation.org, jchapman@katalix.com, linux-kernel@vger.kernel.org, netdev@vger.kernel.org Subject: Re: [PATCH] net: ppp_generic - fix broken userspace Message-ID: <20090111114553.GA19261@localhost> References: <20090110202802.GF29349@localhost> <20090110203112.GG29349@localhost> <18793.15831.279207.252421@cargo.ozlabs.ibm.com> <20090110.233503.222465267.davem@davemloft.net> <18793.51741.28374.460500@cargo.ozlabs.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <18793.51741.28374.460500@cargo.ozlabs.ibm.com> User-Agent: Mutt/1.5.17+20080114 (2008-01-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1267 Lines: 37 [Paul Mackerras - Sun, Jan 11, 2009 at 09:29:49PM +1100] | David Miller writes: | | > PPP was converted to use IDR instead of PPP's duplicate | > implementation. | > | > A bug was introduced in that conversion, and that bug is being fixed | > here. | > | > What is not to like? :-) | | Ah OK, that sounds all right. I was reacting to the "broken | userspace" in the patch description which at first glance seemed to be | saying that userspace (i.e. pppd :) is doing something which has | recently been declared to be wrong and therefore it is "broken". | | I think "fix broken userspace" as a patch headline is quite | ambiguous. Something like "Fix bug in conversion to use IDR" would be | more informative. | | Paul. | David, if you pick this patch could you rename the subject then as Paul proposed? Or I should resend the patch with new title? To Paul: my bad, sorry. I thought like -- I broke user-space pppd so it's broken now and lets fix it -- here is "fix broken userspace" grown from :) - Cyrill - -- 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/