From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754746Ab2GIXlF (ORCPT ); Mon, 9 Jul 2012 19:41:05 -0400 Received: from perches-mx.perches.com ([206.117.179.246]:50887 "EHLO labridge.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752091Ab2GIXlD (ORCPT ); Mon, 9 Jul 2012 19:41:03 -0400 Message-ID: <1341877262.6118.73.camel@joe2Laptop> Subject: Re: [PATCH v3] printk: Have printk() never buffer its data From: Joe Perches To: Kay Sievers Cc: Michael Neuling , linuxppc-dev@ozlabs.org, Linus Torvalds , Greg Kroah-Hartman , Andrew Morton , Steven Rostedt , LKML , Ingo Molnar , Wu Fengguang , "Paul E. McKenney" Date: Mon, 09 Jul 2012 16:41:02 -0700 In-Reply-To: <1341876736.6118.70.camel@joe2Laptop> References: <1340651142.7037.2.camel@gandalf.stny.rr.com> <20120625150722.8cd4f45d.akpm@linux-foundation.org> <20120625235531.GB3652@kroah.com> <20120626002307.GA4389@kroah.com> <1340726856.977.6.camel@mop> <22482.1341471787@neuling.org> <16043.1341483625@neuling.org> <1341492628.834.3.camel@mop> <30233.1341535279@neuling.org> <29711.1341545979@neuling.org> <30343.1341546469@neuling.org> <21892.1341608647@neuling.org> <1341770155.1011.27.camel@mop> <1341870165.6118.42.camel@joe2Laptop> <1341872984.6118.54.camel@joe2Laptop> <1341876736.6118.70.camel@joe2Laptop> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.2.2- Content-Transfer-Encoding: 7bit Mime-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 2012-07-09 at 16:32 -0700, Joe Perches wrote: > Then you've changed semantics and I think you need to > fix it. > > A dev_ call is not guaranteed to be a complete > message. > > There are dev_ and netdev_ calls > followed by pr_cont. > > Maybe these could be fixed up and then they could be > always integral. There don't look to be too many. > > This may be most (all?) of them: Nah, there's a bunch more: $ git grep -E -A10 "\b(netdev|dev)_(info|warn|notice|err|alert|emerg|crit)" drivers | \ grep -B10 -E "\bprintk\s*\(\s*(KERN_CONT|)\s*\"" All of them could be fixed up though.