From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dedi548.your-server.de (dedi548.your-server.de [85.10.215.148]) by sourceware.org (Postfix) with ESMTPS id 824433835829; Wed, 22 Jun 2022 06:22:17 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 824433835829 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=embedded-brains.de Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=embedded-brains.de Received: from sslproxy02.your-server.de ([78.47.166.47]) by dedi548.your-server.de with esmtpsa (TLS1.3) tls TLS_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1o3tkk-000Pis-QZ; Wed, 22 Jun 2022 08:22:15 +0200 Received: from [82.100.198.138] (helo=mail.embedded-brains.de) by sslproxy02.your-server.de with esmtpsa (TLSv1.3:TLS_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1o3tkl-000TOU-08; Wed, 22 Jun 2022 08:22:15 +0200 Received: from localhost (localhost [127.0.0.1]) by mail.embedded-brains.de (Postfix) with ESMTP id AAC3B480107; Wed, 22 Jun 2022 08:22:14 +0200 (CEST) Received: from mail.embedded-brains.de ([127.0.0.1]) by localhost (zimbra.eb.localhost [127.0.0.1]) (amavisd-new, port 10032) with ESMTP id dPAJ3QFqXVLM; Wed, 22 Jun 2022 08:22:14 +0200 (CEST) Received: from localhost (localhost [127.0.0.1]) by mail.embedded-brains.de (Postfix) with ESMTP id 5D0DD480140; Wed, 22 Jun 2022 08:22:14 +0200 (CEST) X-Virus-Scanned: amavisd-new at zimbra.eb.localhost Received: from mail.embedded-brains.de ([127.0.0.1]) by localhost (zimbra.eb.localhost [127.0.0.1]) (amavisd-new, port 10026) with ESMTP id aN5KM6fk4sN9; Wed, 22 Jun 2022 08:22:14 +0200 (CEST) Received: from [10.10.171.14] (unknown [10.10.171.14]) by mail.embedded-brains.de (Postfix) with ESMTPSA id 00162480107; Wed, 22 Jun 2022 08:22:13 +0200 (CEST) Message-ID: <9feeef3a-105b-ba95-3327-8a15320ec215@embedded-brains.de> Date: Wed, 22 Jun 2022 08:22:12 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.10.0 Subject: Re: [PATCH] libstdc++: 60241.cc: tolerate slightly shorter aggregate sleep Content-Language: en-US To: Alexandre Oliva , gcc-patches@gcc.gnu.org, libstdc++@gcc.gnu.org, RTEMS References: From: Sebastian Huber In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: quoted-printable X-Authenticated-Sender: smtp-embedded@poldinet.de X-Virus-Scanned: Clear (ClamAV 0.103.6/26579/Tue Jun 21 10:15:30 2022) X-Spam-Status: No, score=-11.5 required=5.0 tests=BAYES_00, GIT_PATCH_0, KAM_DMARC_STATUS, NICE_REPLY_A, SPF_HELO_NONE, SPF_PASS, 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 X-BeenThere: libstdc++@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Libstdc++ mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 22 Jun 2022 06:22:19 -0000 On 22/06/2022 08:01, Alexandre Oliva via Gcc-patches wrote: >=20 > On rtems under qemu, the frequently-interrupted nanosleep ends up > sleeping shorter than expected, by a margin of less than 0,3%. >=20 > I figured failing the library test over a system (emulator?) bug is > undesirable, so I put in some tolerance for the drift. >=20 > Regstrapped on x86_64-linux-gnu, also tested with a cross to > aarch64-rtems6. Ok to install? >=20 > PS: I see nothing wrong with the implementation of clock_nanosleep (use= d > by nanosleep) on rtems6 that could cause it to wake up too early. I > suspect some artifact of the emulation environment. >=20 >=20 > for libstdc++-v3/ChangeLog >=20 > * testsuite/30_threads/this_thread/60421.cc: Tolerate a > slightly early wakeup. > --- > .../testsuite/30_threads/this_thread/60421.cc | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) >=20 > diff --git a/libstdc++-v3/testsuite/30_threads/this_thread/60421.cc b/l= ibstdc++-v3/testsuite/30_threads/this_thread/60421.cc > index 12dbeba1cc492..f3a5af453c4ad 100644 > --- a/libstdc++-v3/testsuite/30_threads/this_thread/60421.cc > +++ b/libstdc++-v3/testsuite/30_threads/this_thread/60421.cc > @@ -51,9 +51,10 @@ test02() > std::thread t([&result, &sleeping] { > auto start =3D std::chrono::system_clock::now(); > auto time =3D std::chrono::seconds(3); > + auto tolerance =3D std::chrono::milliseconds(10); > sleeping =3D true; > std::this_thread::sleep_for(time); > - result =3D std::chrono::system_clock::now() >=3D (start + time); > + result =3D std::chrono::system_clock::now() + tolerance >=3D (star= t + time); > sleeping =3D false; > }); > while (!sleeping) This looks like a bug in RTEMS or the BSP for the test platform. I would=20 first investigate this and then change the test which looks all right to = me. --=20 embedded brains GmbH Herr Sebastian HUBER Dornierstr. 4 82178 Puchheim Germany email: sebastian.huber@embedded-brains.de phone: +49-89-18 94 741 - 16 fax: +49-89-18 94 741 - 08 Registergericht: Amtsgericht M=C3=BCnchen Registernummer: HRB 157899 Vertretungsberechtigte Gesch=C3=A4ftsf=C3=BChrer: Peter Rasmussen, Thomas= D=C3=B6rfler Unsere Datenschutzerkl=C3=A4rung finden Sie hier: https://embedded-brains.de/datenschutzerklaerung/