Received: by 2002:ad5:474a:0:0:0:0:0 with SMTP id i10csp514493imu; Wed, 9 Jan 2019 01:32:41 -0800 (PST) X-Google-Smtp-Source: ALg8bN6xFKjBTbqT9w8WwcKbZA0x271A1PS+Ra/9UwTIxmhfuSlwfuLJsDIFKATcJoHlGGPgeiyP X-Received: by 2002:a63:4e41:: with SMTP id o1mr4824035pgl.282.1547026361557; Wed, 09 Jan 2019 01:32:41 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1547026361; cv=none; d=google.com; s=arc-20160816; b=QheOPbStUN0unObazfnQLDMSch+YaSVFS538QYEFPCtqUFwvHM2grQ0ebvQO6FgNra 9DhOIW3ojyqw83s/0a/tLNf94NCAmESPj8K4jq6FQSTnB8t78mwW9/7lvwc2Mfk1D1Zf KENr2v7rGAitkt8WRHpEzoouV0pgOa0XHneAVKGfFLrYwLVa80OQmIbt+OnQEeI3oCQg +vvD0mooQupwGT4KqhSuI1w1xarsQnhaMwNTq2W6xOchPoDiu/Z7RssZJI6kkjdjRQrT 2xesYevA5SpuNdf5jZyEqo7n4R4I0lYLDbL+voYqYKHRG2ttW3SW0zXufH09hDMKSGbS z3jA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:mail-followup-to :message-id:subject:cc:to:from:date:dkim-signature; bh=9bt6318KuI4Mb3GDgutajPT/Cvnzguj3ZEGYUBvG1Ss=; b=ndgFApAvbCXDS9MVazee3hdHt+CRpANP+PDAFXkrg/XcHd9jyvAT+abngZoUxpolu3 fQFppTisg8j11Df9l/OkFfIhQlzwkWky6P5RRX2ZvCOudHyKxWh9Nt5elgoqck08s+CY KglZkYmGRzNGcdOuA4x5rzZZVOxfRGxBqAier71AtXqJZM3Z9QuP+7X/RkbsJKiQNxbc fWD7gYJo0JMsCoQ/KQq00AFEb4+Dg2FTtB0+1eX/hfMQLorXiapF3mejZrLqjQgZuPeC mzuxbVFgAsOYS5xJk1+gjyEOFM+/+NPF3CYoQGIuL09rgbyakcXgoO7QFLcXSCOyBX2Y O2iA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=NcnCwQlv; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id g22si78206945pfj.222.2019.01.09.01.32.26; Wed, 09 Jan 2019 01:32:41 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=NcnCwQlv; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1730091AbfAIJaV (ORCPT + 99 others); Wed, 9 Jan 2019 04:30:21 -0500 Received: from mail-ed1-f68.google.com ([209.85.208.68]:39971 "EHLO mail-ed1-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728041AbfAIJaV (ORCPT ); Wed, 9 Jan 2019 04:30:21 -0500 Received: by mail-ed1-f68.google.com with SMTP id g22so6699235edr.7 for ; Wed, 09 Jan 2019 01:30:20 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=date:from:to:cc:subject:message-id:mail-followup-to:references :mime-version:content-disposition:in-reply-to:user-agent; bh=9bt6318KuI4Mb3GDgutajPT/Cvnzguj3ZEGYUBvG1Ss=; b=NcnCwQlvTOMcQ3YvE48Ay6kxdnx/Eqlmsr7L7YX0p9mfk4yx2MDNdrjkIbU6MLMFM+ 67jh7pZSJHhiiES6Y6NUezlrfKYV9d0UIMzrKoLoit84nBbTILHKqufHNhgOYwT7IOJ3 9gX12OZreu1gXhHzqnRmZ9jmWcWZMjDZmHQSD6Oh6p2mczMpIgAhJwDd5olZxVp+i3cs ujRgmGKZtNd2rt7DWLG/x4xIIE7KjF/5D+dMaYD78IwNMHuSpRUTAj+EPBNhB0kNoC8n HRSpKlxIBZs0RypA5raxqm4+BKOjRiQ0uyizV8I8faoMumGbnJ0qeIu4OPqgUP1LfKsq Y7iA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id :mail-followup-to:references:mime-version:content-disposition :in-reply-to:user-agent; bh=9bt6318KuI4Mb3GDgutajPT/Cvnzguj3ZEGYUBvG1Ss=; b=A4UJ5WEE5CHK2wIU6EuLqM8XA9Y8ag47o1/T31yu5YUc1lhz6uTq0WImmqVCjFR2te fA4iL/0zwk9dw/iCMvL4S/mdQpJd58ZTLxS7rv9l3F06k2hzCU5F9kgy9vxY91JbVSr7 e7nOAPLFgI2LqYZcwJ8HBMv9IMB2TIOgwSfHii8zdCH1LoGasKbU/m1p+lgMM+XmjWHu QXuHA+/6Iim1e17I0gaWLs/jJhfBnAE2iq4sGuZwyCHSU1KVOwd8SVTF8bwHiyoDky72 q8+xQA2W7z29dmeJKygH7lJaZdGk5zu1hmH3qLgbEBw+draxrbhfnwXazsDK6RSh2ilI dBzQ== X-Gm-Message-State: AJcUukfwLAhRn5X8/X0qno3Rh2Z2G4Gv8XDwF+nQpicSt5b1jqe3+DbH M2uORBYwRHhnbcvwDqztHK05wj9mbNs= X-Received: by 2002:a17:906:474e:: with SMTP id j14-v6mr4823979ejs.55.1547026219498; Wed, 09 Jan 2019 01:30:19 -0800 (PST) Received: from yubo-2 ([191.101.9.198]) by smtp.gmail.com with ESMTPSA id h51sm1297205edh.17.2019.01.09.01.30.17 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 09 Jan 2019 01:30:19 -0800 (PST) Date: Wed, 9 Jan 2019 04:30:16 -0500 From: YU Bo To: "Rafael J. Wysocki" Cc: Greg Kroah-Hartman , Linux Kernel Mailing List , Joe Perches , yuzibode@126.com Subject: Re: [PATCH V5 2/2] kobject: drop newline from msg string Message-ID: <20190109093014.cjheviy67szigzfg@yubo-2> Mail-Followup-To: "Rafael J. Wysocki" , Greg Kroah-Hartman , Linux Kernel Mailing List , Joe Perches , yuzibode@126.com References: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Disposition: inline In-Reply-To: User-Agent: NeoMutt/20180512-62-03f98a Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Jan 09, 2019 at 10:21:37AM +0100, Rafael J. Wysocki wrote: >On Wed, Jan 9, 2019 at 10:17 AM Bo YU wrote: >> >> There is currently a missing terminating newline in non-switch case >> match when msg == NULL >> >> Signed-off-by: Bo YU > >I have given you an R-by for this one already, so why haven't you added it here? Oh, i got it.This is the first time i encountered the circumstances. > >Reviewed-by: Rafael J. Wysocki Thank you :) > >> --- >> Changes in V4: >> Recovery originly declaration, requested by Greg. >> >> Change in V3: >> Improve the commit log, requested by Rafael. >> >> Changes in V2: >> Drop newline from msg according to Joe's suggestion. >> --- >> lib/kobject_uevent.c | 6 +++--- >> 1 file changed, 3 insertions(+), 3 deletions(-) >> >> diff --git a/lib/kobject_uevent.c b/lib/kobject_uevent.c >> index c87a96c4800e..f05802687ba4 100644 >> --- a/lib/kobject_uevent.c >> +++ b/lib/kobject_uevent.c >> @@ -200,7 +200,7 @@ int kobject_synth_uevent(struct kobject *kobj, const char *buf, size_t count) >> >> r = kobject_action_type(buf, count, &action, &action_args); >> if (r) { >> - msg = "unknown uevent action string\n"; >> + msg = "unknown uevent action string"; >> goto out; >> } >> >> @@ -212,7 +212,7 @@ int kobject_synth_uevent(struct kobject *kobj, const char *buf, size_t count) >> r = kobject_action_args(action_args, >> count - (action_args - buf), &env); >> if (r == -EINVAL) { >> - msg = "incorrect uevent action arguments\n"; >> + msg = "incorrect uevent action arguments"; >> goto out; >> } >> >> @@ -224,7 +224,7 @@ int kobject_synth_uevent(struct kobject *kobj, const char *buf, size_t count) >> out: >> if (r) { >> devpath = kobject_get_path(kobj, GFP_KERNEL); >> - pr_warn("synth uevent: %s: %s", >> + pr_warn("synth uevent: %s: %s\n", >> devpath ?: "unknown device", >> msg ?: "failed to send uevent"); >> kfree(devpath); >> -- >> 2.11.0 >>