Received: by 2002:a05:6500:1b8f:b0:1fa:5c73:8e2d with SMTP id df15csp307251lqb; Tue, 28 May 2024 16:54:32 -0700 (PDT) X-Forwarded-Encrypted: i=3; AJvYcCVGwc73KJVkgSYeR/Ei+YTbRBJP3aVYe2k+xzoCC2N80cxHtz+jo+gGfHKXbwWpsFs93l1m8PfoMdN3SYgdRE3gu6MLo0XO2gGsP6sFEw== X-Google-Smtp-Source: AGHT+IE72dSa6VeTBHzwZUYWqA9LDZaLX3OLrB2XMPUjTmUnikVhCwkZLoe4QNsxV1aoMz4boR8R X-Received: by 2002:a50:99c2:0:b0:56d:fca5:4245 with SMTP id 4fb4d7f45d1cf-5785198776cmr8952346a12.10.1716940472805; Tue, 28 May 2024 16:54:32 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1716940472; cv=pass; d=google.com; s=arc-20160816; b=rGGE1MyEEhBwMsXZ653W7O7/u3y6Djy+H3EucFfIf+tXroak4YkDmlJqazdxAjxEWa Img/jabuQtQdri5IUOYah7usLLLH35DsP6dVEWC9jtf8ZDkYz3tqwBiZB+OH+aNlXt/M Wlk3xtKf+RmHDrisHdfaKKi3J0Z3davVN9p46I2XbOSBOAWSousqWs+1vgSoapqd4ptj WP3uTJ3Lw8sJry1RPP+aDhi9I00MB1uSs058LRWuXZsbOk+P53TeqipFM9W9ouN1Yuwp ukgKZBwCpHZrbxIXPaPYZqiu5PwCx5vRuo1RZ1y9stbvLDHC3iOqM8/nA0GXO4FG7CRP 9TVQ== 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 :user-agent:content-transfer-encoding:references:in-reply-to:date:cc :to:from:subject:message-id:dkim-signature; bh=gShILwMjJBDCk7bxa10+DKpVyyvmAskS6Sj1Oh2LJ34=; fh=2NsD/dzUPeSHruQoRAdy/rjJfu9jrHEIhhVfOOUEpDc=; b=W2s7hCaKBrikuLvHJNM2zwm7Db6izmMosE9gigkLvzDuLvCsVP5JNFbFUTdCJi+9GT FHYla1Xwwrm1z36uKT5y6zWtzFxVob3c8i11JWgP4dyImMRp43vSCfnmCMld1m5AlEDR AWXaig7ERpOBudneUIsSWbQ9DOA+8CXnS/f6Ik4T4XF68WGfls8xL39x4L1XtOCYgPaF qfSiR/hQbMbmQcgyMzwXhDrQixSI8fPhjMvpyXwf8P2f43sMYEXQ7gATts4w47OeVwWi UmcUS53c6mrDbsAa2cdrChBEzlfEUlMiEWUyq1sDC/FOfoKxSGa4lKmmhAShaXYr6iPb dShw==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@codeconstruct.com.au header.s=2022a header.b="nRkk1BG/"; arc=pass (i=1 spf=pass spfdomain=codeconstruct.com.au dkim=pass dkdomain=codeconstruct.com.au dmarc=pass fromdomain=codeconstruct.com.au); spf=pass (google.com: domain of linux-kernel+bounces-193244-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) smtp.mailfrom="linux-kernel+bounces-193244-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=codeconstruct.com.au Return-Path: Received: from am.mirrors.kernel.org (am.mirrors.kernel.org. [2604:1380:4601:e00::3]) by mx.google.com with ESMTPS id 4fb4d7f45d1cf-579f86b5584si1082591a12.226.2024.05.28.16.54.32 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 28 May 2024 16:54:32 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel+bounces-193244-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3; Authentication-Results: mx.google.com; dkim=pass header.i=@codeconstruct.com.au header.s=2022a header.b="nRkk1BG/"; arc=pass (i=1 spf=pass spfdomain=codeconstruct.com.au dkim=pass dkdomain=codeconstruct.com.au dmarc=pass fromdomain=codeconstruct.com.au); spf=pass (google.com: domain of linux-kernel+bounces-193244-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) smtp.mailfrom="linux-kernel+bounces-193244-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=codeconstruct.com.au 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 am.mirrors.kernel.org (Postfix) with ESMTPS id 84FF51F2620C for ; Tue, 28 May 2024 23:54:32 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 247E813FD8B; Tue, 28 May 2024 23:54:22 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=codeconstruct.com.au header.i=@codeconstruct.com.au header.b="nRkk1BG/" Received: from codeconstruct.com.au (pi.codeconstruct.com.au [203.29.241.158]) (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 17AE513D897; Tue, 28 May 2024 23:54:18 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=203.29.241.158 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1716940461; cv=none; b=trrHwYqO0MXeX84/9O7VFdARe0zsqE8m5wqCUw+X61HWeH1UPuqx79nKNI8wvRIWUZqkqJF1I2bljjySnljITso/4lpB2Z2am1ZRoxWfuAgw623EH/ph0xl2gNCZdXhr7JOLhrLy+iPaQu2xNplESdTlJRL7g4mbazIE2fkxwAg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1716940461; c=relaxed/simple; bh=gShILwMjJBDCk7bxa10+DKpVyyvmAskS6Sj1Oh2LJ34=; h=Message-ID:Subject:From:To:Cc:Date:In-Reply-To:References: Content-Type:MIME-Version; b=gEqH06j7IvdK5CnQZ0B9Awwu3SLOQoRaTs7mvIw4N1LlyBccXenGLpXDaqGDFRjyn/daDLo6o9Pt8Xy1thr7knQFfLpaO3f5hXqAZErUlQL/5qNap0/+Vy5B+3+MS+X1ERvCtrGlt7PUgreuB4LUK17UXEs1ZlCEMT4xcS16jwU= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=codeconstruct.com.au; spf=pass smtp.mailfrom=codeconstruct.com.au; dkim=pass (2048-bit key) header.d=codeconstruct.com.au header.i=@codeconstruct.com.au header.b=nRkk1BG/; arc=none smtp.client-ip=203.29.241.158 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=codeconstruct.com.au Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=codeconstruct.com.au Received: from pecola.lan (unknown [159.196.93.152]) by mail.codeconstruct.com.au (Postfix) with ESMTPSA id 36F542014A; Wed, 29 May 2024 07:54:09 +0800 (AWST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=codeconstruct.com.au; s=2022a; t=1716940450; bh=gShILwMjJBDCk7bxa10+DKpVyyvmAskS6Sj1Oh2LJ34=; h=Subject:From:To:Cc:Date:In-Reply-To:References; b=nRkk1BG/PQkujWt1gGUGlfPIr2mNWhLNiXDwGjQrvFd2wfZ42lW0FBl16JVCnFw1H DRhC3I+LwDUPo6txx+/w+9uzAMvmrqmlklfDoE+w3ZekmqRrL1e9mVew1CL8fKJt07 a/YUN8qI/1Ud+AuBK70J28hi+xaQA9fJunPfc/H6r0kSYZ9E79jBVWGbn+6pBY8axH jK86a4YygmyCj5Xls+Hb4zW6hJwyL+nimqM2booGDYXMAgiqV3l6lZQuFybmoioImU imyG0P5RivxyscBDrfF0UpQHo6A2eY3Fb7ebATuzAgks21lrIFztn0CjU1UX2eKyRC QNlBGhRNsDIGA== Message-ID: <520cf8db945cf8dce4afdaddb59ceda65463a406.camel@codeconstruct.com.au> Subject: Re: [PATCH] mctp i2c: Add rx trace From: Jeremy Kerr To: Tal Yacobi , matt@codeconstruct.com.au, rostedt@goodmis.org, mhiramat@kernel.org, mathieu.desnoyers@efficios.com Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, linux-trace-kernel@vger.kernel.org Date: Wed, 29 May 2024 07:54:08 +0800 In-Reply-To: <20240528143420.742611-1-talycb8@gmail.com> References: <20240528143420.742611-1-talycb8@gmail.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.46.4-2 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Hi Tal, Thanks for the contribution! Some comments: > mctp-i2c rx implementation doesn't call > __i2c_transfer which calls the i2c reply trace function. No, but we can trace the i2c rx path through the trace_i2c_slave tracepoint. It is a little messier than tracing trace_i2c_write, but has been sufficient with the debugging I've needed in the past. > Add an mctp_reply trace function that will be used instead. Can you elaborate a little on what you were/are looking to inspect here? (mainly: which packet fields are you interested in?) That will help to determine the best approach here. Cheers, Jeremy