From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from simark.ca by simark.ca with LMTP id 4IyDLTtlpmCHWQAAWB0awg (envelope-from ) for ; Thu, 20 May 2021 09:33:47 -0400 Received: by simark.ca (Postfix, from userid 112) id B87AE1F11C; Thu, 20 May 2021 09:33:47 -0400 (EDT) X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on simark.ca X-Spam-Level: X-Spam-Status: No, score=-1.1 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,MAILING_LIST_MULTI,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.2 Received: from lists.lttng.org (lists.lttng.org [167.114.26.123]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by simark.ca (Postfix) with ESMTPS id 13DB61E54D for ; Thu, 20 May 2021 09:33:47 -0400 (EDT) Received: from lists-lttng01.efficios.com (localhost [IPv6:::1]) by lists.lttng.org (Postfix) with ESMTP id 4Fm9gG4Pjwz1rRc; Thu, 20 May 2021 09:33:46 -0400 (EDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=lists.lttng.org; s=default; t=1621517626; bh=6HA/oPM4sj7auEt3v5V/7ftYtvrBBTgVjwzBwLB8lEM=; h=Date:To:Cc:In-Reply-To:References:Subject:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From:Reply-To:From; b=aitIi4DKObEBq2XLyW6RfBamnHTxbkFsU28AII7zIN7Q++3HrkPcgUNpepmKCmb1i MCGXuAU5tYgXCHrgqDCedDlkb5jN/gsSsC5IttzMjbHnuauAp77pVofxKtb3x//4SU /F60oSX1iG9UCqCC6jcOAIAEZKZZTej/Y8IhojuuA188AKAxrzVt2UKhWGE+tRMJHX bdlPIZIVaHgOqN0NDv9ofpCcCmSrYxEsvqFX8AtoRVoxb73YUYU5V017OOkLhL7P+x jjfz6lWF1wp7iez2PH7ssYzE7LQSHWcx1GaRUad4n82NtDJXxKDiWrGuvmsTJY4HVC 5MMp/MrZeYz9A== Received: from mail.efficios.com (mail.efficios.com [167.114.26.124]) by lists.lttng.org (Postfix) with ESMTPS id 4Fm9gF1T2Bz1rg5 for ; Thu, 20 May 2021 09:33:45 -0400 (EDT) Received: from localhost (localhost [127.0.0.1]) by mail.efficios.com (Postfix) with ESMTP id F26B63356FF for ; Thu, 20 May 2021 09:33:38 -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 9m0OXXIYWTnU; Thu, 20 May 2021 09:33:38 -0400 (EDT) Received: from localhost (localhost [127.0.0.1]) by mail.efficios.com (Postfix) with ESMTP id A23E53356FD; Thu, 20 May 2021 09:33:38 -0400 (EDT) DKIM-Filter: OpenDKIM Filter v2.10.3 mail.efficios.com A23E53356FD 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 Hm8T3VtJRI9n; Thu, 20 May 2021 09:33:38 -0400 (EDT) Received: from mail03.efficios.com (mail03.efficios.com [167.114.26.124]) by mail.efficios.com (Postfix) with ESMTP id 979DB335931; Thu, 20 May 2021 09:33:38 -0400 (EDT) Date: Thu, 20 May 2021 09:33:38 -0400 (EDT) To: Norbert Lange Cc: lttng-dev Message-ID: <1632491203.52217.1621517618464.JavaMail.zimbra@efficios.com> In-Reply-To: References: <20210519165648.792939-1-nolange79@gmail.com> <1498635684.51674.1621450841774.JavaMail.zimbra@efficios.com> MIME-Version: 1.0 X-Originating-IP: [167.114.26.124] X-Mailer: Zimbra 8.8.15_GA_4018 (ZimbraWebClient - FF88 (Linux)/8.8.15_GA_4007) Thread-Topic: Avoid using the heap for small strings with tracef/tracelog Thread-Index: jLbz6ebOlgxv73ukM73VeNuFx/fdAQ== Subject: Re: [lttng-dev] [PATCH] Avoid using the heap for small strings with tracef/tracelog X-BeenThere: lttng-dev@lists.lttng.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: LTTng development list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , From: Mathieu Desnoyers via lttng-dev Reply-To: Mathieu Desnoyers Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: lttng-dev-bounces@lists.lttng.org Sender: "lttng-dev" ----- On May 20, 2021, at 8:20 AM, Norbert Lange nolange79@gmail.com wrote: > This patch was intended for 2.12, no src/common/tracer.h there. You should post feature patches against the master branch. Or if you just want to send a RFC, and you work on an older branch, please state that the patch is RFC and which branch the patch is against, but that work won't be merged unless it's rebased against master. > Also this patch is brocken, as a va_list cant be iterated twice. Right. But you could easily do: va_start() [ first use ] va_end() then va_start() [ second use ] va_end(), right ? Thanks, Mathieu -- Mathieu Desnoyers EfficiOS Inc. http://www.efficios.com _______________________________________________ lttng-dev mailing list lttng-dev@lists.lttng.org https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev