From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx0a-00069f02.pphosted.com (mx0a-00069f02.pphosted.com [205.220.165.32]) by sourceware.org (Postfix) with ESMTPS id D04543858433 for ; Fri, 23 Jun 2023 04:45:15 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org D04543858433 Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=oracle.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=oracle.com Received: from pps.filterd (m0246629.ppops.net [127.0.0.1]) by mx0b-00069f02.pphosted.com (8.17.1.19/8.17.1.19) with ESMTP id 35N4dKgN028127 for ; Fri, 23 Jun 2023 04:45:15 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.com; h=from : to : cc : subject : date : message-id : in-reply-to : references : content-transfer-encoding : content-type : mime-version; s=corp-2023-03-30; bh=LQJPIpdJNsH5MBq43bNzCMQ7f8cRt+30xCqaktLOazI=; b=3W4L6FNM348SYYYHZ0DK8M6KOJnsl8/lE40yNhqO2nhiSkb7EKymjjQ/YNG1qk7/PTcz aT8U4FRFeSJoannkXg4T0ND9ZXys5mzzJ98RKpb+WNOJTEFkrXSDHVJFpSXgsfTYxpsA FSoykBtYBBeDMcG1uYMvMzha2ZrKD1+nJvdKcsAO5NA+Zt9Ynkt8oIj2IBpaDmaH4CiH XZyuKeZNCu0PofNL4oCXaJI9YFCopLPRyKRYb3Zk5pwU7qjl1ZVTgB+Gh8ThvoBjk5jh 0FvUpY6jcPiRhKxOKSz1Ka4OJk/UVFBTnbTtmWBXuhnCnIDrgUPx6aDPJBc3EyhH2ADN 7g== Received: from iadpaimrmta01.imrmtpd1.prodappiadaev1.oraclevcn.com (iadpaimrmta01.appoci.oracle.com [130.35.100.223]) by mx0b-00069f02.pphosted.com (PPS) with ESMTPS id 3r94qab17u-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK) for ; Fri, 23 Jun 2023 04:45:14 +0000 Received: from pps.filterd (iadpaimrmta01.imrmtpd1.prodappiadaev1.oraclevcn.com [127.0.0.1]) by iadpaimrmta01.imrmtpd1.prodappiadaev1.oraclevcn.com (8.17.1.19/8.17.1.19) with ESMTP id 35N2ASsS038671 for ; Fri, 23 Jun 2023 04:45:13 GMT Received: from nam12-mw2-obe.outbound.protection.outlook.com (mail-mw2nam12lp2041.outbound.protection.outlook.com [104.47.66.41]) by iadpaimrmta01.imrmtpd1.prodappiadaev1.oraclevcn.com (PPS) with ESMTPS id 3r9398g437-2 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK) for ; Fri, 23 Jun 2023 04:45:13 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Yud45leSuDUC1q3L5ezsRg532lhV1aS60OI8zWuJ43s8HXbdplifGVsQf5JA23YRzwVt+BUnsJNhGDVOEZ/fx2QVC9LpwwgTU/33bWsWEpBeaVxvOWxokDSTg90LLFW3ieuLJW0lAOJ60Y6w6hdaPJgBzOxWuVkqsJcayMmeIUZRMRo3UgVxgN1YCX6VsgXiAz7ffpBqv4HYOPPikl2gKOjbuy/4nvdntuRwQXcsqWqkVYLvgtKyLP+2Nfr4SK4inE0FRVdP1vE2L1hwMG3PihuP95Xg3gbALU6fuSmv6Kw227eWmK3zSKtxHLpP7MGQL25P4cU7WfJqyoVnZS4ouw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=LQJPIpdJNsH5MBq43bNzCMQ7f8cRt+30xCqaktLOazI=; b=ERGzsvH6kv3EwtIcFn4cBwI24vTokbc31CG1JCPXi8N9hZLUYaH8R/0YCtzGaKmgcggxADzoCqaDVrvJj/pBrjpR83IVWh4wgWRbMGB8hCchHQUWiMO+Bm2gAO77THJyE8EQCUw66hhok9Q/AbAo1LiLztm8wRKk2FDzN1qofc6CSemzNVSW0nMeRRhu5NZm98dwYZvAAwXuqRlJ0ITMDEGrKKtRVJ4NcCd/jYG+hCGfUF8b4XOIO74Z1s87vgYFxpVgqzEreNZBl/8ey76W8NyZ489lDKK6uBh6LHrcQuKXDDsMXgD5LxbAnJZlhx8zyU6IKH4550fL5Ha17F/c0g== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=oracle.com; dmarc=pass action=none header.from=oracle.com; dkim=pass header.d=oracle.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.onmicrosoft.com; s=selector2-oracle-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=LQJPIpdJNsH5MBq43bNzCMQ7f8cRt+30xCqaktLOazI=; b=zwwxENGQTmOkp9hiHVLKq0TvZw6b/+w7/YagfuZCS8ZYa4R+W+kejEkdX9NCLsR1ab991C906cWqOlnL6B7NI3W2WgQQnBSnXebp3WPaKB5ob/AjhMjM81oaMc85NSh+B2WMFRkCeIfbnHJY4/6I5XOHkCSPl2Dlt42RxSSYtnE= Received: from MWHPR1001MB2158.namprd10.prod.outlook.com (2603:10b6:301:2d::17) by DM4PR10MB5966.namprd10.prod.outlook.com (2603:10b6:8:b2::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6521.24; Fri, 23 Jun 2023 04:45:10 +0000 Received: from MWHPR1001MB2158.namprd10.prod.outlook.com ([fe80::fdbb:b921:1ef:2d44]) by MWHPR1001MB2158.namprd10.prod.outlook.com ([fe80::fdbb:b921:1ef:2d44%4]) with mapi id 15.20.6521.026; Fri, 23 Jun 2023 04:45:10 +0000 From: Indu Bhagat To: binutils@sourceware.org Cc: Indu Bhagat Subject: [PATCH 05/10] libsframe: update the semantics of sframe_fre_get_fp_offset Date: Thu, 22 Jun 2023 21:44:43 -0700 Message-Id: <20230623044448.2617101-6-indu.bhagat@oracle.com> X-Mailer: git-send-email 2.39.2 In-Reply-To: <20230623044448.2617101-1-indu.bhagat@oracle.com> References: <20230623044448.2617101-1-indu.bhagat@oracle.com> Content-Transfer-Encoding: 8bit Content-Type: text/plain X-ClientProxiedBy: MW4PR04CA0093.namprd04.prod.outlook.com (2603:10b6:303:83::8) To MWHPR1001MB2158.namprd10.prod.outlook.com (2603:10b6:301:2d::17) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: MWHPR1001MB2158:EE_|DM4PR10MB5966:EE_ X-MS-Office365-Filtering-Correlation-Id: 1cbda64f-5664-4a52-f7d3-08db73a4a023 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: Se0Kf2JjIXT5gpwX69N50R6lz2LmxK1qSSOAtep0wh14vKP7ngChs46iLQfz9tc8jRfo9IvrAePM60EvJdY/ohvM2UfK4SqlUoPo/YM8Uzb1ux54mjMWos+Z3raBp8fjSX3amohce5ys+lojwBPB0dmx8dvsOzE+PU9Yha3uV85IFgW2dwdEOKF2HNhFKEk3c3YvXt2WG42ww2d096nKhZk4s+gdOlxlHGda1OVntdkvTtTUoHXeBnEefzkaZya25WHDyZ8Ihtt29PWQ0SGMnZo9Tev9+9aWJAFeVL7OVwBY4c4NGdj44ax12ld36ai0RA63t1A/ZDdiIqsinbNl2yBVvxKHt065AQgRwWFqbAufJIGhsittVqkWu8+1CGPjMllkDVUQE3Y3zk32OUGxQtnMWMrYJBVh6N0eXsKs7JD2aojNl+pgLDjPzHRvxjGJQZ8E/YSIT2dl78CIGcTSToqOMbvtPbIZJp0yqSu2Ue4YknslcBGFBG4YOWPe56B7+d10K5MrVu6lFzlRjAH9Nl2XU2mQysIRUVXSKzThmXNIvKs+802HEzhnbbYm2ok8 X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:MWHPR1001MB2158.namprd10.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230028)(396003)(136003)(346002)(366004)(39860400002)(376002)(451199021)(2906002)(478600001)(38100700002)(2616005)(36756003)(6666004)(5660300002)(6486002)(66946007)(66556008)(66476007)(83380400001)(316002)(6916009)(86362001)(41300700001)(44832011)(8936002)(8676002)(4326008)(6506007)(6512007)(1076003)(107886003)(186003)(26005);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?ByE/ZNrLuSCjVJ1JUxcSOv10U7IpTBVZxIEoY0hQBRT2xssLzirOX3PNikKN?= =?us-ascii?Q?CceiuHvlf4arwA1wNVYa2qVCnMNdyQneOVPb2xy6vNB68wRxONl+A0iDx3vS?= =?us-ascii?Q?CA6202jdPvrRlwKPNBEr+qKS/PVzrwDhuFwKXfdfv1MAZl5HjeFhTTwbS6sH?= =?us-ascii?Q?vxXHpJif3p+5X4os2iVGLeRGAoP7vCy/KpoGZY2qseyfoSde1WpZztw8YQCf?= =?us-ascii?Q?nhPRFlwkapd5WyLvVJM4uMt1++SW6JfwhYTbIL1ge6cveC8m9kzg0XRMh7T4?= =?us-ascii?Q?opT4fImUlR5ON1RCvc54urBSS3GSKgSRbx8IPoxFPL3sguhPm32awyBKSx8D?= =?us-ascii?Q?U83YEQQxpJaqKbFBDbac6E6Xf5O7R4+La3EYwvgtJQH5CS/EtUKRN7r+lSUb?= =?us-ascii?Q?EKJwwd/1a58GHqrerXTJSK6kHFgW3jQGyZq/n5C8aSsNRhPrqHhdmetwrD24?= =?us-ascii?Q?JRGWXWqfsO+CTjtAEnvfahpXYZ+g5QdbkHR23WPB+PhNXeR9f0jNXMQ6kkF/?= =?us-ascii?Q?CoNkpprueHEgp4V8YjSTCOv8TTTMf9aXWWnbxr4+ji3FBFaXfSaGwvn1/PV0?= =?us-ascii?Q?zwfvDKPZRj5c+v1mv7jfEcf2fcQidBA3FjPHvwOUFOuH8Pq/fvsRhXSF1RB5?= =?us-ascii?Q?zKttYJ1vkwXUAKzxiXSmbfOlfSAD3S+uk0rW+xIg0SKFDLCSQqL0v8go5dTL?= =?us-ascii?Q?NeCFjfmt302MuSpgP5rQdfiQP0i96k1i2JmM1YMPg7VyvC+kvpIvOQWyq+7O?= =?us-ascii?Q?+EcfnFCblly6K+948OvfbhLEzr0NEinSqQlA3eVQPA5UGiu0mBUh1LNPrBnU?= =?us-ascii?Q?ASco39xyj9SoswgEz83xVuvt0E9xmBAyh5JadvHj9PaAl2CT3ogKiOMwpHuH?= =?us-ascii?Q?6Af1o0K9yrQ8R7vkDEI+uxn3EmMQeI96FSmJnxLJPrRaXI3f6P1pGiqzSEWa?= =?us-ascii?Q?VPE4hEvmWx8pa0CUL1Dlpm5aonlkqiflmQiu5UA2bBnQHJooyy6hBPD/TcGy?= =?us-ascii?Q?30CHBUAy7QxHClcyNza29rkm9DpSQ3lXdSoJksiKOIcjuZSKvdiLIuLEu3Fr?= =?us-ascii?Q?UwRVuyOcIS4Ji5bE7C2NlXKarQBijO2I5nkP2CvfHS86rZG9fPsfYXlre2O3?= =?us-ascii?Q?ffWAmE2J8LKTz5YXdEjcbQjSuo4/W8fHfclFVNfj5mQcnukUIHIEdQCO3vhc?= =?us-ascii?Q?7GS3XVzVX3fT20m/AYYx/vsDLF/9zn/o2tN4oI2quMWFhCrlSRgyn4rCq1lj?= =?us-ascii?Q?c9BHut13yh4RVXGGobDwMXf/OCVm4Q942pifkFuG51YZoCoGZcf7TPxGtwTb?= =?us-ascii?Q?1SNee3iRnGLufvjm4tSkTIw3o0kJN8of6Cpn/RJrU6s4FUjkEVu09Q+EIhhw?= =?us-ascii?Q?TMWt96prKpiT5Yp9Fg89dYmdCeTHxODHhoYKpZ91716HH+2Y2MHYRfvQK7uc?= =?us-ascii?Q?/bT/bLVGc7vUsU2WcNxlzOBr3gY2z3GBpBwOpD6YNYChKPGhzxJdraWM/jou?= =?us-ascii?Q?jqnB5fT2rmZHo7+KCWUvAtY2yezdqdwU+94SOKkd+rJoInYGEIi1WTDsLqjn?= =?us-ascii?Q?rQlRkIFPMyPNw+ax8NlYxJ03lom8ExkzmGdxh4mt?= X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: LGfY1w5qlU3BlKSoIyPkDrrKu8jsWgr/7FsJ4Q3zHuyuJCjP65jDSDR4YkxbgMPvB+BgxK0IJ0U2hRTwAi7mQe8ILOMfTDY5uiG0BWMH7AmioJjylo9NWEQH23oghgfxExyOY1hSpw3a7sSS3nN2RnUBAAAGURiTQyh4MG1s+EMjjLAJjNS7Y89Ew8fHIvAwLpa94CzaNsSmADIuIh0b4dSr08gtqLwbBzmTSTwnImFPwEYAWn2SZ6Cj2u6JKejBPI9ftoYEAVigtqNfjg8dNk/mup3FGpZkFsaADsJUE1jRaKic74n/fbWxUWtSTR2YDORpSxb9xWU0TpPCIJnV1xRUknzP8dmjQAR60anAM6X8PB8DSj+wTQdxVM33Jh+H+cQLRHK9IE2ahgTlAWbKYBxQoGC8p8kS2Mu2cks9r/aFV6qn8jfTW9C4/9NSyidB5a41cb3Q5FlniQP4wUOSQTGKD53uqq+ciJ8B7WG2lmK9A73Yy7nsF5HEn1O/Gmu/7IDFmLFgD7VfnxS2po0zOecoSVItf6h9jt+Ew3f3JHOHTUBfOPwM+ayQfeMi1prj0Gkw4kGMQJVX8wXi7Nnp56jZDVPukGsZFkf4XN1/14EYVuYScXh3MqAdJOwgOehR7Bweseu3+VqMBhOM0ReEw6FU0uBRL1uoxsmSXZ7nA/j562GJMBaFgilxWRQYAwrqcqz5Rc63lXXbpydlBazQkXQQ0UjZ1qKlspv0CFSAM1gDFxRjZvreNKhYcWm6dLKd X-OriginatorOrg: oracle.com X-MS-Exchange-CrossTenant-Network-Message-Id: 1cbda64f-5664-4a52-f7d3-08db73a4a023 X-MS-Exchange-CrossTenant-AuthSource: MWHPR1001MB2158.namprd10.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Jun 2023 04:45:10.8098 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: cnpxvV6KiEg43aNqo+qnAvzv14wwgOIQDa9dkyGbqaDF0ZULrJywA/CfxUTdw6JtU1xKhQi+Bp0ZgLidr1gXVg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR10MB5966 X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.254,Aquarius:18.0.957,Hydra:6.0.591,FMLib:17.11.176.26 definitions=2023-06-23_02,2023-06-22_02,2023-05-22_02 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 spamscore=0 bulkscore=0 suspectscore=0 adultscore=0 mlxscore=0 mlxlogscore=815 malwarescore=0 phishscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2305260000 definitions=main-2306230041 X-Proofpoint-GUID: eEK7LA4jjCUIeZt2vOd2JAXoH2dwEzRj X-Proofpoint-ORIG-GUID: eEK7LA4jjCUIeZt2vOd2JAXoH2dwEzRj X-Spam-Status: No, score=-13.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,GIT_PATCH_0,RCVD_IN_DNSWL_LOW,RCVD_IN_MSPIKE_H5,RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,SPF_NONE,TXREP,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: Until now, sframe_fre_get_fp_offset () would return SFRAME_ERR_FREOFFSET_NOPRESENT if the ABI uses fixed FP offset. A stack tracer, then, would call an explicit sframe_decoder_get_fixed_fp_offset () to get the FP offset. On second look, it appears to make sense to hide these details of whether the FP offset is fixed or not in an ABI from the consumer. Now, with the changed semantics, the call to sframe_fre_get_fp_offset () will fetch the fixed FP offset if applicable, or get the FP offset from FRE when there is no fixed FP offset. This patch changes the behavior of sframe_fre_get_fp_offset (): it turns an error into non-error. This change will be included with the next release of libsframe, where all the exposed symbols will be versioned with version node LIBSFRAME_1.0 for the first time. libsframe/ * sframe.c (sframe_fre_get_fp_offset): Return the fixed offset, if applicable. Else return the FP offset from the FRE. --- libsframe/sframe.c | 17 +++++++++++------ 1 file changed, 11 insertions(+), 6 deletions(-) diff --git a/libsframe/sframe.c b/libsframe/sframe.c index 3d1b5575f0b..912d34352a1 100644 --- a/libsframe/sframe.c +++ b/libsframe/sframe.c @@ -644,16 +644,21 @@ sframe_fre_get_fp_offset (sframe_decoder_ctx *dctx, sframe_frame_row_entry *fre, int *errp) { uint32_t fp_offset_idx = 0; - sframe_header *dhp = sframe_decoder_get_header (dctx); - /* If the FP offset is not being tracked, return an error code so the caller - can gather the fixed FP offset from the SFrame header. */ - if (dhp->sfh_cfa_fixed_fp_offset != SFRAME_CFA_FIXED_FP_INVALID) - return sframe_set_errno (errp, SFRAME_ERR_FREOFFSET_NOPRESENT); + int8_t fp_offset = sframe_decoder_get_fixed_fp_offset (dctx); + /* If the FP offset is not being tracked, return the fixed FP offset + from the SFrame header. */ + if (fp_offset != SFRAME_CFA_FIXED_FP_INVALID) + { + if (errp) + *errp = 0; + return fp_offset; + } /* In some ABIs, the stack offset to recover RA (using the CFA) from is fixed (like AMD64). In such cases, the stack offset to recover FP will appear at the second index. */ - fp_offset_idx = ((dhp->sfh_cfa_fixed_ra_offset != SFRAME_CFA_FIXED_RA_INVALID) + fp_offset_idx = ((sframe_decoder_get_fixed_ra_offset (dctx) + != SFRAME_CFA_FIXED_RA_INVALID) ? SFRAME_FRE_RA_OFFSET_IDX : SFRAME_FRE_FP_OFFSET_IDX); return sframe_get_fre_offset (fre, fp_offset_idx, errp); -- 2.39.2