From patchwork Thu Feb 22 21:14:16 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Steven Rostedt X-Patchwork-Id: 205067 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:7300:a81b:b0:108:e6aa:91d0 with SMTP id bq27csp213741dyb; Thu, 22 Feb 2024 13:14:39 -0800 (PST) X-Forwarded-Encrypted: i=3; AJvYcCXVPdwwzJJtRACthwayg+AJQouYY/4ed1TGlo/LYgCj1nHGBz3ayN6m1LfTKePqezEIAp26scbBazu4mNRxS+BPeH+Bpw== X-Google-Smtp-Source: AGHT+IGkxNvwysSfdaXuimLi6n1jYgT0tGYGS+ZvwO7YCVemCbN+YzVi5/wglNplTONTCrvHq6lW X-Received: by 2002:a05:6214:daa:b0:68f:6bce:8500 with SMTP id h10-20020a0562140daa00b0068f6bce8500mr441620qvh.13.1708636479241; Thu, 22 Feb 2024 13:14:39 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1708636479; cv=pass; d=google.com; s=arc-20160816; b=iHN7ksXhEW/qYS71Z09HXd6GfGm3oAW4S6AWUnpZE96MV5EGAoZ+2KGGMfFjk5gS+I mdlovc/qoDJtZhQjW5ZVsTvFfBYa6hEBTSf2Srbc9ehi4/B3L0lpXCLjzr3D051wLPZQ yyCJe4fRgD5XEwXfwEc7DQ4Wmo0fTfgh4wLU8pTeLswMCmQee8eTEci4SsGRMHVCg8bf 0hpFjcPYNiqIuWpy9V+wrDDcKXB1bmGDeT/j51r/iuuRcB1u3GB81eq3kvtMj8tYkzGl nmNQXdSlLuwV2gojcJntBOdWB6OK8j9x8//Ae7UJm1fZVHAmyFkKngzVlRR5WiZ2ZP/p 81ig== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=mime-version:list-unsubscribe:list-subscribe:list-id:precedence :references:subject:cc:to:from:date:user-agent:message-id; bh=LzolFwYcN7VKlZQOXza6vBTSmkIinVc8MSdD69iRCNA=; fh=6lU0d7+j2gvP0RlfbRyidxN6N8cP9012O+fDGzz+FVE=; b=nWBcYwIOrSMaVdglrm2KIJ5FFqHRkgesmLuFv52l+/LkOrAUV6u3VxekQ3L/6rkUbd bBl9hz3jlAPFAFk+gAiE8Vmra5Wf95PRso1PtaDb7kmUq1XVQtCHwGv/6YCM7Axo6mWD yhYoVRKYeytAlzt2ZBLfB673BMJPZNGyITr3ttijWpBSYl8JvYIKo75b190VYAGd0nDS 0H0DmvJYxdywQF9k7J/vp/Xmg2dJgQwzlKid1t4BKEFQx27GzRHcYduaWiZyhU7Burtf lnZnd9ImZY5wdWUSmke9PwH7TM5ZlRgVDRVJpa4BUFXEx4Ab6GMPZQHpj7VbVrXqpjER vUeA==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; arc=pass (i=1); spf=pass (google.com: domain of linux-kernel+bounces-77376-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-kernel+bounces-77376-ouuuleilei=gmail.com@vger.kernel.org" Received: from ny.mirrors.kernel.org (ny.mirrors.kernel.org. [147.75.199.223]) by mx.google.com with ESMTPS id ge10-20020a05621427ca00b0068fb8235816si2499854qvb.501.2024.02.22.13.14.39 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 22 Feb 2024 13:14:39 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-77376-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223; Authentication-Results: mx.google.com; arc=pass (i=1); spf=pass (google.com: domain of linux-kernel+bounces-77376-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-kernel+bounces-77376-ouuuleilei=gmail.com@vger.kernel.org" Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ny.mirrors.kernel.org (Postfix) with ESMTPS id 0EA161C22ABB for ; Thu, 22 Feb 2024 21:14:39 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id DE3A213BAFB; Thu, 22 Feb 2024 21:12:53 +0000 (UTC) Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id CDCB573F28; Thu, 22 Feb 2024 21:12:51 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1708636371; cv=none; b=S9XWNugs+sLwSOMAceZgO0Zt4phyVEX5lsGbsJFGEfvrXyDYlBfCSvFVYl101YLUMFJAgppDQNig4E3xwPXLY9LcGUHC9t/UAhpHyIK2y5jTmVRuP2iUVdCCR8nsna4HYdxPtFmbzl4L0v19qFvBpuFIr5IRnSKsILxiUsMwOIg= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1708636371; c=relaxed/simple; bh=IMJwLqRaMA+uC9DlBNf87xu5iBKGaOtKb/jWuGFEwPA=; h=Message-ID:Date:From:To:Cc:Subject:References:MIME-Version: Content-Type; b=QWngmH+io18IDv8s8Qf50P8Cr9mFjVhnGgebPP65rB4aRYPVd3CPkodXXYbtvI/yz0arD+GfCbRBTB0ySDDXj9JKi0Z9QtpoTn6r6IPUVrBhmWz0+LEsDeLlkZ6uQRZZw8PhLMgelg4iovMMCXJZH7OT0bxCeoKvgJNjYkqcHb0= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 Received: by smtp.kernel.org (Postfix) with ESMTPSA id 8208BC433C7; Thu, 22 Feb 2024 21:12:51 +0000 (UTC) Received: from rostedt by gandalf with local (Exim 4.97) (envelope-from ) id 1rdGOw-00000006hqP-3FDW; Thu, 22 Feb 2024 16:14:42 -0500 Message-ID: <20240222211442.634192653@goodmis.org> User-Agent: quilt/0.67 Date: Thu, 22 Feb 2024 16:14:16 -0500 From: Steven Rostedt To: linux-kernel@vger.kernel.org, linux-trace-kernel@vger.kernel.org Cc: Masami Hiramatsu , Mark Rutland , Mathieu Desnoyers , Andrew Morton , =?utf-8?b?VmlsbGUgU3lyasOkbMOk?= , Rodrigo Vivi , Chuck Lever Subject: [PATCH v2 1/4] tracing: Rework __assign_str() and __string() to not duplicate getting the string References: <20240222211415.255659509@goodmis.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1791635204886449809 X-GMAIL-MSGID: 1791635204886449809 From: "Steven Rostedt (Google)" The TRACE_EVENT() macro handles dynamic strings by having: TP_PROTO(struct some_struct *s), TP_ARGS(s), TP_STRUCT__entry( __string(my_string, s->string) ), TP_fast_assign( __assign_str(my_string, s->string); ) TP_printk("%s", __get_str(my_string)) There's even some code that may call a function helper to find the s->string value. The problem with the above is that the work to get the s->string is done twice. Once at the __string() and again in the __assign_str(). But the __string() uses dynamic_array() which has a helper structure that is created holding the offsets and length of the string fields. Instead of finding the string twice, just save it off in another field from that helper structure, and have __assign_str() use that instead. Note, this also means that the second parameter of __assign_str() isn't even used anymore, and may be removed in the future. Signed-off-by: Steven Rostedt (Google) --- include/trace/stages/stage2_data_offsets.h | 4 ++-- include/trace/stages/stage5_get_offsets.h | 15 ++++++++++----- include/trace/stages/stage6_event_callback.h | 12 ++++++++---- 3 files changed, 20 insertions(+), 11 deletions(-) diff --git a/include/trace/stages/stage2_data_offsets.h b/include/trace/stages/stage2_data_offsets.h index 469b6a64293d..8b0cff06d346 100644 --- a/include/trace/stages/stage2_data_offsets.h +++ b/include/trace/stages/stage2_data_offsets.h @@ -24,7 +24,7 @@ #define __array(type, item, len) #undef __dynamic_array -#define __dynamic_array(type, item, len) u32 item; +#define __dynamic_array(type, item, len) u32 item; const void *item##_ptr_; #undef __string #define __string(item, src) __dynamic_array(char, item, -1) @@ -45,7 +45,7 @@ #define __sockaddr(field, len) __dynamic_array(u8, field, len) #undef __rel_dynamic_array -#define __rel_dynamic_array(type, item, len) u32 item; +#define __rel_dynamic_array(type, item, len) u32 item; const void *item##_ptr_; #undef __rel_string #define __rel_string(item, src) __rel_dynamic_array(char, item, -1) diff --git a/include/trace/stages/stage5_get_offsets.h b/include/trace/stages/stage5_get_offsets.h index e30a13be46ba..45f8151cf622 100644 --- a/include/trace/stages/stage5_get_offsets.h +++ b/include/trace/stages/stage5_get_offsets.h @@ -47,10 +47,12 @@ #undef __string #define __string(item, src) __dynamic_array(char, item, \ - strlen((src) ? (const char *)(src) : "(null)") + 1) + strlen((src) ? (const char *)(src) : "(null)") + 1) \ + __data_offsets->item##_ptr_ = src; #undef __string_len -#define __string_len(item, src, len) __dynamic_array(char, item, (len) + 1) +#define __string_len(item, src, len) __dynamic_array(char, item, (len) + 1)\ + __data_offsets->item##_ptr_ = src; #undef __vstring #define __vstring(item, fmt, ap) __dynamic_array(char, item, \ @@ -67,11 +69,14 @@ __data_size += __item_length; #undef __rel_string -#define __rel_string(item, src) __rel_dynamic_array(char, item, \ - strlen((src) ? (const char *)(src) : "(null)") + 1) +#define __rel_string(item, src) __rel_dynamic_array(char, item, \ + strlen((src) ? (const char *)(src) : "(null)") + 1) \ + __data_offsets->item##_ptr_ = src; #undef __rel_string_len -#define __rel_string_len(item, src, len) __rel_dynamic_array(char, item, (len) + 1) +#define __rel_string_len(item, src, len) __rel_dynamic_array(char, item, (len) + 1)\ + __data_offsets->item##_ptr_ = src; + /* * __bitmask_size_in_bytes_raw is the number of bytes needed to hold * num_possible_cpus(). diff --git a/include/trace/stages/stage6_event_callback.h b/include/trace/stages/stage6_event_callback.h index 919b1a4da980..b3e2f321e787 100644 --- a/include/trace/stages/stage6_event_callback.h +++ b/include/trace/stages/stage6_event_callback.h @@ -32,12 +32,14 @@ #undef __assign_str #define __assign_str(dst, src) \ - strcpy(__get_str(dst), (src) ? (const char *)(src) : "(null)"); + strcpy(__get_str(dst), __data_offsets.dst##_ptr_ ? \ + __data_offsets.dst##_ptr_ : "(null)") #undef __assign_str_len #define __assign_str_len(dst, src, len) \ do { \ - memcpy(__get_str(dst), (src), (len)); \ + memcpy(__get_str(dst), __data_offsets.dst##_ptr_ ? \ + __data_offsets.dst##_ptr_ : "(null)", len); \ __get_str(dst)[len] = '\0'; \ } while(0) @@ -92,12 +94,14 @@ #undef __assign_rel_str #define __assign_rel_str(dst, src) \ - strcpy(__get_rel_str(dst), (src) ? (const char *)(src) : "(null)"); + strcpy(__get_rel_str(dst), __data_offsets.dst##_ptr_ ? \ + __data_offsets.dst##_ptr_ : "(null)") #undef __assign_rel_str_len #define __assign_rel_str_len(dst, src, len) \ do { \ - memcpy(__get_rel_str(dst), (src), (len)); \ + memcpy(__get_rel_str(dst), __data_offsets.dst##_ptr_ ? \ + __data_offsets.dst##_ptr_ : "(null)", len); \ __get_rel_str(dst)[len] = '\0'; \ } while (0) From patchwork Thu Feb 22 21:14:17 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Steven Rostedt X-Patchwork-Id: 205066 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:7300:a81b:b0:108:e6aa:91d0 with SMTP id bq27csp213720dyb; Thu, 22 Feb 2024 13:14:36 -0800 (PST) X-Forwarded-Encrypted: i=3; AJvYcCXEPxonjrN1wn2I4LEz1DtyzLgcgBBvY/6nfrSSPpoIIdhMdWEMSJJUPUjnjzjrUm6bYIEng6iVvN3pK/tHS7omDx6tJg== X-Google-Smtp-Source: AGHT+IF0wEBSraW+hYuBU/lKPCUJxdNTRFrS0G3jQPJSxIumCedzOrD+HjsDaMjei8TEWZsoU10r X-Received: by 2002:a25:6f8a:0:b0:dcb:d0cb:651e with SMTP id k132-20020a256f8a000000b00dcbd0cb651emr381192ybc.28.1708636476057; Thu, 22 Feb 2024 13:14:36 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1708636476; cv=pass; d=google.com; s=arc-20160816; b=WJeC6z2CDiYedMuG2iNKw3MpPHWQtxXUc6vyjy8f1SNxOHh+w5CWaobYgcshlrkC7y 4iyG9UM6H0+Eb1QOpAURjKB/B9mRqN1KhZ28P1YJJQisndtEcoVVBDjgJKJZaJ2tA9dR buwC3Drzg8Z295uOhqTvvLDWl0L7J1MWIimEPLPau4SITFzWtCLvXbMo0FGiypSS7AOp g+beIimOJZPUbnRAUmEjOxH7kch8hr7+CsGmU1eDA8ig3gQWuGFIhuTn+QS2Fr89DlBW fs/uFtUavVqy7Fh9rvV+pHJ8eetd9AM/WRThXKVyEFALsxWaLvIcRFaPX8TsZLsKeDI4 LHVQ== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=mime-version:list-unsubscribe:list-subscribe:list-id:precedence :references:subject:cc:to:from:date:user-agent:message-id; bh=DhHmbMpaA1k4qN5yLQZTbOyHIppdPNq95Qj4wRPuVYs=; fh=6lU0d7+j2gvP0RlfbRyidxN6N8cP9012O+fDGzz+FVE=; b=sxlRQ3cFshjkdCvuT+jwBhTXn//cekaLH5tsfaTwnKGQjcJD0iYBdxI72IAKmDVT8Q +/nR4Tuqe2aEKKTBgDETg5qjGomvs2wLnoEY1Vx5hznW3U6dJiArPzQqqtwsiACe9izG Dlb0dZmmapA6FcP9YvafYKmdiyAo4qSoUgXmyjk7cjfAkKx7V0WSM3DVbaroD+f4Uxba +tDCGWO1Hbd7UWcTmTvmbRMtl+KfYV/30cb1943pTpVW9tNWxfL9/luyKAvSsTqPrjuc Mps/irZ5OyXHQmtEdiITuh8xEnK1dqB0kf6jMpQvwhynkgVyhu9Esr8bMKnITMrppzgK Yzeg==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; arc=pass (i=1); spf=pass (google.com: domain of linux-kernel+bounces-77377-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-kernel+bounces-77377-ouuuleilei=gmail.com@vger.kernel.org" Received: from ny.mirrors.kernel.org (ny.mirrors.kernel.org. [147.75.199.223]) by mx.google.com with ESMTPS id l21-20020a05620a0c1500b007874ee5955esi12607311qki.475.2024.02.22.13.14.35 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 22 Feb 2024 13:14:36 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-77377-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223; Authentication-Results: mx.google.com; arc=pass (i=1); spf=pass (google.com: domain of linux-kernel+bounces-77377-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-kernel+bounces-77377-ouuuleilei=gmail.com@vger.kernel.org" Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ny.mirrors.kernel.org (Postfix) with ESMTPS id D19DF1C2377E for ; Thu, 22 Feb 2024 21:14:35 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id C205B13BAF4; Thu, 22 Feb 2024 21:12:53 +0000 (UTC) Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 089F012D1F9; Thu, 22 Feb 2024 21:12:51 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1708636372; cv=none; b=gHZnUNM7uCeJRP18DSFZ3IPImu5A4bFXYUiq1zROh26ec+RcpSLwHGLpPiNsAsWDTTPyy1DIO5CVD7Srk/LAdG78K4kFwD5N1UgODh26kgIaJLh0xw3eNWeD8XDvZDBdylS0W0k9yI6+QFLq/6wrJpma8BPOG4nDbKYhRazqNTc= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1708636372; c=relaxed/simple; bh=ASKKBEQ/kPwz5mh0vibiuZNuVaq8X/g1jUk/IeZwsxE=; h=Message-ID:Date:From:To:Cc:Subject:References:MIME-Version: Content-Type; b=ak7iD70+I0Mk0DIRHsL0IgGl4vtilJglWGs+jkb4cGpqR3FwEo+6zqefTfqFybrbZzOoXs+2kUML0lpWQYDuk/njfs3gMwJddItdJw6sLzgHUHBf01ZZe1sVBedqSN3U+5DqBDZTlGsmm8uxPIB7Ua1bUkUnNn3RXDLUhm1V7Tw= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 Received: by smtp.kernel.org (Postfix) with ESMTPSA id B556DC433A6; Thu, 22 Feb 2024 21:12:51 +0000 (UTC) Received: from rostedt by gandalf with local (Exim 4.97) (envelope-from ) id 1rdGOw-00000006hqt-3tgc; Thu, 22 Feb 2024 16:14:42 -0500 Message-ID: <20240222211442.793074999@goodmis.org> User-Agent: quilt/0.67 Date: Thu, 22 Feb 2024 16:14:17 -0500 From: Steven Rostedt To: linux-kernel@vger.kernel.org, linux-trace-kernel@vger.kernel.org Cc: Masami Hiramatsu , Mark Rutland , Mathieu Desnoyers , Andrew Morton , =?utf-8?b?VmlsbGUgU3lyasOkbMOk?= , Rodrigo Vivi , Chuck Lever Subject: [PATCH v2 2/4] tracing: Do not calculate strlen() twice for __string() fields References: <20240222211415.255659509@goodmis.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1791635201421362761 X-GMAIL-MSGID: 1791635201421362761 From: "Steven Rostedt (Google)" The TRACE_EVENT() macro handles dynamic strings by having: TP_PROTO(struct some_struct *s), TP_ARGS(s), TP_STRUCT__entry( __string(my_string, s->string) ), TP_fast_assign( __assign_str(my_string, s->string); ) TP_printk("%s", __get_str(my_string)) There's even some code that may call a function helper to find the s->string value. The problem with the above is that the work to get the s->string is done twice. Once at the __string() and again in the __assign_str(). The length of the string is calculated via a strlen(), not once, but twice. Once during the __string() macro and again in __assign_str(). But the length is actually already recorded in the data location and here's no reason to call strlen() again. Just use the saved length that was saved in the __string() code for the __assign_str() code. Signed-off-by: Steven Rostedt (Google) --- include/trace/stages/stage6_event_callback.h | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/include/trace/stages/stage6_event_callback.h b/include/trace/stages/stage6_event_callback.h index b3e2f321e787..c0e5d097324e 100644 --- a/include/trace/stages/stage6_event_callback.h +++ b/include/trace/stages/stage6_event_callback.h @@ -32,8 +32,9 @@ #undef __assign_str #define __assign_str(dst, src) \ - strcpy(__get_str(dst), __data_offsets.dst##_ptr_ ? \ - __data_offsets.dst##_ptr_ : "(null)") + memcpy(__get_str(dst), __data_offsets.dst##_ptr_ ? \ + __data_offsets.dst##_ptr_ : "(null)", \ + __get_dynamic_array_len(dst)) #undef __assign_str_len #define __assign_str_len(dst, src, len) \ @@ -94,8 +95,9 @@ #undef __assign_rel_str #define __assign_rel_str(dst, src) \ - strcpy(__get_rel_str(dst), __data_offsets.dst##_ptr_ ? \ - __data_offsets.dst##_ptr_ : "(null)") + memcpy(__get_rel_str(dst), __data_offsets.dst##_ptr_ ? \ + __data_offsets.dst##_ptr_ : "(null)", \ + __get_rel_dynamic_array_len(dst)) #undef __assign_rel_str_len #define __assign_rel_str_len(dst, src, len) \