{"id":2232084,"url":"http://patchwork.ozlabs.org/api/patches/2232084/?format=json","web_url":"http://patchwork.ozlabs.org/project/linuxppc-dev/patch/20260502143237.23456-1-atrajeev@linux.ibm.com/","project":{"id":2,"url":"http://patchwork.ozlabs.org/api/projects/2/?format=json","name":"Linux PPC development","link_name":"linuxppc-dev","list_id":"linuxppc-dev.lists.ozlabs.org","list_email":"linuxppc-dev@lists.ozlabs.org","web_url":"https://github.com/linuxppc/wiki/wiki","scm_url":"https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git","webscm_url":"https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git/","list_archive_url":"https://lore.kernel.org/linuxppc-dev/","list_archive_url_format":"https://lore.kernel.org/linuxppc-dev/{}/","commit_url_format":"https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git/commit/?id={}"},"msgid":"<20260502143237.23456-1-atrajeev@linux.ibm.com>","list_archive_url":"https://lore.kernel.org/linuxppc-dev/20260502143237.23456-1-atrajeev@linux.ibm.com/","date":"2026-05-02T14:32:36","name":"[V4,1/2] powerpc tools perf: Initialize error code in auxtrace_record_init function","commit_ref":null,"pull_url":null,"state":"new","archived":false,"hash":"f4b8b1973afaea02837e2c67f4e6607b73bf2fd7","submitter":{"id":90290,"url":"http://patchwork.ozlabs.org/api/people/90290/?format=json","name":"Athira Rajeev","email":"atrajeev@linux.ibm.com"},"delegate":null,"mbox":"http://patchwork.ozlabs.org/project/linuxppc-dev/patch/20260502143237.23456-1-atrajeev@linux.ibm.com/mbox/","series":[{"id":502532,"url":"http://patchwork.ozlabs.org/api/series/502532/?format=json","web_url":"http://patchwork.ozlabs.org/project/linuxppc-dev/list/?series=502532","date":"2026-05-02T14:32:36","name":"[V4,1/2] powerpc tools perf: Initialize error code in auxtrace_record_init function","version":4,"mbox":"http://patchwork.ozlabs.org/series/502532/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/patches/2232084/comments/","check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/2232084/checks/","tags":{},"related":[],"headers":{"Return-Path":"\n ","X-Original-To":["incoming@patchwork.ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Delivered-To":"patchwork-incoming@legolas.ozlabs.org","Authentication-Results":["legolas.ozlabs.org;\n\tdkim=pass (2048-bit key;\n unprotected) header.d=ibm.com header.i=@ibm.com header.a=rsa-sha256\n header.s=pp1 header.b=ZXXY0bci;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=lists.ozlabs.org\n (client-ip=2404:9400:21b9:f100::1; helo=lists.ozlabs.org;\n envelope-from=linuxppc-dev+bounces-20382-incoming=patchwork.ozlabs.org@lists.ozlabs.org;\n receiver=patchwork.ozlabs.org)","lists.ozlabs.org;\n arc=none smtp.remote-ip=148.163.158.5","lists.ozlabs.org;\n dmarc=pass (p=none dis=none) header.from=linux.ibm.com","lists.ozlabs.org;\n\tdkim=pass (2048-bit key;\n unprotected) header.d=ibm.com header.i=@ibm.com header.a=rsa-sha256\n header.s=pp1 header.b=ZXXY0bci;\n\tdkim-atps=neutral","lists.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=linux.ibm.com\n (client-ip=148.163.158.5; helo=mx0b-001b2d01.pphosted.com;\n envelope-from=atrajeev@linux.ibm.com; receiver=lists.ozlabs.org)"],"Received":["from lists.ozlabs.org (lists.ozlabs.org\n [IPv6:2404:9400:21b9:f100::1])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519)\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4g79P7395Rz1y04\n\tfor ; Sun, 03 May 2026 00:32:58 +1000 (AEST)","from boromir.ozlabs.org (localhost [127.0.0.1])\n\tby lists.ozlabs.org (Postfix) with ESMTP id 4g79P55nlKz2xtC;\n\tSun, 03 May 2026 00:32:57 +1000 (AEST)","from mx0b-001b2d01.pphosted.com (mx0b-001b2d01.pphosted.com\n [148.163.158.5])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519 server-signature RSA-PSS (2048 bits) server-digest\n SHA256)\n\t(No client certificate requested)\n\tby lists.ozlabs.org (Postfix) with ESMTPS id 4g79P44NQHz2xnh\n\tfor ; Sun, 03 May 2026 00:32:56 +1000 (AEST)","from pps.filterd (m0360072.ppops.net [127.0.0.1])\n\tby mx0a-001b2d01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id\n 6427NKNt2333320;\n\tSat, 2 May 2026 14:32:50 GMT","from ppma22.wdc07v.mail.ibm.com\n (5c.69.3da9.ip4.static.sl-reverse.com [169.61.105.92])\n\tby mx0a-001b2d01.pphosted.com (PPS) with ESMTPS id 4dw9y495cu-1\n\t(version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT);\n\tSat, 02 May 2026 14:32:49 +0000 (GMT)","from pps.filterd (ppma22.wdc07v.mail.ibm.com [127.0.0.1])\n\tby ppma22.wdc07v.mail.ibm.com (8.18.1.7/8.18.1.7) with ESMTP id\n 642EOmWM009315;\n\tSat, 2 May 2026 14:32:48 GMT","from smtprelay01.fra02v.mail.ibm.com ([9.218.2.227])\n\tby ppma22.wdc07v.mail.ibm.com (PPS) with ESMTPS id 4dw9fs9dxd-1\n\t(version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT);\n\tSat, 02 May 2026 14:32:48 +0000 (GMT)","from smtpav02.fra02v.mail.ibm.com (smtpav02.fra02v.mail.ibm.com\n [10.20.54.101])\n\tby smtprelay01.fra02v.mail.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id\n 642EWjs735782920\n\t(version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK);\n\tSat, 2 May 2026 14:32:45 GMT","from smtpav02.fra02v.mail.ibm.com (unknown [127.0.0.1])\n\tby IMSVA (Postfix) with ESMTP id 136E320043;\n\tSat, 2 May 2026 14:32:45 +0000 (GMT)","from smtpav02.fra02v.mail.ibm.com (unknown [127.0.0.1])\n\tby IMSVA (Postfix) with ESMTP id CAF2B20040;\n\tSat, 2 May 2026 14:32:39 +0000 (GMT)","from localhost.localdomain (unknown [9.124.223.37])\n\tby smtpav02.fra02v.mail.ibm.com (Postfix) with ESMTP;\n\tSat, 2 May 2026 14:32:39 +0000 (GMT)"],"ARC-Seal":"i=1; a=rsa-sha256; d=lists.ozlabs.org; s=201707; t=1777732377;\n\tcv=none;\n b=Q9/XPBwMaX/R+8gDIEAtQeuhMPxaOldeG8vzulXzP6vtjfVtxaBKuD8o8H6wM4As+aefO/KYNfk07Wuk+4iyTyhHWHiH7LGnodZs5uKIcQQOtoz/hFzV8GjrGO3zb2/4U+Bk10tz+yr5cuB12kkIJBTo6hsmUV0zMzS3suJs0F3BLmXmnp4tHfWzwyxk9IMBlkPikaHQTAxL5b9WWqfe1RF4Zj6DpWAX5EloC9H/0VjI9bAJypGIwPQOXZA2xFnDpxWYNVJn6UiKd4VYPpPrTmXZpFWr7lm+E0P09udbAf0DeueqPQEkdiIV5JDdRM4DkHsKLrD7jbvNSkO9s/r/Vg==","ARC-Message-Signature":"i=1; a=rsa-sha256; d=lists.ozlabs.org; s=201707;\n\tt=1777732377; c=relaxed/relaxed;\n\tbh=kuHqVdlej+Yn0HOXIpjPZZdn1xMOEqKbsgmm1UXRwMU=;\n\th=From:To:Cc:Subject:Date:Message-Id:MIME-Version;\n b=f3geCBvxfwS9NdP6b5y5P/5CJa3RYXgwkIRRzBsQV16XX96eqFFUSfMqhVU93IOhsJjikQUUN60hNFOW7rQcCCxdTqsfjIYcMpZTA4REIuJU+VHkj/bJfP/IQ6v/tB9A5uW7PU+0lWOq0QaLVzRX9JAyVHsXDCSyQS4armY05YVyr05Vup5CEiNYf9SBCVHUsp3x3dSDNE9XsqT1scXhkSHcW+5KszaHHk/eY7a5nVhi8cztmhIrUJvgm7vRo2vkxBvmmcjYGui3nh5B+bfxtZhhuRzxGoUp3zxpMwb7E+nnU/eSPsylrTKpFfSjYqXoap79/fD0A0Z16jDirsiBDw==","ARC-Authentication-Results":"i=1; lists.ozlabs.org;\n dmarc=pass (p=none dis=none) header.from=linux.ibm.com;\n dkim=pass (2048-bit key;\n unprotected) header.d=ibm.com header.i=@ibm.com header.a=rsa-sha256\n header.s=pp1 header.b=ZXXY0bci; dkim-atps=neutral;\n spf=pass (client-ip=148.163.158.5; helo=mx0b-001b2d01.pphosted.com;\n envelope-from=atrajeev@linux.ibm.com;\n receiver=lists.ozlabs.org) smtp.mailfrom=linux.ibm.com","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=ibm.com; h=cc\n\t:content-transfer-encoding:date:from:message-id:mime-version\n\t:subject:to; s=pp1; bh=kuHqVdlej+Yn0HOXIpjPZZdn1xMOEqKbsgmm1UXRw\n\tMU=; b=ZXXY0bcimFS7s+UuKIRYE6gYUWW/BXtHAvxgkOyFfZ9XTVuTFlNyV/xSW\n\taz3N3Z2JFKiQEgq6vSse/rJqp3Vv22bCDXEp5c8HMFUY5WxMON1KXhce6FR8BW5f\n\ti9yzeVVnFUmWQ6YrcN721u0/T7TkmcAIqwyyPSv68HxRphmBl4YDnLNBOfpzXO7j\n\t8z5ZvDWx6WY0+5UKn+CM6HrYQAa2pr3x77bi+NveGCtZaTvKEYEcWGOQw+SaLfEY\n\t8IpXROzu5d8r1/udbTpHxNTUfpWw6EJZMGXAxljuVOUY7vnIELN0lKb4MkuZGVeK\n\tZ9HTI/BQe7LatXzOSDkbbf2qZG6qw==","From":"Athira Rajeev ","To":"acme@kernel.org, jolsa@kernel.org, adrian.hunter@intel.com,\n mpetlan@redhat.com, tmricht@linux.ibm.com, maddy@linux.ibm.com,\n irogers@google.com, namhyung@kernel.org","Cc":"linux-perf-users@vger.kernel.org, linuxppc-dev@lists.ozlabs.org,\n atrajeev@linux.ibm.com, hbathini@linux.vnet.ibm.com,\n Tejas.Manhas1@ibm.com, Tanushree.Shah@ibm.com, shivani@linux.ibm.com","Subject":"[PATCH V4 1/2] powerpc tools perf: Initialize error code in\n auxtrace_record_init function","Date":"Sat, 2 May 2026 20:02:36 +0530","Message-Id":"<20260502143237.23456-1-atrajeev@linux.ibm.com>","X-Mailer":"git-send-email 2.39.5 (Apple Git-154)","X-Mailing-List":"linuxppc-dev@lists.ozlabs.org","List-Id":"","List-Help":"","List-Owner":"","List-Post":"","List-Archive":",\n ","List-Subscribe":",\n ,\n ","List-Unsubscribe":"","Precedence":"list","MIME-Version":"1.0","Content-Transfer-Encoding":"8bit","X-TM-AS-GCONF":"00","X-Proofpoint-Reinject":"loops=2 maxloops=12","X-Proofpoint-Spam-Details-Enc":"AW1haW4tMjYwNTAyMDE0MSBTYWx0ZWRfX+GjDJc/uByfZ\n x5pXiEs0i+/wemKBf8qOSBAF900UmW9wZc2CaZiMFnbBkSzA/HKVOpx7w7XCUAEmjIQC4S2OUPB\n WE2Sptqf+QHGRjvpTWY4UyrFKQnsqpxD4UFCMS0Q+2oStG11OVtLOZUSRE3xwmoNzNhwjJTivfl\n 3KPpQ6x4ZPT5c5g/b32Q+4mUdhk9JvGSVyEHScsNl8sAlua0NNAHgGD03PIDCMijXAHtJ1a/04k\n UJCQFgfM+Rvawxqf22UorkgBBJpPMqrXFd31Qb2RFFHHS4l8aH/fNaW38/5SFIybWTC0BdBStie\n qgkcqFa0fgeebRTdJnGqRqRDMbZArfBviQKquz/vhp8ys70cOnJANaEPNYSduhdPRFJKL5qVt51\n VANLUWk/Sw/geQ8Una6eX2XicrmrNnZrOBhRYuvOh0Yfgmy8v7mKwmgTaXSQshNYpqnstuwhE3l\n 6zQT0B7AeI49opQ+TyA==","X-Authority-Analysis":"v=2.4 cv=J4GaKgnS c=1 sm=1 tr=0 ts=69f60b11 cx=c_pps\n a=5BHTudwdYE3Te8bg5FgnPg==:117 a=5BHTudwdYE3Te8bg5FgnPg==:17\n a=NGcC8JguVDcA:10 a=VkNPw1HP01LnGYTKEx00:22 a=RnoormkPH1_aCDwRdu11:22\n a=RzCfie-kr_QcCd8fBx8p:22 a=VnNF1IyMAAAA:8 a=UYfmDeaf6v5XVjAx5qsA:9","X-Proofpoint-GUID":"cfOsUC42KIWAZtrn4Zl9Swbig2OcC7po","X-Proofpoint-ORIG-GUID":"-ZT8zzJPUQgicBy1YPMdRHKy-ouycN71","X-Proofpoint-Virus-Version":"vendor=baseguard\n engine=ICAP:2.0.293,Aquarius:18.0.1143,Hydra:6.1.51,FMLib:17.12.100.49\n definitions=2026-05-02_03,2026-04-30_02,2025-10-01_01","X-Proofpoint-Spam-Details":"rule=outbound_notspam policy=outbound score=0\n impostorscore=0 clxscore=1015 malwarescore=0 bulkscore=0 suspectscore=0\n priorityscore=1501 spamscore=0 phishscore=0 adultscore=0 lowpriorityscore=0\n classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0\n reason=mlx scancount=1 engine=8.22.0-2604200000 definitions=main-2605020141","X-Spam-Status":"No, score=-0.7 required=3.0 tests=DKIM_SIGNED,DKIM_VALID,\n\tRCVD_IN_DNSWL_LOW,RCVD_IN_MSPIKE_H4,RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,\n\tSPF_PASS autolearn=disabled version=4.0.1 OzLabs 8","X-Spam-Checker-Version":"SpamAssassin 4.0.1 (2024-03-25) on lists.ozlabs.org"},"content":"perf trace record fails some cases in powerpc\n\n # ./perf test \"perf trace record and replay\"\n 128: perf trace record and replay : FAILED!\n\n # ./perf trace record sleep 1\n # echo $?\n 32\n\nThis is happening because of non-zero err value from\nauxtrace_record__init() function.\n\n static int record__auxtrace_init(struct record *rec)\n {\n int err;\n\n if ((rec->opts.auxtrace_snapshot_opts || rec->opts.auxtrace_sample_opts)\n && record__threads_enabled(rec)) {\n pr_err(\"AUX area tracing options are not available in parallel streaming mode.\\n\");\n return -EINVAL;\n }\n\n if (!rec->itr) {\n rec->itr = auxtrace_record__init(rec->evlist, &err);\n if (err)\n return err;\n }\n\nHere \"int err\" is not initialised. The code expects \"err\" to be set\nfrom auxtrace_record__init() function.\n\nUpdate auxtrace_record__init() in arch/powerpc/util/auxtrace.c to clear\nerr value in the beginning.\n\n- Clear err value in beginning of function. Any fail later will\nset appropriate return code to err.\n- Even if we haven't found any event for auxtrace, perf record\nshould continue for other events. NULL return\nwill indicate that there is no auxtrace record initialized.\n- Not having \"err\" set here will affect monitoring of other events\nalso because perf record will fail seeing random value in err.\n\nSet err to -EINVAL before invoking auxtrace_record__init() in\nbuiltin-record.c\n\nWith the fix,\n\n # ./perf trace record sleep 1\n [ perf record: Woken up 2 times to write data ]\n [ perf record: Captured and wrote 0.033 MB perf.data (228 samples) ]\n\nFixes: 1dbfaf94cf66 (\"perf powerpc: Add basic CONFIG_AUXTRACE support for VPA pmu on powerpc\")\nSigned-off-by: Athira Rajeev \n---\nChangelog:\nv1 -> v2\nAddressed review comment from Adrian:\n- Set err to -EINVAL before invoking auxtrace_record__init() in\n builtin-record.c\n- Added kernel-doc to auxtrace_record__init() in tools/perf/util/auxtrace.c\nAddressed review comment from Namhyung:\n- Added fixes tag\n\n tools/perf/arch/powerpc/util/auxtrace.c | 6 ++++++\n tools/perf/builtin-record.c | 1 +\n 2 files changed, 7 insertions(+)","diff":"diff --git a/tools/perf/arch/powerpc/util/auxtrace.c b/tools/perf/arch/powerpc/util/auxtrace.c\nindex e39deff6c857..4600a1661b4f 100644\n--- a/tools/perf/arch/powerpc/util/auxtrace.c\n+++ b/tools/perf/arch/powerpc/util/auxtrace.c\n@@ -71,6 +71,12 @@ struct auxtrace_record *auxtrace_record__init(struct evlist *evlist,\n \tstruct evsel *pos;\n \tint found = 0;\n \n+\t/*\n+\t * Set err value to zero here. Any fail later\n+\t * will set appropriate return code to err.\n+\t */\n+\t*err = 0;\n+\n \tevlist__for_each_entry(evlist, pos) {\n \t\tif (strstarts(pos->name, \"vpa_dtl\")) {\n \t\t\tfound = 1;\ndiff --git a/tools/perf/builtin-record.c b/tools/perf/builtin-record.c\nindex 4a5eba498c02..708825747af5 100644\n--- a/tools/perf/builtin-record.c\n+++ b/tools/perf/builtin-record.c\n@@ -865,6 +865,7 @@ static int record__auxtrace_init(struct record *rec)\n \t}\n \n \tif (!rec->itr) {\n+\t\terr = -EINVAL;\n \t\trec->itr = auxtrace_record__init(rec->evlist, &err);\n \t\tif (err)\n \t\t\treturn err;\n","prefixes":["V4","1/2"]}