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=-3.9 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no 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 1543AC4727E for ; Fri, 25 Sep 2020 17:05:48 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 7287B21741 for ; Fri, 25 Sep 2020 17:05:47 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=efficios.com header.i=@efficios.com header.b="rkuhom2R" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 7287B21741 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=efficios.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 8EFAF6B007B; Fri, 25 Sep 2020 13:05:46 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 89FFB6B007E; Fri, 25 Sep 2020 13:05:46 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 6F28B6B0081; Fri, 25 Sep 2020 13:05:46 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0212.hostedemail.com [216.40.44.212]) by kanga.kvack.org (Postfix) with ESMTP id 546E16B007B for ; Fri, 25 Sep 2020 13:05:46 -0400 (EDT) Received: from smtpin09.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay05.hostedemail.com (Postfix) with ESMTP id 15641181AE864 for ; Fri, 25 Sep 2020 17:05:46 +0000 (UTC) X-FDA: 77302210692.09.price87_450013627169 Received: from filter.hostedemail.com (10.5.16.251.rfc1918.com [10.5.16.251]) by smtpin09.hostedemail.com (Postfix) with ESMTP id ECD7D180AD801 for ; Fri, 25 Sep 2020 17:05:45 +0000 (UTC) X-HE-Tag: price87_450013627169 X-Filterd-Recvd-Size: 4483 Received: from mail.efficios.com (mail.efficios.com [167.114.26.124]) by imf06.hostedemail.com (Postfix) with ESMTP for ; Fri, 25 Sep 2020 17:05:45 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by mail.efficios.com (Postfix) with ESMTP id 0311C2D0649; Fri, 25 Sep 2020 13:05:45 -0400 (EDT) Received: from mail.efficios.com ([127.0.0.1]) by localhost (mail03.efficios.com [127.0.0.1]) (amavisd-new, port 10032) with ESMTP id UWotcFCy9elo; Fri, 25 Sep 2020 13:05:44 -0400 (EDT) Received: from localhost (localhost [127.0.0.1]) by mail.efficios.com (Postfix) with ESMTP id 8B6232D083C; Fri, 25 Sep 2020 13:05:44 -0400 (EDT) DKIM-Filter: OpenDKIM Filter v2.10.3 mail.efficios.com 8B6232D083C DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=efficios.com; s=default; t=1601053544; bh=9u5etktlFhErqPQLUUxkrQhVKY8cRDGh0UhHCb7uXno=; h=Date:From:To:Message-ID:MIME-Version; b=rkuhom2RCZyD3mtKD/vF1qrzRbKitcRKV3xVAkorSFr0FETXYRypacQNKrGy2P2HV Ykoj7CwObLsLPezZOPww7NkkZhOnzhCXNJ9tlOdTnENORuBNZxy0vdWfOiQ2SnPBgh gwqUnhOgMxvJ+XJnFf8LH7wCU73JrrIPGXxYAQoBxsaCaupiDocNFIEHSXGeVa45Ym n7CWdRTMI/zwkqXxrDyhmLR5Fgu4563jveizvzQERF326my8GS2kR6PMXxGYzQ7YuW aiqgWeLZuK8X0eUDIEkfZloCv41oN8w5ihwS2iKBlSl2Ib5JxE8vBodsxSVcXb5UqS 29ja6JU9cXg+A== X-Virus-Scanned: amavisd-new at efficios.com Received: from mail.efficios.com ([127.0.0.1]) by localhost (mail03.efficios.com [127.0.0.1]) (amavisd-new, port 10026) with ESMTP id KYOSqdHR5bZP; Fri, 25 Sep 2020 13:05:44 -0400 (EDT) Received: from mail03.efficios.com (mail03.efficios.com [167.114.26.124]) by mail.efficios.com (Postfix) with ESMTP id 7A49D2D0533; Fri, 25 Sep 2020 13:05:44 -0400 (EDT) Date: Fri, 25 Sep 2020 13:05:44 -0400 (EDT) From: Mathieu Desnoyers To: rostedt Cc: paulmck , Michael Jeanson , linux-kernel , Yafang Shao , Axel Rasmussen , Andrew Morton , Vlastimil Babka , Michel Lespinasse , Daniel Jordan , Davidlohr Bueso , linux-mm , Ingo Molnar , Joonsoo Kim Message-ID: <897050665.69743.1601053544373.JavaMail.zimbra@efficios.com> In-Reply-To: <20200925122647.230decde@oasis.local.home> References: <20200924170928.466191266@goodmis.org> <20200924161328.760f5e79@oasis.local.home> <1430794518.69084.1600979254425.JavaMail.zimbra@efficios.com> <20200924163331.0080b943@oasis.local.home> <176393901.69671.1601044916547.JavaMail.zimbra@efficios.com> <20200925111415.60f5334c@oasis.local.home> <965650354.69699.1601047806662.JavaMail.zimbra@efficios.com> <20200925122647.230decde@oasis.local.home> Subject: Re: [PATCH 1/2] tracepoints: Add helper to test if tracepoint is enabled in a header MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Originating-IP: [167.114.26.124] X-Mailer: Zimbra 8.8.15_GA_3965 (ZimbraWebClient - FF80 (Linux)/8.8.15_GA_3963) Thread-Topic: tracepoints: Add helper to test if tracepoint is enabled in a header Thread-Index: OfP6K53urTP5+emvUDcmbH1lEHoGnA== X-Bogosity: Ham, tests=bogofilter, spamicity=0.000001, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: ----- On Sep 25, 2020, at 12:26 PM, rostedt rostedt@goodmis.org wrote: > On Fri, 25 Sep 2020 11:30:06 -0400 (EDT) > Mathieu Desnoyers wrote: > >> > Anyway, I don't see any issues with the current patch set as is >> > (besides the documentation fix, which I already updated locally). And >> > will add this to my queue for linux-next. >> >> The only thing I would change in the documentation is to word this as >> "here is a trampoline trick which can be used to work-around rare cases >> of tracepoint header circular dependency issues" rather than "always use >> this when instrumenting a header". >> > > I rather not have tracepoints in headers. Period! > > It's not just about circular dependencies, it also bloats the code. Fair enough. We could indeed argue that having a tracepoint in a header's static inline function will end up replicating that tracepoint at every site where the function is used. So in terms of code size, it's better to use the trampoline approach. Thanks, Mathieu -- Mathieu Desnoyers EfficiOS Inc. http://www.efficios.com