From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.2 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id BE050C32755 for ; Mon, 23 Sep 2019 13:36:02 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 97F3F20673 for ; Mon, 23 Sep 2019 13:36:02 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2439791AbfIWNgB (ORCPT ); Mon, 23 Sep 2019 09:36:01 -0400 Received: from www62.your-server.de ([213.133.104.62]:50092 "EHLO www62.your-server.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1730669AbfIWNgA (ORCPT ); Mon, 23 Sep 2019 09:36:00 -0400 Received: from [178.197.248.15] (helo=localhost) by www62.your-server.de with esmtpsa (TLSv1.2:DHE-RSA-AES256-GCM-SHA384:256) (Exim 4.89_1) (envelope-from ) id 1iCOVK-0004dR-Id; Mon, 23 Sep 2019 15:35:50 +0200 Date: Mon, 23 Sep 2019 15:35:50 +0200 From: Daniel Borkmann To: Petr Mladek Cc: Linus Torvalds , Andrii Nakryiko , Arnd Bergmann , Andy Whitcroft , "David S. Miller" , Martin KaFai Lau , Song Liu , Yonghong Song , Sergey Senozhatsky , Kefeng Wang , Peter Zijlstra , Alexei Starovoitov , Andrew Morton , Greg Kroah-Hartman , Joe Perches , Arnaldo Carvalho de Melo , Ingo Molnar , bpf , open list Subject: Re: [PATCH 30/32] tools lib bpf: Renaming pr_warning to pr_warn Message-ID: <20190923133550.GA9880@pc-63.home> References: <20190920062544.180997-1-wangkefeng.wang@huawei.com> <20190920062544.180997-31-wangkefeng.wang@huawei.com> <20190923082039.GA2530@pc-63.home> <20190923110306.hrgeqwo5ogd55vfo@pathway.suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190923110306.hrgeqwo5ogd55vfo@pathway.suse.cz> User-Agent: Mutt/1.12.1 (2019-06-15) X-Authenticated-Sender: daniel@iogearbox.net X-Virus-Scanned: Clear (ClamAV 0.101.4/25581/Mon Sep 23 10:20:21 2019) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Sep 23, 2019 at 01:03:06PM +0200, Petr Mladek wrote: > On Mon 2019-09-23 10:20:39, Daniel Borkmann wrote: > > On Sun, Sep 22, 2019 at 02:07:21PM -0700, Andrii Nakryiko wrote: > > > On Fri, Sep 20, 2019 at 10:06 AM Kefeng Wang wrote: > > > > > > > > For kernel logging macro, pr_warning is completely removed and > > > > replaced by pr_warn, using pr_warn in tools lib bpf for symmetry > > > > to kernel logging macro, then we could drop pr_warning in the > > > > whole linux code. > > > > > > > > Signed-off-by: Kefeng Wang > > > > --- > > > > tools/lib/bpf/btf.c | 56 +-- > > > > tools/lib/bpf/btf_dump.c | 20 +- > > > > tools/lib/bpf/libbpf.c | 652 ++++++++++++++++---------------- > > > > tools/lib/bpf/libbpf_internal.h | 2 +- > > > > tools/lib/bpf/xsk.c | 4 +- > > > > 5 files changed, 363 insertions(+), 371 deletions(-) > > > > > > Thanks! This will allow to get rid of tons warnings from checkpatch.pl. > > > > > > Alexei, Daniel, can we take this through bpf-next tree once it's open? > > > > I'd be fine with that, in fact, it probably should be in order to avoid > > merge conflicts since pr_warn{ing}() is used all over the place in libbpf. > > The entire patchset modifies many files all over the tree. > This is from https://lkml.kernel.org/r/20190920062544.180997-1-wangkefeng.wang@huawei.com > > 120 files changed, 882 insertions(+), 927 deletions(-) > > Would it make sense to push everything at the end of the merge window > or for 5.4-rc2 after master settles down? If all over the tree it would probably make more sense for e.g. Andrew Morton to pick it up if there are no other objections, and try to merge it during mentioned time frame. Thanks, Daniel