From patchwork Thu Oct 27 09:12:40 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Mukesh Ojha X-Patchwork-Id: 11693 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a5d:6687:0:0:0:0:0 with SMTP id l7csp127804wru; Thu, 27 Oct 2022 02:35:22 -0700 (PDT) X-Google-Smtp-Source: AMsMyM5Xi3n4jh3N7FUh5U54NsnlueKgxAdObjZOrC9p6fmNYisziVsBJqBD3xCs8O0I3I4N8xju X-Received: by 2002:a63:d008:0:b0:46e:c359:7fa2 with SMTP id z8-20020a63d008000000b0046ec3597fa2mr26844015pgf.285.1666863322666; Thu, 27 Oct 2022 02:35:22 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1666863322; cv=none; d=google.com; s=arc-20160816; b=FMxvjl2g6qhSYAY5x6f7QXQGNV9be9XEXHyz0pbAE/zdaVxbY8b+cOk1k1RVAiKMfi FBw9Y0Xg4efQDxkvw/3kYXbCuaokpFf/VugTI67k9dmSrzLQEoxFdyAKwVI3tyeagxX7 VW2k7S6qCSv/KeUBO/Rwm/trwDq5F+VBQWr5hPgVUEJBFNV3Mr9wD2YC1povR4m1tlc8 ty/iCLXEBW1OKw4f/q8JCozibaIBC+NrXIPtKrmgBo77dUzLS1WrFyxX58fLG0CKvcEa YeuTMUJNJiKFTqoo6ezoJKWaa2EyYuv8kbMwigFy0r5HiP+FdhwzuRKzjESqGFJF0PU7 GqLg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:mime-version:message-id:date:subject:cc:to:from :dkim-signature; bh=FEV68A32v+oOBg8wVp3Otkn00cWdPegP8fO0J269VmU=; b=S3d+qjCcQAf1jRRqLBoKKV5/Z7PkyR2HrMApqS4lqYI+ID4BDlSWczk2H3SVrnTZvz oCLUAMVZqDDicAHeu+D/4jVLDotITKibw+mtaLPmlILOcIXmPGwLgx53st0K0XeHo/Os 1t0jLnu1eYpL9g+W9q67OQRc5oAkMTdC+I6l97MmqLowI0mOZG6JN43gZGOKKDlaHK0E UNu73+fo5sFEUB794GL/HElQobpYltGR0PdSNNHcacDiyDfqXkafG5QZc1x7eeFq2q30 pDHmH+2o7p7WPOEG9DpTh1zExfYQ6ndCpG79wmi9frv/bkFyVJ6LDM8HJklLNwc6RFly 7vIg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@quicinc.com header.s=qcppdkim1 header.b=kl0CZfLh; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=quicinc.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id bj11-20020a170902850b00b001867db1d29csi913566plb.60.2022.10.27.02.35.09; Thu, 27 Oct 2022 02:35:22 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@quicinc.com header.s=qcppdkim1 header.b=kl0CZfLh; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=quicinc.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235439AbiJ0JUc (ORCPT + 99 others); Thu, 27 Oct 2022 05:20:32 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:41542 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235353AbiJ0JUW (ORCPT ); Thu, 27 Oct 2022 05:20:22 -0400 Received: from mx0a-0031df01.pphosted.com (mx0a-0031df01.pphosted.com [205.220.168.131]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id E31D91FFAA for ; Thu, 27 Oct 2022 02:20:14 -0700 (PDT) Received: from pps.filterd (m0279866.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.17.1.5/8.17.1.5) with ESMTP id 29R5J10Z029019; Thu, 27 Oct 2022 09:19:56 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=quicinc.com; h=from : to : cc : subject : date : message-id : mime-version : content-type; s=qcppdkim1; bh=FEV68A32v+oOBg8wVp3Otkn00cWdPegP8fO0J269VmU=; b=kl0CZfLheEoNmxtL91PiVORGn+aYtSEPUb51CMMETYSoxZmI20ZbJeA7xXVGV6tpsT/F 6c5cEqMPQ0o9YDVMFfWavFQnIlBQU+YphWKcPoB/s3JNzNMJ7ed1QCX19wmjX9E05oqZ YrMC0VW5GgGJ2Cl3Q7dHyj2I3l17J7QHGUqIfiiM5SqQrKM5JJE9mjAXwqmyRd8UDu1I iR6sJJgued61POzIbL0Ii073PV2/LvZMMZAsF20ba7+8bOjZ68dDPtc8xt8H5emsjMlh +07rwk6tfbF0khazcWz002PbJ5Xyu5ct95kcwY5LS84zt9LyYDj5DLyILdY3YlCBvlJP DQ== Received: from nasanppmta02.qualcomm.com (i-global254.qualcomm.com [199.106.103.254]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 3kfahca1y9-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Thu, 27 Oct 2022 09:19:56 +0000 Received: from nasanex01c.na.qualcomm.com (corens_vlan604_snip.qualcomm.com [10.53.140.1]) by NASANPPMTA02.qualcomm.com (8.17.1.5/8.17.1.5) with ESMTPS id 29R9JtoD027680 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Thu, 27 Oct 2022 09:19:55 GMT Received: from hu-mojha-hyd.qualcomm.com (10.80.80.8) by nasanex01c.na.qualcomm.com (10.45.79.139) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.986.29; Thu, 27 Oct 2022 02:19:52 -0700 From: Mukesh Ojha To: , , CC: , , , Mukesh Ojha Subject: [PATCH v2 1/2] f2fs: fix the assign logic of iocb Date: Thu, 27 Oct 2022 14:42:40 +0530 Message-ID: <1666861961-12924-1-git-send-email-quic_mojha@quicinc.com> X-Mailer: git-send-email 2.7.4 MIME-Version: 1.0 X-Originating-IP: [10.80.80.8] X-ClientProxiedBy: nasanex01b.na.qualcomm.com (10.46.141.250) To nasanex01c.na.qualcomm.com (10.45.79.139) X-QCInternal: smtphost X-Proofpoint-Virus-Version: vendor=nai engine=6200 definitions=5800 signatures=585085 X-Proofpoint-ORIG-GUID: R5lVbCzdee-6cVQiQTaQ7LAAcnteaSjr X-Proofpoint-GUID: R5lVbCzdee-6cVQiQTaQ7LAAcnteaSjr X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.205,Aquarius:18.0.895,Hydra:6.0.545,FMLib:17.11.122.1 definitions=2022-10-27_03,2022-10-26_01,2022-06-22_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 mlxscore=0 suspectscore=0 priorityscore=1501 mlxlogscore=999 malwarescore=0 bulkscore=0 phishscore=0 spamscore=0 impostorscore=0 clxscore=1015 adultscore=0 lowpriorityscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2210170000 definitions=main-2210270052 X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE, SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1747832875755672346?= X-GMAIL-MSGID: =?utf-8?q?1747832875755672346?= commit 18ae8d12991b ("f2fs: show more DIO information in tracepoint") introduces iocb field in 'f2fs_direct_IO_enter' trace event And it only assigns the pointer and later it accesses its field in trace print log. Unable to handle kernel paging request at virtual address ffffffc04cef3d30 Mem abort info: ESR = 0x96000007 EC = 0x25: DABT (current EL), IL = 32 bits pc : trace_raw_output_f2fs_direct_IO_enter+0x54/0xa4 lr : trace_raw_output_f2fs_direct_IO_enter+0x2c/0xa4 sp : ffffffc0443cbbd0 x29: ffffffc0443cbbf0 x28: ffffff8935b120d0 x27: ffffff8935b12108 x26: ffffff8935b120f0 x25: ffffff8935b12100 x24: ffffff8935b110c0 x23: ffffff8935b10000 x22: ffffff88859a936c x21: ffffff88859a936c x20: ffffff8935b110c0 x19: ffffff8935b10000 x18: ffffffc03b195060 x17: ffffff8935b11e76 x16: 00000000000000cc x15: ffffffef855c4f2c x14: 0000000000000001 x13: 000000000000004e x12: ffff0000ffffff00 x11: ffffffef86c350d0 x10: 00000000000010c0 x9 : 000000000fe0002c x8 : ffffffc04cef3d28 x7 : 7f7f7f7f7f7f7f7f x6 : 0000000002000000 x5 : ffffff8935b11e9a x4 : 0000000000006250 x3 : ffff0a00ffffff04 x2 : 0000000000000002 x1 : ffffffef86a0a31f x0 : ffffff8935b10000 Call trace: trace_raw_output_f2fs_direct_IO_enter+0x54/0xa4 print_trace_fmt+0x9c/0x138 print_trace_line+0x154/0x254 tracing_read_pipe+0x21c/0x380 vfs_read+0x108/0x3ac ksys_read+0x7c/0xec __arm64_sys_read+0x20/0x30 invoke_syscall+0x60/0x150 el0_svc_common.llvm.1237943816091755067+0xb8/0xf8 do_el0_svc+0x28/0xa0 Fix it by copying the required variables for printing and while at it fix the similar issue at some other places in the same file. Fixes: 18ae8d12991b ("f2fs: show more DIO information in tracepoint") Signed-off-by: Mukesh Ojha Reviewed-by: Chao Yu --- Changes in v2: - Added commit text with crash information. - Fixed the similar issue at some other places in the file. - Only copy the required variables instead of memcpy. include/trace/events/f2fs.h | 34 +++++++++++++++++++--------------- 1 file changed, 19 insertions(+), 15 deletions(-) diff --git a/include/trace/events/f2fs.h b/include/trace/events/f2fs.h index c6b3724..ff57e7f 100644 --- a/include/trace/events/f2fs.h +++ b/include/trace/events/f2fs.h @@ -322,7 +322,7 @@ TRACE_EVENT(f2fs_unlink_enter, __field(ino_t, ino) __field(loff_t, size) __field(blkcnt_t, blocks) - __field(const char *, name) + __string(name, dentry->d_name.name) ), TP_fast_assign( @@ -330,7 +330,7 @@ TRACE_EVENT(f2fs_unlink_enter, __entry->ino = dir->i_ino; __entry->size = dir->i_size; __entry->blocks = dir->i_blocks; - __entry->name = dentry->d_name.name; + __assign_str(name, dentry->d_name.name); ), TP_printk("dev = (%d,%d), dir ino = %lu, i_size = %lld, " @@ -338,7 +338,7 @@ TRACE_EVENT(f2fs_unlink_enter, show_dev_ino(__entry), __entry->size, (unsigned long long)__entry->blocks, - __entry->name) + __get_str(name)) ); DEFINE_EVENT(f2fs__inode_exit, f2fs_unlink_exit, @@ -940,25 +940,29 @@ TRACE_EVENT(f2fs_direct_IO_enter, TP_STRUCT__entry( __field(dev_t, dev) __field(ino_t, ino) - __field(struct kiocb *, iocb) + __field(loff_t, ki_pos) + __field(int, ki_flags) + __field(u16, ki_ioprio) __field(unsigned long, len) __field(int, rw) ), TP_fast_assign( - __entry->dev = inode->i_sb->s_dev; - __entry->ino = inode->i_ino; - __entry->iocb = iocb; - __entry->len = len; - __entry->rw = rw; + __entry->dev = inode->i_sb->s_dev; + __entry->ino = inode->i_ino; + __entry->ki_pos = iocb->ki_pos; + __entry->ki_flags = iocb->ki_flags; + __entry->ki_ioprio = iocb->ki_ioprio; + __entry->len = len; + __entry->rw = rw; ), TP_printk("dev = (%d,%d), ino = %lu pos = %lld len = %lu ki_flags = %x ki_ioprio = %x rw = %d", show_dev_ino(__entry), - __entry->iocb->ki_pos, + __entry->ki_pos, __entry->len, - __entry->iocb->ki_flags, - __entry->iocb->ki_ioprio, + __entry->ki_flags, + __entry->ki_ioprio, __entry->rw) ); @@ -1407,19 +1411,19 @@ TRACE_EVENT(f2fs_write_checkpoint, TP_STRUCT__entry( __field(dev_t, dev) __field(int, reason) - __field(char *, msg) + __string(dest_msg, msg) ), TP_fast_assign( __entry->dev = sb->s_dev; __entry->reason = reason; - __entry->msg = msg; + __assign_str(dest_msg, msg); ), TP_printk("dev = (%d,%d), checkpoint for %s, state = %s", show_dev(__entry->dev), show_cpreason(__entry->reason), - __entry->msg) + __get_str(dest_msg)) ); DECLARE_EVENT_CLASS(f2fs_discard, From patchwork Thu Oct 27 09:12:41 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Mukesh Ojha X-Patchwork-Id: 11692 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a5d:6687:0:0:0:0:0 with SMTP id l7csp125977wru; Thu, 27 Oct 2022 02:30:50 -0700 (PDT) X-Google-Smtp-Source: AMsMyM7XLJ5TpKQFzKQARN+KK9FtzIFeb/qquAspBUoZgv39YMAO30nBAsT831OrGPHstSdw8JgK X-Received: by 2002:a05:6a00:174c:b0:565:c73a:9117 with SMTP id j12-20020a056a00174c00b00565c73a9117mr47751424pfc.23.1666863050326; Thu, 27 Oct 2022 02:30:50 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1666863050; cv=none; d=google.com; s=arc-20160816; b=OgTjbavlhTTTVwK7amgJeStv9jMrtWzszRmHEzndi8urN2kEz6gdtkpidwXcQFRzyt BZS/9gyGpxRLNPYuXnv7lH7f3OUn/k5voS9hckAKaJ7ljaF554h+ZXkAZugThL3ssumi Vspc9ggzV0U68H8CxSaMID+C6txJoCoADuQQyA4AiGUUsbMYU04iIKr0odTDBHamKsU9 Fx3ECgtfiWAnT+kIPW5m2bqIsI34Rs9CIFcu/0qqp4UhyjB83JLhhv0ApUZM2o7rYDLa RlD4lbhjg/VarbvR+EoWjWB9/fBe2CukaIJqZr9GHgM6nrvUp+exriJcxZMu4EO8pQsc quCQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:mime-version:references:in-reply-to:message-id :date:subject:cc:to:from:dkim-signature; bh=X4HZsC2UHwmsaiEO54dlFpkcDdriC3RIpdAYueIPt+w=; b=UgOv9tw9XBKcTRNac1+CK3A69Mvv5DoupONAFm/9aNeVhtKZ5KeNVb2KYpUZF9Uoom GCsCcPbs6b5+iBFYMZbk3fVPGRKw5R59Q2oOxL7lkXXe8LziPweXE6uq6xTypyIbkBpo t7dOiS2yy6EZqP4VcAn9/O8KsUODjBEW890dtmdvW6/v0H+9fEZ0inIxjIFaH4uIKNMK QWTG41whnFo6Suwlc4fODpiE3dU31nlbl4UrF5hrUwcwecz8pLI8b0pL3p9jdtWuRfrN KiUX51kfH6gyRNTqoTx47sV/PYv3z+OoNoPqZBnyPQiaVRoFbB6YEsTUup8QTrbrNy32 YD1w== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@quicinc.com header.s=qcdkim header.b=lFkp7Q5i; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=quicinc.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id qa2-20020a17090b4fc200b0020d3d140012si5455624pjb.45.2022.10.27.02.30.35; Thu, 27 Oct 2022 02:30:50 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@quicinc.com header.s=qcdkim header.b=lFkp7Q5i; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=quicinc.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235323AbiJ0JUM (ORCPT + 99 others); Thu, 27 Oct 2022 05:20:12 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:39606 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234605AbiJ0JUF (ORCPT ); Thu, 27 Oct 2022 05:20:05 -0400 Received: from alexa-out-sd-01.qualcomm.com (alexa-out-sd-01.qualcomm.com [199.106.114.38]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id EC2951707E for ; Thu, 27 Oct 2022 02:20:03 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=quicinc.com; i=@quicinc.com; q=dns/txt; s=qcdkim; t=1666862403; x=1698398403; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version; bh=X4HZsC2UHwmsaiEO54dlFpkcDdriC3RIpdAYueIPt+w=; b=lFkp7Q5i+uKX3PkeCHCAU+SZftjBd9L+9/XjFBwXzj06EcmBa5KvjVp5 EPlNXGX1gvcEa0nIvZJ3K5TR7NZnNH4I+NAwhsBdH3RfFcgk/H4eZJdRg Ew6FrZ/Hce9WlCBbxU8zPUZ0t/P0OUDXtLFivtKENgQkCO5w4yrBXQ0mt k=; Received: from unknown (HELO ironmsg04-sd.qualcomm.com) ([10.53.140.144]) by alexa-out-sd-01.qualcomm.com with ESMTP; 27 Oct 2022 02:20:02 -0700 X-QCInternal: smtphost Received: from nasanex01c.na.qualcomm.com ([10.45.79.139]) by ironmsg04-sd.qualcomm.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 27 Oct 2022 02:20:01 -0700 Received: from hu-mojha-hyd.qualcomm.com (10.80.80.8) by nasanex01c.na.qualcomm.com (10.45.79.139) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.986.29; Thu, 27 Oct 2022 02:19:58 -0700 From: Mukesh Ojha To: , , CC: , , , Mukesh Ojha Subject: [PATCH 2/2] f2fs: fix the msg data type Date: Thu, 27 Oct 2022 14:42:41 +0530 Message-ID: <1666861961-12924-2-git-send-email-quic_mojha@quicinc.com> X-Mailer: git-send-email 2.7.4 In-Reply-To: <1666861961-12924-1-git-send-email-quic_mojha@quicinc.com> References: <1666861961-12924-1-git-send-email-quic_mojha@quicinc.com> MIME-Version: 1.0 X-Originating-IP: [10.80.80.8] X-ClientProxiedBy: nasanex01b.na.qualcomm.com (10.46.141.250) To nasanex01c.na.qualcomm.com (10.45.79.139) X-Spam-Status: No, score=-4.4 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED,SPF_HELO_NONE, SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1747832589950438104?= X-GMAIL-MSGID: =?utf-8?q?1747832589950438104?= Data type of msg in f2fs_write_checkpoint trace should be const char * instead of char *. Signed-off-by: Mukesh Ojha Reviewed-by: Chao Yu --- include/trace/events/f2fs.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/trace/events/f2fs.h b/include/trace/events/f2fs.h index ff57e7f..7fbfce4 100644 --- a/include/trace/events/f2fs.h +++ b/include/trace/events/f2fs.h @@ -1404,7 +1404,7 @@ TRACE_EVENT(f2fs_readpages, TRACE_EVENT(f2fs_write_checkpoint, - TP_PROTO(struct super_block *sb, int reason, char *msg), + TP_PROTO(struct super_block *sb, int reason, const char *msg), TP_ARGS(sb, reason, msg),