Received: by 2002:ac0:a5a6:0:0:0:0:0 with SMTP id m35-v6csp202517imm; Thu, 13 Sep 2018 18:56:44 -0700 (PDT) X-Google-Smtp-Source: ANB0VdYdfbZLHQzwnuVenXnpIj7/eG5dwtSZ1XOAuPTpJZPt5WcIzqqnALIH94e4yy6PG7zcORyf X-Received: by 2002:a65:520d:: with SMTP id o13-v6mr7851630pgp.282.1536890204004; Thu, 13 Sep 2018 18:56:44 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1536890203; cv=none; d=google.com; s=arc-20160816; b=my37vLbfn99d4q6VIwFoMBU+RA1omu5CewSKh3ivixVHtCTYCpP8RwyvR8trWlSCLs Qy62Y8KKGGUz43ECEiO9kBQu0pqM9aPwyLe+0mxqk4AxthuDkSi+mxkL1QcVAJBVN7TP g8PhXxhuNuYkeiBTapwe7cwBHhuTUzC+8S21YaQ0cYRx+117WtrXTiiFMHlHb0dP7ltw ltPWG/XvxFNN1i6qPMmdOgqcA/BZHiQfbHFoVbqCy3rw8B9etY8TgNvb6e2N+bWfDg7a XvoxSAmO5yf7WCyClWZL9hfbWyRRs/lnCWPjFo7JRe9gZF5XxRJBZOpROETX/7imz5tV /acw== 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:message-id:subject:cc :to:from:date:dkim-signature; bh=O4OKcV2sMFnGzGzL/9+Cpt1JG7IyKFHG/ze9KlVPzA0=; b=eD3Ue4CRRH5O/qgMbmzO2XKLCMXn5J3Dk5/hjA8/HnrcOv6B4hZwNhucwGdS28Siy+ JYgt4Q5xrM5oyfhLg/ylU9OK6r5VGgiqA28lW9O+IRRcZi2coz/9XZZbTFfTCgUwpXxQ GXKWn1Q+WNxvNvD5Kb9kSz1m88jlVZtQpPvhNpzG/9wd3Gar//5w02xYzSCMUEoASVvz KZ67upfPcWZELcZa+wcKuaDLk01JPocBEuEHRo9Zq0VLiagSbahvVo0FWknOP/mzrrJR hLDS6prYNmoj8nh769mZhm9Iw8UzNJCeUxM+uSnYUjMT1hUuQH3SiXHmWH5KPDjn5LXp BJHg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=d8Iuyid4; 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 v132-v6si5772284pfc.193.2018.09.13.18.56.28; Thu, 13 Sep 2018 18:56:43 -0700 (PDT) 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=d8Iuyid4; 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 S1728416AbeINHH1 (ORCPT + 99 others); Fri, 14 Sep 2018 03:07:27 -0400 Received: from mail-pl1-f193.google.com ([209.85.214.193]:33216 "EHLO mail-pl1-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726823AbeINHH0 (ORCPT ); Fri, 14 Sep 2018 03:07:26 -0400 Received: by mail-pl1-f193.google.com with SMTP id b30-v6so3442042pla.0 for ; Thu, 13 Sep 2018 18:55:21 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=O4OKcV2sMFnGzGzL/9+Cpt1JG7IyKFHG/ze9KlVPzA0=; b=d8Iuyid4O4UR0LnbyvNqzDW6kCvfrxb1hG0Q/YhEXX+4tkVd2hJP+894oZoMd1BLhH 9dI0ZdoI5Wbb0bep1m813xEuBOOrZA/YzCb+CUe7vkaxMyQW/Spsxq0ws1iE9iDFnWvs 28D62d2DMFbxniR0LFlTxq2idIWY2t4FIK3UWvDDalHwla3lHx2SG8N0TfJG6YwB/7nJ 4f9DP/OrH5cLnjMiVHEmQ3T7FdbN8Gd68n+2sR34T3KTQ0rhonC4xnKdSGTnmf/U8oaU +/Wl4RlbPiLfY0bvcoCbjdK45QlGpNz6ymJKrQ7oNEFzhx0N/KVELxLq4GfsI1LVV1jV zmRA== 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:references :mime-version:content-disposition:in-reply-to:user-agent; bh=O4OKcV2sMFnGzGzL/9+Cpt1JG7IyKFHG/ze9KlVPzA0=; b=A6JDAgVxzAxgtx4y492T8HfgPgEGKR1CL0bz1IYz49KOlim3AxGzM6A0G0qNJby5/c SitUYkLGzwP4waiWW51A3ezrWsER/IPSQfkvueAI3kr3ZP/+tQqBBBz7uO9o5M6lGpky qyVt7a+Pd7XMYC4/DAw7wGu6uttThCAYhZUhmyPCRpB58ZG1Lv7Jo0GheYQX5+qz4X72 B9cdKbomcdc14YGsBmBbORCMkpU4YROc728dErn0xg5+a/79kl335bjKa0Xa4ZlvkbCi +iwhWCjKXU/ypzNCQwIKgto+ls6HgolXcToUX6D93fg6GpOHyHgbcHf3AdVP6Hl7aLEQ CIZg== X-Gm-Message-State: APzg51AJ03v0OCMaI6Fx2rH42LOr/UwoSXsJ017nYUebo1XfT4OMyWBs YnAEGVaeURj85BwqcI/99PA= X-Received: by 2002:a17:902:246a:: with SMTP id m39-v6mr9516550plg.57.1536890121399; Thu, 13 Sep 2018 18:55:21 -0700 (PDT) Received: from localhost ([39.7.18.191]) by smtp.gmail.com with ESMTPSA id b21-v6sm13608691pfm.97.2018.09.13.18.55.19 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Thu, 13 Sep 2018 18:55:20 -0700 (PDT) Date: Fri, 14 Sep 2018 10:55:16 +0900 From: Sergey Senozhatsky To: Steven Rostedt Cc: Sergey Senozhatsky , Alexander Potapenko , Sergey Senozhatsky , Dmitriy Vyukov , penguin-kernel@i-love.sakura.ne.jp, kbuild test robot , pmladek@suse.com, syzkaller , LKML , Linus Torvalds , Andrew Morton Subject: Re: [PATCH] printk: inject caller information into the body of message Message-ID: <20180914015516.GA1108@jagdpanzerIV> References: <20180620091541.GB444@jagdpanzerIV> <20180620110759.GD444@jagdpanzerIV> <20180620130628.GA1000@tigerII.localdomain> <20180912065307.GA606@jagdpanzerIV> <20180912120548.4280f04a@vmware.local.home> <20180913071204.GA604@jagdpanzerIV> <20180913211232.06e6ad64@vmware.local.home> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20180913211232.06e6ad64@vmware.local.home> User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On (09/13/18 21:12), Steven Rostedt wrote: > > > > +#define __SEQ_BUF_INITIALIZER(buf, length) { \ > > + .buffer = (buf), \ > > + .size = (length), \ > > + .len = 0, \ > > + .readpos = 0, } > > Nit, but the end bracket '}' should be on it's own line. Even when > part of a macro. No prob, will change. I thought about putting it on its own line, but then checked include/linux/wait.h - __WAITQUEUE_INITIALIZER and __WAIT_QUEUE_HEAD_INITIALIZER. > > +#define DEFINE_PR_LINE(lev, name) \ > > + char __line[__PR_LINE_BUF_SZ]; \ > > To protect against name space collision could you use: > > char __line_##name[__PR_LINE_BUF_SZ]; Yes. > The rest looks fine to me. > > Acked-by: Steven Rostedt (VMware) Thanks. Just, to make sure, we are OK with seq_buf dependency and want anyone who wants to use pr_line to include linux/seq_buf.h? -ss