From patchwork Wed Mar 9 08:26:07 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: He Zhe X-Patchwork-Id: 4984 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from aws-us-west-2-korg-lkml-1.web.codeaurora.org (localhost.localdomain [127.0.0.1]) by smtp.lore.kernel.org (Postfix) with ESMTP id 297BBC433EF for ; Wed, 9 Mar 2022 08:26:26 +0000 (UTC) Received: from mx0b-0064b401.pphosted.com (mx0b-0064b401.pphosted.com [205.220.178.238]) by mx.groups.io with SMTP id smtpd.web11.3402.1646814384253155772 for ; Wed, 09 Mar 2022 00:26:25 -0800 Authentication-Results: mx.groups.io; dkim=fail reason="body hash did not verify" header.i=@windriver.com header.s=pps06212021 header.b=lHAxqrJc; spf=permerror, err=parse error for token &{10 18 %{ir}.%{v}.%{d}.spf.has.pphosted.com}: invalid domain name (domain: windriver.com, ip: 205.220.178.238, mailfrom: prvs=00670e95d8=zhe.he@windriver.com) Received: from pps.filterd (m0250811.ppops.net [127.0.0.1]) by mx0a-0064b401.pphosted.com (8.16.1.2/8.16.1.2) with ESMTP id 2297EG6k031808 for ; Wed, 9 Mar 2022 08:26:23 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=windriver.com; h=from : to : subject : date : message-id : content-type : content-transfer-encoding : mime-version; s=PPS06212021; bh=TvktAlp7JAghflCKr+X9e5KrLwkNY73XIJ/vuZE7pKw=; b=lHAxqrJc0vlcVaCzRC2vGhKvl3RodCPz3zsyIKXUH1ucgYTZFHpnNzDvrBT72awx4HFr BcwkaBq4A29QiivJcpNmySJvyEEQn4bDcxCfJQAZGHq01lQ/u8/KmguqHqo3w+h3XBXe OMDCfaMMdZ4VzNe8hhXiJW80s0ZkzNECkWP2noTfSeBKmye2BJMABeTaaFZTWCnt7Yon 91yiG0QGCJbZ5aKiyEpbHCEqrz37W3syMHW+TbNUUOHpdGPBYCH0ZUwb0dFJhxe42UAF +ShWIMVXIi3YNMIRK1cjs+1SnssJP7HKMZ5ouioHz1cQ9YYH3DwgnA6fxZeIPYqPKOG0 Iw== Received: from nam12-mw2-obe.outbound.protection.outlook.com (mail-mw2nam12lp2049.outbound.protection.outlook.com [104.47.66.49]) by mx0a-0064b401.pphosted.com (PPS) with ESMTPS id 3ekw3039b1-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT) for ; Wed, 09 Mar 2022 08:26:23 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Lz9EUnv6BMsw6PlRHyIfE3ZNzvwgydNNlPfr/LJFk91fdlLP7qiI5i5iqHhIqknmOtGlKnU7u8+kKzZxRas3kdt7thDyMrL0KmYo+l50dsHox5YSrctF3IxYgiGkXGss7JIJpdclFtbjrJ+KCl0FQfiiQ2ulM3hOmeTEIhXM+ZYMvABaR7XDSnc+q+YfbkqR3dFf29jREDB2mRzKA3wYWyBpJ3c1pJ+HrBDhyVHbzd/5YwgysK8yfX2GYAUqWNiI8D1K56thoxodH6XQ8fBlxwuEs8NYDd8rHAHnDserNeIvomgvkFr6AV88/IlpGeSDVZgJ3afyxq3gI08zhgF5cA== 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=TvktAlp7JAghflCKr+X9e5KrLwkNY73XIJ/vuZE7pKw=; b=EeQBkkrLfxLi40Wor1QEOxQRKy7fe/UB12P0qlnavfSWhXAxePi/JncPKg+zNE6yobdZMLPGWXwiuwogNw+WSSQu3ls4YM7jxXeGIIQ4A/jdfQARySYps0yNaObBN5BVY97VnSsG57nHbViS0yLfe6nsWkJMtLopwcx/S5MmT5VWjHuQAK1Sf8diozr4ibnKk/QOAB40bWrad3xQOEiEqxrP9xf0wWYR/nuPIygeWmf+nNElnCrrc6JpNPUHC7TjRYzo40Xac6wvXGIOHH/oG5chDFMocSNEJ1VgBiIYdrkgwpQN6tXLgrLzyRrWdgmq6/KykCNVM/YM2frGAEKhHQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=windriver.com; dmarc=pass action=none header.from=windriver.com; dkim=pass header.d=windriver.com; arc=none Received: from MWHPR11MB1358.namprd11.prod.outlook.com (2603:10b6:300:23::8) by CH0PR11MB5524.namprd11.prod.outlook.com (2603:10b6:610:d7::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5038.14; Wed, 9 Mar 2022 08:26:20 +0000 Received: from MWHPR11MB1358.namprd11.prod.outlook.com ([fe80::2cb7:7dbc:9904:2c3e]) by MWHPR11MB1358.namprd11.prod.outlook.com ([fe80::2cb7:7dbc:9904:2c3e%5]) with mapi id 15.20.5038.027; Wed, 9 Mar 2022 08:26:20 +0000 From: He Zhe To: openembedded-core@lists.openembedded.org Subject: [OE-core] [hardknott][PATCH] babeltrace: Handle negative time and offset from Epoch Date: Wed, 9 Mar 2022 16:26:07 +0800 Message-Id: <20220309082607.2165890-1-zhe.he@windriver.com> X-Mailer: git-send-email 2.25.1 X-ClientProxiedBy: SJ0PR03CA0058.namprd03.prod.outlook.com (2603:10b6:a03:33e::33) To MWHPR11MB1358.namprd11.prod.outlook.com (2603:10b6:300:23::8) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: f8bf5b9c-4092-4681-7234-08da01a67cdd X-MS-TrafficTypeDiagnostic: CH0PR11MB5524:EE_ X-Microsoft-Antispam-PRVS: X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: nwNWVrcJGwGfYobPP28slzUFVPrvBVPeIWgRG9zTWhojrWdXs3gK2YAjutowdd5j8BMq51le605riaTFesJ9Kqd7J9DBxM+Fpg61+fUJAV/OOmI6+YJBOqugIUC0dLnGnNNC/bpWO6HDvGke9i48l8y+zb40n7ua2WgCpXG5cjEBUtiNbdpOdy0lqfTZbDvSjBVihhkzhjZY9RHavWYtqCmdZR8ZKgdtIABo9SISV5cuxWnNuRlfmzcr340S9nlmRplP3swihOG5Mg0sc/vCYLaYThhDE5mCXL1w52t9cy7WpMGJS1UtBmKqJKGlK8PvtxEzV0Ii/fsmopQfN21uf3VyXjWvOuZcqslnyPigNHPpKzH5jnh1n8EjXpZOKhXQn032syu71lOo353tDh5ZDyHBwBDbPXGglMroHWzB3dmlVvHMGKqKV5hCoae6g7n0mnzPB6kxzDpsRzMaLibPUD/0fKl7sw5UggHQaNWoEuB68z/YTMlDPu6B/57PdEvA22Gvll5XCSNazl4FOcjYpB6pVVQW42l5tgaomlM7afXnTPx7LDC2hsx/aeiy6QpR5V0b+i3vf1aUY6cK/DGIhL/ZA5rJutHOSOdR8UxjApXmOnv93BDZNCd1xyDHxSS26hR3Y56fcS+pA7vME6Wipb1u29NBCuDfI/8+SaEle7/blDa/Xstl5ctn4UiMNq1rzt7HBrMn2pIU6aESav/HdaF/916l3UH6M3p/34E1tSzra9ZhPGo+x5ml2FlzN4aUVy0yUXHeCRz+2+eFqP2li79dXTXra3l0jeproU6vFyw= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:MWHPR11MB1358.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230001)(4636009)(366004)(6486002)(508600001)(316002)(6506007)(52116002)(86362001)(1076003)(6666004)(2616005)(36756003)(186003)(26005)(83380400001)(6512007)(66574015)(66556008)(38100700002)(66946007)(66476007)(8676002)(38350700002)(2906002)(8936002)(6916009)(5660300002)(30864003);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?Y5UrIfkAxtEx67mZ6gh5InWyrO3C?= =?utf-8?q?z2K4kt7P8Ftw1Zas5bfU8gfk/TVws2fRiG8hdlGG4uU5SCuVJYQLG6gEg9rZmd0Jj?= =?utf-8?q?RZGy/yopCHtSkDMwEcStBPS8Tr/BhSyS0Xj+rMO1J90M+3hOSq1YZe2d98CUw2OKD?= =?utf-8?q?DxP5XcwVinkXEp+lmeY53YhtcDFKg2akhYQl+TwR1PVSAByIH4+W6mcgJu2eNjWbj?= =?utf-8?q?9QTt+DuPK0IEOwbHh3H/TLAj746grDm7esUKgVYCphURzmac2ickzVrcjLxg9nGiL?= =?utf-8?q?Tzf6yrGsj4mkJZon2s74W19r4Ma4OumEkh5NoQIKE9B4/uFter9JHMmPBgQ1qZa5t?= =?utf-8?q?mlt1hFcHzUI2n9A+3Eu0KjXezl3b4jm8RWvo7BJ3A0F/h23YigSDTaBFSlSVLTN3Y?= =?utf-8?q?zSfD6csoE/8t+cEUqqBjvI8zO0j7dv668pjj05RiFgOq6e5HdwHZ2ts56oP2b8zke?= =?utf-8?q?M3wRRMEz6alxXmG8EY3M6VNa9dB//Q+D3xX9jgIZVTNF9gjVjdHFeWAFnIinCDg/L?= =?utf-8?q?9/oSq4SKhaAW+xSerojzrpTyhdB86d5ZXetr5oopjjo+3vC29xsJukESd+LgXHnO3?= =?utf-8?q?SliUjjSc2l+rCfafAWG9JumfVu7EBudk/yXqyBiFHAQrfZT9b0Y4CRmrjqjTXIl3r?= =?utf-8?q?KwApGuN/Cfo7ob6GGW2CL4ZlGnpuypvqqU5LoOaaH55wn4M3nocXVswaBAYZb3bc2?= =?utf-8?q?tTtvzPk7VGeAfWKmd0IY135DUU+9f6LXtPVhQ5jR46r9Zh5Hm+NLM/z9+HwfwF3SD?= =?utf-8?q?XFMT6NKhzjGU3FPMsq9jIeqhmlaUW1CCYilnfpSKQYzkceC08dcgBFRyKqZPmM6IL?= =?utf-8?q?BeUGciHyab9xYNVqEXIeL9paOt0uBDGQWt2rMbQL+Xs9v9R6PBez+GhAKAGfF0222?= =?utf-8?q?8BKAgM/sHiL0YeyeS8pQYy9YoPBzboj2lt9dhVXqfN9Cl4EdpnryL9kBj+zqDmy3V?= =?utf-8?q?wMYl4Jb/PUQ4xBcKe7yltnSDfiUgHALQchQGvQrAZg0SV1JOp2fuNXuCu2vg82fPr?= =?utf-8?q?8Elw8WSV8JY7WLXgWGaM70VI7hN+RUeFz9zRa9utq1E2lhtYd58g93+ksFBZtAmSN?= =?utf-8?q?zhQKEefV23oD9sKX1eLZ7nmC+6iY3i9tqW6buyFTO7H5gOb8Zc9dApDMs0r3WwrEY?= =?utf-8?q?rUDQ92ar0uavuCy6Itelvr304sRqeORbehXS831rTtvsQ8izXFtJqiw77JE5GU/mU?= =?utf-8?q?Ts1B9c1Z0fy+sR9pkTn0w6cTFso3+scq1sKRMIvCLPsmCppqzorsdAoQKY9dzOie/?= =?utf-8?q?Z14nHiODdbKSTULfAaC3JWEd1gYBEx+xVqv5P8rrw2tifXdSil0eAhaYVUBeEsWZl?= =?utf-8?q?fupnZOaNyX3kP4zvWTA/o5a+XWttCVKcGIZfaiSD+lwVMG8/s5oMqlYab8hFpPmN7?= =?utf-8?q?X9bIpdu9cN8E4zPLVILa0akUFR3osMmEd7ZIDh/HqCAyhCkOO4KS/pQE9OXRab9uv?= =?utf-8?q?HFvNpehTAyXY1N9X/KxkL0VSxXUN5CsDAO1/rysvsVh6D4O3xDvM7HD3xW+Fqt7Op?= =?utf-8?q?G8YH8+HCgMXeoLL0C63TYYLnBL6kA2gCP8s2qnmYHkdsS1pTu0Tdv0A=3D?= X-OriginatorOrg: windriver.com X-MS-Exchange-CrossTenant-Network-Message-Id: f8bf5b9c-4092-4681-7234-08da01a67cdd X-MS-Exchange-CrossTenant-AuthSource: MWHPR11MB1358.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Mar 2022 08:26:20.2120 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 8ddb2873-a1ad-4a18-ae4e-4644631433be X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: Wdm3ipF9UQYiAUcT8NP95BLEbuqn3oCohEAWgiu/72tNhZvgXuhl42Zdo8yJRj+KrtaJRV2G9vSnKzmlFJeLKg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH0PR11MB5524 X-Proofpoint-ORIG-GUID: N_lKnZNriQuKQqYgr51TnousTxi6xP7S X-Proofpoint-GUID: N_lKnZNriQuKQqYgr51TnousTxi6xP7S X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.205,Aquarius:18.0.816,Hydra:6.0.425,FMLib:17.11.64.514 definitions=2022-03-09_03,2022-03-04_01,2022-02-23_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 spamscore=0 adultscore=0 impostorscore=0 mlxlogscore=999 malwarescore=0 suspectscore=0 phishscore=0 mlxscore=0 priorityscore=1501 lowpriorityscore=0 clxscore=1011 bulkscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2202240000 definitions=main-2203090044 X-MIME-Autoconverted: from 8bit to quoted-printable by mx0a-0064b401.pphosted.com id 2297EG6k031808 List-Id: X-Webhook-Received: from li982-79.members.linode.com [45.33.32.79] by aws-us-west-2-korg-lkml-1.web.codeaurora.org with HTTPS for ; Wed, 09 Mar 2022 08:26:26 -0000 X-Groupsio-URL: https://lists.openembedded.org/g/openembedded-core/message/162952 Backport a patch from upstream to handle negative time and offset from Epoch Signed-off-by: He Zhe --- ...-negative-time-and-offset-from-Epoch.patch | 853 ++++++++++++++++++ meta/recipes-kernel/lttng/babeltrace_1.5.8.bb | 1 + 2 files changed, 854 insertions(+) create mode 100644 meta/recipes-kernel/lttng/babeltrace/0001-Handle-negative-time-and-offset-from-Epoch.patch diff --git a/meta/recipes-kernel/lttng/babeltrace/0001-Handle-negative-time-and-offset-from-Epoch.patch b/meta/recipes-kernel/lttng/babeltrace/0001-Handle-negative-time-and-offset-from-Epoch.patch new file mode 100644 index 0000000000..37ca698bd8 --- /dev/null +++ b/meta/recipes-kernel/lttng/babeltrace/0001-Handle-negative-time-and-offset-from-Epoch.patch @@ -0,0 +1,853 @@ +From 78f62745313a51ebdaa64d4c59131ef87ef00917 Mon Sep 17 00:00:00 2001 +From: Mathieu Desnoyers +Date: Wed, 22 Apr 2020 03:00:02 +0000 +Subject: [PATCH] Handle negative time and offset from Epoch + +Handle cases where a trace have a negative offset from Epoch. +If Epoch is arbitrary (e.g. embedded system starting at 0, without any +network access), the "0" can be used as correlation point between +various components, and some components could start before the +correlation point. Therefore, especially in traces where the time is +meant to be shown in nanoseconds or cycles from the correlation point, +it also makes sense to have a negative time value. + +It introduces API-breaking changes in the C and Python APIs, since we +need to be able to return negative time values, which were previously +used as errors (-1ULL). + +The --offset and --offset-ns command line parameters can now take +negative offset (seconds and nanoseconds) values too. + +The [sec.ns] format is used as fallback so we don't attempt to pass +a negative time value to POSIX time-formatting APIs. + +This also fixes an inaccurate return value in an error path of +bt_ctf_event_populate_event_header(). + +Signed-off-by: Mathieu Desnoyers +Signed-off-by: Jérémie Galarneau + +Upstream-Status: Backport [https://git.efficios.com/?p=babeltrace.git;a=commit;h=61cf588beae752e5ddfc60b6b5310f769ac9e852] + +Signed-off-by: He Zhe +--- + converter/babeltrace.c | 4 +- + formats/ctf-text/ctf-text.c | 2 +- + formats/ctf/ctf.c | 115 +++++++++++++----- + formats/ctf/events-private.h | 10 +- + formats/ctf/events.c | 11 +- + .../metadata/ctf-visitor-generate-io-struct.c | 1 - + include/babeltrace/babeltrace-internal.h | 8 +- + include/babeltrace/clock-internal.h | 2 +- + include/babeltrace/ctf-ir/clock-internal.h | 2 +- + include/babeltrace/ctf/events.h | 8 +- + include/babeltrace/ctf/types.h | 6 +- + include/babeltrace/format.h | 10 +- + include/babeltrace/trace-handle-internal.h | 8 +- + include/babeltrace/trace-handle.h | 25 ++-- + lib/context.c | 44 +++++-- + lib/trace-handle.c | 39 +++--- + tests/lib/test_ctf_writer.c | 6 +- + tests/lib/test_seek.c | 28 ++--- + 18 files changed, 209 insertions(+), 120 deletions(-) + +diff --git a/converter/babeltrace.c b/converter/babeltrace.c +index ef783ed4..2ac16179 100644 +--- a/converter/babeltrace.c ++++ b/converter/babeltrace.c +@@ -375,7 +375,7 @@ static int parse_options(int argc, char **argv) + goto end; + } + errno = 0; +- opt_clock_offset = strtoull(str, &endptr, 0); ++ opt_clock_offset = strtoll(str, &endptr, 0); + if (*endptr != '\0' || str == endptr || errno != 0) { + fprintf(stderr, "[error] Incorrect --clock-offset argument: %s\n", str); + ret = -EINVAL; +@@ -400,7 +400,7 @@ static int parse_options(int argc, char **argv) + goto end; + } + errno = 0; +- opt_clock_offset_ns = strtoull(str, &endptr, 0); ++ opt_clock_offset_ns = strtoll(str, &endptr, 0); + if (*endptr != '\0' || str == endptr || errno != 0) { + fprintf(stderr, "[error] Incorrect --clock-offset-ns argument: %s\n", str); + ret = -EINVAL; +diff --git a/formats/ctf-text/ctf-text.c b/formats/ctf-text/ctf-text.c +index 16f156de..e001a548 100644 +--- a/formats/ctf-text/ctf-text.c ++++ b/formats/ctf-text/ctf-text.c +@@ -43,7 +43,7 @@ + #include + #include + +-#define NSEC_PER_SEC 1000000000ULL ++#define NSEC_PER_SEC 1000000000LL + + int opt_all_field_names, + opt_scope_field_names, +diff --git a/formats/ctf/ctf.c b/formats/ctf/ctf.c +index 1ba9017f..3c774188 100644 +--- a/formats/ctf/ctf.c ++++ b/formats/ctf/ctf.c +@@ -70,7 +70,7 @@ + */ + #define WRITE_PACKET_LEN (getpagesize() * 8 * CHAR_BIT) + +-#define NSEC_PER_SEC 1000000000ULL ++#define NSEC_PER_SEC 1000000000LL + + #define INDEX_PATH "./index/%s.idx" + +@@ -79,8 +79,8 @@ int opt_clock_cycles, + opt_clock_date, + opt_clock_gmt; + +-uint64_t opt_clock_offset; +-uint64_t opt_clock_offset_ns; ++int64_t opt_clock_offset; ++int64_t opt_clock_offset_ns; + + extern int yydebug; + char *opt_debug_info_dir; +@@ -114,11 +114,13 @@ void ctf_set_handle(struct bt_trace_descriptor *descriptor, + static + int ctf_close_trace(struct bt_trace_descriptor *descriptor); + static +-uint64_t ctf_timestamp_begin(struct bt_trace_descriptor *descriptor, +- struct bt_trace_handle *handle, enum bt_clock_type type); ++int ctf_timestamp_begin(struct bt_trace_descriptor *descriptor, ++ struct bt_trace_handle *handle, enum bt_clock_type type, ++ int64_t *timestamp); + static +-uint64_t ctf_timestamp_end(struct bt_trace_descriptor *descriptor, +- struct bt_trace_handle *handle, enum bt_clock_type type); ++int ctf_timestamp_end(struct bt_trace_descriptor *descriptor, ++ struct bt_trace_handle *handle, enum bt_clock_type type, ++ int64_t *timestamp); + static + int ctf_convert_index_timestamp(struct bt_trace_descriptor *tdp); + +@@ -167,17 +169,20 @@ void bt_ctf_hook(void) + } + + static +-uint64_t ctf_timestamp_begin(struct bt_trace_descriptor *descriptor, +- struct bt_trace_handle *handle, enum bt_clock_type type) ++int ctf_timestamp_begin(struct bt_trace_descriptor *descriptor, ++ struct bt_trace_handle *handle, enum bt_clock_type type, ++ int64_t *timestamp) + { + struct ctf_trace *tin; +- uint64_t begin = ULLONG_MAX; +- int i, j; ++ int64_t begin = LLONG_MAX; ++ int i, j, ret; + + tin = container_of(descriptor, struct ctf_trace, parent); + +- if (!tin) ++ if (!tin || !timestamp) { ++ ret = -EINVAL; + goto error; ++ } + + /* for each stream_class */ + for (i = 0; i < tin->streams->len; i++) { +@@ -198,8 +203,10 @@ uint64_t ctf_timestamp_begin(struct bt_trace_descriptor *descriptor, + parent); + stream_pos = &cfs->pos; + +- if (!stream_pos->packet_index) ++ if (!stream_pos->packet_index) { ++ ret = -EINVAL; + goto error; ++ } + + if (stream_pos->packet_index->len <= 0) + continue; +@@ -214,29 +221,38 @@ uint64_t ctf_timestamp_begin(struct bt_trace_descriptor *descriptor, + if (index->ts_cycles.timestamp_begin < begin) + begin = index->ts_cycles.timestamp_begin; + } else { ++ ret = -EINVAL; + goto error; + } + } + } + +- return begin; ++ if (begin == LLONG_MAX) { ++ ret = -ENOENT; ++ goto error; ++ } ++ *timestamp = begin; ++ return 0; + + error: +- return -1ULL; ++ return ret; + } + + static +-uint64_t ctf_timestamp_end(struct bt_trace_descriptor *descriptor, +- struct bt_trace_handle *handle, enum bt_clock_type type) ++int ctf_timestamp_end(struct bt_trace_descriptor *descriptor, ++ struct bt_trace_handle *handle, enum bt_clock_type type, ++ int64_t *timestamp) + { + struct ctf_trace *tin; +- uint64_t end = 0; +- int i, j; ++ int64_t end = LLONG_MIN; ++ int i, j, ret; + + tin = container_of(descriptor, struct ctf_trace, parent); + +- if (!tin) ++ if (!tin || !timestamp) { ++ ret = -EINVAL; + goto error; ++ } + + /* for each stream_class */ + for (i = 0; i < tin->streams->len; i++) { +@@ -257,8 +273,10 @@ uint64_t ctf_timestamp_end(struct bt_trace_descriptor *descriptor, + parent); + stream_pos = &cfs->pos; + +- if (!stream_pos->packet_index) ++ if (!stream_pos->packet_index) { ++ ret = -EINVAL; + goto error; ++ } + + if (stream_pos->packet_index->len <= 0) + continue; +@@ -273,15 +291,21 @@ uint64_t ctf_timestamp_end(struct bt_trace_descriptor *descriptor, + if (index->ts_cycles.timestamp_end > end) + end = index->ts_cycles.timestamp_end; + } else { ++ ret = -EINVAL; + goto error; + } + } + } + +- return end; ++ if (end == LLONG_MIN) { ++ ret = -ENOENT; ++ goto error; ++ } ++ *timestamp = end; ++ return 0; + + error: +- return -1ULL; ++ return ret; + } + + /* +@@ -328,7 +352,9 @@ void ctf_print_timestamp_real(FILE *fp, + struct ctf_stream_definition *stream, + uint64_t timestamp) + { +- uint64_t ts_sec = 0, ts_nsec; ++ int64_t ts_sec = 0, ts_nsec; ++ uint64_t ts_sec_abs, ts_nsec_abs; ++ bool is_negative; + + ts_nsec = timestamp; + +@@ -341,9 +367,40 @@ void ctf_print_timestamp_real(FILE *fp, + ts_sec += ts_nsec / NSEC_PER_SEC; + ts_nsec = ts_nsec % NSEC_PER_SEC; + ++ if (ts_sec >= 0 && ts_nsec >= 0) { ++ is_negative = false; ++ ts_sec_abs = ts_sec; ++ ts_nsec_abs = ts_nsec; ++ } else if (ts_sec > 0 && ts_nsec < 0) { ++ is_negative = false; ++ ts_sec_abs = ts_sec - 1; ++ ts_nsec_abs = NSEC_PER_SEC + ts_nsec; ++ } else if (ts_sec == 0 && ts_nsec < 0) { ++ is_negative = true; ++ ts_sec_abs = ts_sec; ++ ts_nsec_abs = -ts_nsec; ++ } else if (ts_sec < 0 && ts_nsec > 0) { ++ is_negative = true; ++ ts_sec_abs = -(ts_sec + 1); ++ ts_nsec_abs = NSEC_PER_SEC - ts_nsec; ++ } else if (ts_sec < 0 && ts_nsec == 0) { ++ is_negative = true; ++ ts_sec_abs = -ts_sec; ++ ts_nsec_abs = ts_nsec; ++ } else { /* (ts_sec < 0 && ts_nsec < 0) */ ++ is_negative = true; ++ ts_sec_abs = -ts_sec; ++ ts_nsec_abs = -ts_nsec; ++ } ++ + if (!opt_clock_seconds) { + struct tm tm; +- time_t time_s = (time_t) ts_sec; ++ time_t time_s = (time_t) ts_sec_abs; ++ ++ if (is_negative) { ++ fprintf(stderr, "[warning] Fallback to [sec.ns] for printing negative time value. Use --clock-seconds.\n"); ++ goto seconds; ++ } + + if (!opt_clock_gmt) { + struct tm *res; +@@ -377,12 +434,12 @@ void ctf_print_timestamp_real(FILE *fp, + } + /* Print time in HH:MM:SS.ns */ + fprintf(fp, "%02d:%02d:%02d.%09" PRIu64, +- tm.tm_hour, tm.tm_min, tm.tm_sec, ts_nsec); ++ tm.tm_hour, tm.tm_min, tm.tm_sec, ts_nsec_abs); + goto end; + } + seconds: +- fprintf(fp, "%3" PRIu64 ".%09" PRIu64, +- ts_sec, ts_nsec); ++ fprintf(fp, "%s%" PRId64 ".%09" PRIu64, ++ is_negative ? "-" : "", ts_sec_abs, ts_nsec_abs); + + end: + return; +@@ -401,7 +458,7 @@ void ctf_print_timestamp_cycles(FILE *fp, + + void ctf_print_timestamp(FILE *fp, + struct ctf_stream_definition *stream, +- uint64_t timestamp) ++ int64_t timestamp) + { + if (opt_clock_cycles) { + ctf_print_timestamp_cycles(fp, stream, timestamp); +diff --git a/formats/ctf/events-private.h b/formats/ctf/events-private.h +index 9bea75d4..c47fd7d8 100644 +--- a/formats/ctf/events-private.h ++++ b/formats/ctf/events-private.h +@@ -35,20 +35,20 @@ + #include + + static inline +-uint64_t ctf_get_real_timestamp(struct ctf_stream_definition *stream, +- uint64_t timestamp) ++int64_t ctf_get_real_timestamp(struct ctf_stream_definition *stream, ++ uint64_t ts_cycles) + { +- uint64_t ts_nsec; ++ int64_t ts_nsec; + struct ctf_trace *trace = stream->stream_class->trace; + struct trace_collection *tc = trace->parent.collection; +- uint64_t tc_offset; ++ int64_t tc_offset; + + if (tc->clock_use_offset_avg) + tc_offset = tc->single_clock_offset_avg; + else + tc_offset = clock_offset_ns(trace->parent.single_clock); + +- ts_nsec = clock_cycles_to_ns(stream->current_clock, timestamp); ++ ts_nsec = clock_cycles_to_ns(stream->current_clock, ts_cycles); + ts_nsec += tc_offset; /* Add offset */ + return ts_nsec; + } +diff --git a/formats/ctf/events.c b/formats/ctf/events.c +index bd195b93..c5aefd82 100644 +--- a/formats/ctf/events.c ++++ b/formats/ctf/events.c +@@ -315,18 +315,19 @@ int bt_ctf_event_get_handle_id(const struct bt_ctf_event *ctf_event) + return ret; + } + +-uint64_t bt_ctf_get_timestamp(const struct bt_ctf_event *ctf_event) ++int bt_ctf_get_timestamp(const struct bt_ctf_event *ctf_event, int64_t *timestamp) + { + const struct ctf_event_definition *event; + +- if (!ctf_event) +- return -1ULL; ++ if (!ctf_event || !timestamp) ++ return -1; + + event = ctf_event->parent; + if (event && event->stream->has_timestamp) +- return event->stream->real_timestamp; ++ *timestamp = event->stream->real_timestamp; + else +- return -1ULL; ++ return -1; ++ return 0; + } + + uint64_t bt_ctf_get_cycles(const struct bt_ctf_event *ctf_event) +diff --git a/formats/ctf/metadata/ctf-visitor-generate-io-struct.c b/formats/ctf/metadata/ctf-visitor-generate-io-struct.c +index 8d348d66..dd8374fc 100644 +--- a/formats/ctf/metadata/ctf-visitor-generate-io-struct.c ++++ b/formats/ctf/metadata/ctf-visitor-generate-io-struct.c +@@ -219,7 +219,6 @@ int get_unary_signed(struct bt_list_head *head, int64_t *value) + + bt_list_for_each_entry(node, head, siblings) { + if (node->type != NODE_UNARY_EXPRESSION +- || node->u.unary_expression.type != UNARY_UNSIGNED_CONSTANT + || (node->u.unary_expression.type != UNARY_UNSIGNED_CONSTANT && node->u.unary_expression.type != UNARY_SIGNED_CONSTANT) + || node->u.unary_expression.link != UNARY_LINK_UNKNOWN + || i != 0) +diff --git a/include/babeltrace/babeltrace-internal.h b/include/babeltrace/babeltrace-internal.h +index 6f8e2006..3e137e4c 100644 +--- a/include/babeltrace/babeltrace-internal.h ++++ b/include/babeltrace/babeltrace-internal.h +@@ -178,8 +178,8 @@ struct trace_collection { + GPtrArray *array; /* struct bt_trace_descriptor */ + GHashTable *clocks; /* struct ctf_clock */ + +- uint64_t single_clock_offset_avg; +- uint64_t offset_first; ++ int64_t single_clock_offset_avg; ++ int64_t offset_first; + int64_t delta_offset_first_sum; + int offset_nr; + int clock_use_offset_avg; +@@ -208,8 +208,8 @@ extern int opt_all_field_names, + opt_clock_force_correlate, + opt_debug_info_full_path; + +-extern uint64_t opt_clock_offset; +-extern uint64_t opt_clock_offset_ns; ++extern int64_t opt_clock_offset; ++extern int64_t opt_clock_offset_ns; + extern int babeltrace_ctf_console_output; + extern char *opt_debug_info_dir; + extern char *opt_debug_info_target_prefix; +diff --git a/include/babeltrace/clock-internal.h b/include/babeltrace/clock-internal.h +index cd6bdbae..86954b78 100644 +--- a/include/babeltrace/clock-internal.h ++++ b/include/babeltrace/clock-internal.h +@@ -49,7 +49,7 @@ uint64_t clock_cycles_to_ns(struct ctf_clock *clock, uint64_t cycles) + * mantissa. + */ + static inline +-uint64_t clock_offset_ns(struct ctf_clock *clock) ++int64_t clock_offset_ns(struct ctf_clock *clock) + { + return clock->offset_s * 1000000000ULL + + clock_cycles_to_ns(clock, clock->offset); +diff --git a/include/babeltrace/ctf-ir/clock-internal.h b/include/babeltrace/ctf-ir/clock-internal.h +index 75677707..f1e896b6 100644 +--- a/include/babeltrace/ctf-ir/clock-internal.h ++++ b/include/babeltrace/ctf-ir/clock-internal.h +@@ -42,7 +42,7 @@ struct bt_ctf_clock { + uint64_t precision; + int64_t offset_s; /* Offset in seconds */ + int64_t offset; /* Offset in ticks */ +- uint64_t value; /* Current clock value */ ++ int64_t value; /* Current clock value */ + uuid_t uuid; + int uuid_set; + int absolute; +diff --git a/include/babeltrace/ctf/events.h b/include/babeltrace/ctf/events.h +index c81d8852..15830a30 100644 +--- a/include/babeltrace/ctf/events.h ++++ b/include/babeltrace/ctf/events.h +@@ -109,10 +109,12 @@ const char *bt_ctf_event_name(const struct bt_ctf_event *event); + uint64_t bt_ctf_get_cycles(const struct bt_ctf_event *event); + + /* +- * bt_ctf_get_timestamp: returns the timestamp of the event offsetted +- * with the system clock source (in ns) or -1ULL on error ++ * bt_ctf_get_timestamp: get the timestamp of the event offsetted ++ * with the system clock source (in ns) in *timestamp. ++ * ++ * Return 0 on success, or -1ULL on error. + */ +-uint64_t bt_ctf_get_timestamp(const struct bt_ctf_event *event); ++int bt_ctf_get_timestamp(const struct bt_ctf_event *event, int64_t *timestamp); + + /* + * bt_ctf_get_field_list: obtain the list of fields for compound type +diff --git a/include/babeltrace/ctf/types.h b/include/babeltrace/ctf/types.h +index 0bc003c8..4b626b39 100644 +--- a/include/babeltrace/ctf/types.h ++++ b/include/babeltrace/ctf/types.h +@@ -46,8 +46,8 @@ + struct bt_stream_callbacks; + + struct packet_index_time { +- uint64_t timestamp_begin; +- uint64_t timestamp_end; ++ int64_t timestamp_begin; ++ int64_t timestamp_end; + }; + + struct packet_index { +@@ -253,7 +253,7 @@ int ctf_pos_get_event(struct ctf_stream_pos *pos) + } + + void ctf_print_timestamp(FILE *fp, struct ctf_stream_definition *stream, +- uint64_t timestamp); ++ int64_t timestamp); + int ctf_append_trace_metadata(struct bt_trace_descriptor *tdp, + FILE *metadata_fp); + void ctf_print_discarded_lost(FILE *fp, struct ctf_stream_definition *stream); +diff --git a/include/babeltrace/format.h b/include/babeltrace/format.h +index dea8e0e5..bf33a239 100644 +--- a/include/babeltrace/format.h ++++ b/include/babeltrace/format.h +@@ -73,10 +73,12 @@ struct bt_format { + struct bt_context *ctx); + void (*set_handle)(struct bt_trace_descriptor *descriptor, + struct bt_trace_handle *handle); +- uint64_t (*timestamp_begin)(struct bt_trace_descriptor *descriptor, +- struct bt_trace_handle *handle, enum bt_clock_type type); +- uint64_t (*timestamp_end)(struct bt_trace_descriptor *descriptor, +- struct bt_trace_handle *handle, enum bt_clock_type type); ++ int (*timestamp_begin)(struct bt_trace_descriptor *descriptor, ++ struct bt_trace_handle *handle, enum bt_clock_type type, ++ int64_t *timestamp); ++ int (*timestamp_end)(struct bt_trace_descriptor *descriptor, ++ struct bt_trace_handle *handle, enum bt_clock_type type, ++ int64_t *timestamp); + int (*convert_index_timestamp)(struct bt_trace_descriptor *descriptor); + }; + +diff --git a/include/babeltrace/trace-handle-internal.h b/include/babeltrace/trace-handle-internal.h +index 5e9c1c6a..924c730c 100644 +--- a/include/babeltrace/trace-handle-internal.h ++++ b/include/babeltrace/trace-handle-internal.h +@@ -46,10 +46,10 @@ struct bt_trace_handle { + struct bt_trace_descriptor *td; + struct bt_format *format; + char path[PATH_MAX]; +- uint64_t real_timestamp_begin; +- uint64_t real_timestamp_end; +- uint64_t cycles_timestamp_begin; +- uint64_t cycles_timestamp_end; ++ int64_t real_timestamp_begin; ++ int64_t real_timestamp_end; ++ int64_t cycles_timestamp_begin; ++ int64_t cycles_timestamp_end; + }; + + /* +diff --git a/include/babeltrace/trace-handle.h b/include/babeltrace/trace-handle.h +index 96e4a81b..55c850f8 100644 +--- a/include/babeltrace/trace-handle.h ++++ b/include/babeltrace/trace-handle.h +@@ -53,20 +53,25 @@ struct bt_ctf_event; + const char *bt_trace_handle_get_path(struct bt_context *ctx, int handle_id); + + /* +- * bt_trace_handle_get_timestamp_begin : returns the creation time (in +- * nanoseconds or cycles depending on type) of the buffers of a trace +- * or -1ULL on error. ++ * bt_trace_handle_get_timestamp_begin : get the creation time (in ++ * nanoseconds or cycles depending on type) of the buffers of a trace. ++ * ++ * Returns 0 on success, -1 on error. + */ +-uint64_t bt_trace_handle_get_timestamp_begin(struct bt_context *ctx, +- int handle_id, enum bt_clock_type type); ++int bt_trace_handle_get_timestamp_begin(struct bt_context *ctx, ++ int handle_id, enum bt_clock_type type, ++ int64_t *timestamp); + + /* +- * bt_trace_handle_get_timestamp_end : returns the destruction timestamp +- * (in nanoseconds or cycles depending on type) of the buffers of a trace +- * or -1ULL on error. ++ * bt_trace_handle_get_timestamp_end : get the destruction time ++ * (in nanoseconds or cycles depending on type) of the buffers of a ++ * trace. ++ * ++ * Returns 0 on success, -1 on error. + */ +-uint64_t bt_trace_handle_get_timestamp_end(struct bt_context *ctx, +- int handle_id, enum bt_clock_type type); ++int bt_trace_handle_get_timestamp_end(struct bt_context *ctx, ++ int handle_id, enum bt_clock_type type, ++ int64_t *timestamp); + + /* + * bt_ctf_event_get_handle_id : get the handle id associated with an event +diff --git a/lib/context.c b/lib/context.c +index 87901b3c..34a1bc82 100644 +--- a/lib/context.c ++++ b/lib/context.c +@@ -139,18 +139,38 @@ int bt_context_add_trace(struct bt_context *ctx, const char *path, + goto error_collection_del; + } + +- if (fmt->timestamp_begin) +- handle->real_timestamp_begin = fmt->timestamp_begin(td, +- handle, BT_CLOCK_REAL); +- if (fmt->timestamp_end) +- handle->real_timestamp_end = fmt->timestamp_end(td, handle, +- BT_CLOCK_REAL); +- if (fmt->timestamp_begin) +- handle->cycles_timestamp_begin = fmt->timestamp_begin(td, +- handle, BT_CLOCK_CYCLES); +- if (fmt->timestamp_end) +- handle->cycles_timestamp_end = fmt->timestamp_end(td, handle, +- BT_CLOCK_CYCLES); ++ if (fmt->timestamp_begin) { ++ ret = fmt->timestamp_begin(td, handle, BT_CLOCK_REAL, ++ &handle->real_timestamp_begin); ++ if (ret < 0 && ret != -ENOENT) { ++ ret = -1; ++ goto error_collection_del; ++ } ++ } ++ if (fmt->timestamp_end) { ++ ret = fmt->timestamp_end(td, handle, BT_CLOCK_REAL, ++ &handle->real_timestamp_end); ++ if (ret < 0 && ret != -ENOENT) { ++ ret = -1; ++ goto error_collection_del; ++ } ++ } ++ if (fmt->timestamp_begin) { ++ ret = fmt->timestamp_begin(td, handle, BT_CLOCK_CYCLES, ++ &handle->cycles_timestamp_begin); ++ if (ret < 0 && ret != -ENOENT) { ++ ret = -1; ++ goto error_collection_del; ++ } ++ } ++ if (fmt->timestamp_end) { ++ ret = fmt->timestamp_end(td, handle, BT_CLOCK_CYCLES, ++ &handle->cycles_timestamp_end); ++ if (ret < 0 && ret != -ENOENT) { ++ ret = -1; ++ goto error_collection_del; ++ } ++ } + + /* Add new handle to container */ + g_hash_table_insert(ctx->trace_handles, +diff --git a/lib/trace-handle.c b/lib/trace-handle.c +index d5b906aa..8f11c7cc 100644 +--- a/lib/trace-handle.c ++++ b/lib/trace-handle.c +@@ -66,54 +66,57 @@ const char *bt_trace_handle_get_path(struct bt_context *ctx, int handle_id) + return handle->path; + } + +-uint64_t bt_trace_handle_get_timestamp_begin(struct bt_context *ctx, +- int handle_id, enum bt_clock_type type) ++int bt_trace_handle_get_timestamp_begin(struct bt_context *ctx, ++ int handle_id, enum bt_clock_type type, ++ int64_t *timestamp) + { + struct bt_trace_handle *handle; +- uint64_t ret; + +- if (!ctx) +- return -1ULL; ++ int ret = 0; ++ ++ if (!ctx || !timestamp) ++ return -1; + + handle = g_hash_table_lookup(ctx->trace_handles, + (gpointer) (unsigned long) handle_id); + if (!handle) { +- ret = -1ULL; ++ ret = -1; + goto end; + } + if (type == BT_CLOCK_REAL) { +- ret = handle->real_timestamp_begin; ++ *timestamp = handle->real_timestamp_begin; + } else if (type == BT_CLOCK_CYCLES) { +- ret = handle->cycles_timestamp_begin; ++ *timestamp = handle->cycles_timestamp_begin; + } else { +- ret = -1ULL; ++ ret = -1; + } + + end: + return ret; + } + +-uint64_t bt_trace_handle_get_timestamp_end(struct bt_context *ctx, +- int handle_id, enum bt_clock_type type) ++int bt_trace_handle_get_timestamp_end(struct bt_context *ctx, ++ int handle_id, enum bt_clock_type type, ++ int64_t *timestamp) + { + struct bt_trace_handle *handle; +- uint64_t ret; ++ int ret = 0; + +- if (!ctx) +- return -1ULL; ++ if (!ctx || !timestamp) ++ return -1; + + handle = g_hash_table_lookup(ctx->trace_handles, + (gpointer) (unsigned long) handle_id); + if (!handle) { +- ret = -1ULL; ++ ret = -1; + goto end; + } + if (type == BT_CLOCK_REAL) { +- ret = handle->real_timestamp_end; ++ *timestamp = handle->real_timestamp_end; + } else if (type == BT_CLOCK_CYCLES) { +- ret = handle->cycles_timestamp_end; ++ *timestamp = handle->cycles_timestamp_end; + } else { +- ret = -1ULL; ++ ret = -1; + } + + end: +diff --git a/tests/lib/test_ctf_writer.c b/tests/lib/test_ctf_writer.c +index 53613c91..29d76df8 100644 +--- a/tests/lib/test_ctf_writer.c ++++ b/tests/lib/test_ctf_writer.c +@@ -45,7 +45,7 @@ + #define SEQUENCE_TEST_LENGTH 10 + #define PACKET_RESIZE_TEST_LENGTH 100000 + +-static uint64_t current_time; ++static int64_t current_time; + + void validate_metadata(char *parser_path, char *metadata_path) + { +@@ -695,8 +695,8 @@ int main(int argc, char **argv) + const char *clock_name = "test_clock"; + const char *clock_description = "This is a test clock"; + const uint64_t frequency = 1000000000; +- const uint64_t offset_s = 1351530929945824323; +- const uint64_t offset = 1234567; ++ const int64_t offset_s = 1351530929945824323; ++ const int64_t offset = 1234567; + const uint64_t precision = 10; + char *metadata_string; + struct bt_ctf_writer *writer; +diff --git a/tests/lib/test_seek.c b/tests/lib/test_seek.c +index 3c78e8ad..04beaf97 100644 +--- a/tests/lib/test_seek.c ++++ b/tests/lib/test_seek.c +@@ -33,7 +33,7 @@ + #include + #include "common.h" + +-#define NR_TESTS 29 ++#define NR_TESTS 36 + + void run_seek_begin(char *path, uint64_t expected_begin) + { +@@ -42,8 +42,8 @@ void run_seek_begin(char *path, uint64_t expected_begin) + struct bt_ctf_event *event; + struct bt_iter_pos newpos; + int ret; +- uint64_t timestamp_begin; +- uint64_t timestamp_seek_begin; ++ int64_t timestamp_begin; ++ int64_t timestamp_seek_begin; + unsigned int nr_seek_begin_test; + + nr_seek_begin_test = 5; +@@ -67,7 +67,7 @@ void run_seek_begin(char *path, uint64_t expected_begin) + ok(event, "Event valid"); + + /* Validate that the first timestamp is right */ +- timestamp_begin = bt_ctf_get_timestamp(event); ++ ok1(bt_ctf_get_timestamp(event, ×tamp_begin) == 0); + + ok1(timestamp_begin == expected_begin); + +@@ -81,8 +81,8 @@ void run_seek_begin(char *path, uint64_t expected_begin) + + ok(event, "Event valid"); + +- timestamp_seek_begin = bt_ctf_get_timestamp(event); +- ++ ok1(bt_ctf_get_timestamp(event, ×tamp_seek_begin) == 0); ++ + ok1(timestamp_begin == timestamp_seek_begin); + + bt_context_put(ctx); +@@ -96,7 +96,7 @@ void run_seek_last(char *path, uint64_t expected_last) + struct bt_ctf_event *event; + struct bt_iter_pos newpos; + int ret; +- uint64_t timestamp_last; ++ int64_t timestamp_last; + unsigned int nr_seek_last_tests; + + nr_seek_last_tests = 6; +@@ -129,7 +129,7 @@ void run_seek_last(char *path, uint64_t expected_last) + + ok(event, "Event valid at last position"); + +- timestamp_last = bt_ctf_get_timestamp(event); ++ ok1(bt_ctf_get_timestamp(event, ×tamp_last) == 0); + + ok1(timestamp_last == expected_last); + +@@ -152,7 +152,7 @@ void run_seek_time_at_last(char *path, uint64_t expected_last) + struct bt_ctf_event *event; + struct bt_iter_pos newpos; + int ret; +- uint64_t timestamp_last; ++ int64_t timestamp_last; + unsigned int nr_seek_time_at_last_tests; + + nr_seek_time_at_last_tests = 6; +@@ -188,7 +188,7 @@ void run_seek_time_at_last(char *path, uint64_t expected_last) + + ok(event, "Event valid at last position"); + +- timestamp_last = bt_ctf_get_timestamp(event); ++ ok1(bt_ctf_get_timestamp(event, ×tamp_last) == 0); + + ok1(timestamp_last == expected_last); + +@@ -213,7 +213,7 @@ void run_seek_cycles(char *path, + struct bt_ctf_event *event; + struct bt_iter_pos newpos; + int ret; +- uint64_t timestamp; ++ int64_t timestamp; + + unsigned int nr_seek_cycles_tests; + +@@ -247,7 +247,7 @@ void run_seek_cycles(char *path, + + ok(event, "Event valid at last position"); + +- timestamp = bt_ctf_get_timestamp(event); ++ ok1(bt_ctf_get_timestamp(event, ×tamp) == 0); + + ok1(timestamp == expected_last); + +@@ -270,7 +270,7 @@ void run_seek_cycles(char *path, + + ok(event, "Event valid at first position"); + +- timestamp = bt_ctf_get_timestamp(event); ++ ok1(bt_ctf_get_timestamp(event, ×tamp) == 0); + + ok1(timestamp == expected_begin); + +@@ -284,7 +284,7 @@ void run_seek_cycles(char *path, + + ok(event, "Event valid at last position"); + +- timestamp = bt_ctf_get_timestamp(event); ++ ok1(bt_ctf_get_timestamp(event, ×tamp) == 0); + + ok1(timestamp == expected_last); + +-- +2.25.1 + diff --git a/meta/recipes-kernel/lttng/babeltrace_1.5.8.bb b/meta/recipes-kernel/lttng/babeltrace_1.5.8.bb index 9e5d3a7a97..86664b14ba 100644 --- a/meta/recipes-kernel/lttng/babeltrace_1.5.8.bb +++ b/meta/recipes-kernel/lttng/babeltrace_1.5.8.bb @@ -9,6 +9,7 @@ DEPENDS = "glib-2.0 util-linux popt bison-native flex-native" SRC_URI = "git://git.efficios.com/babeltrace.git;branch=stable-1.5 \ file://run-ptest \ + file://0001-Handle-negative-time-and-offset-from-Epoch.patch \ " SRCREV = "054a54ae10b01a271afc4f19496c041b10fb414c" UPSTREAM_CHECK_GITTAGREGEX = "v(?P1(\.\d+)+)$"