Message ID | gerrit.1572857958000.I37bc20f3449b9e358f32879ed231720c969965b4@gnutoolchain-gerrit.osci.io |
---|---|
State | Committed, archived |
Headers |
Received: (qmail 113486 invoked by alias); 4 Nov 2019 08:59:23 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: <libc-alpha.sourceware.org> List-Unsubscribe: <mailto:libc-alpha-unsubscribe-##L=##H@sourceware.org> List-Subscribe: <mailto:libc-alpha-subscribe@sourceware.org> List-Archive: <http://sourceware.org/ml/libc-alpha/> List-Post: <mailto:libc-alpha@sourceware.org> List-Help: <mailto:libc-alpha-help@sourceware.org>, <http://sourceware.org/ml/#faqs> Sender: libc-alpha-owner@sourceware.org Delivered-To: mailing list libc-alpha@sourceware.org Received: (qmail 113471 invoked by uid 89); 4 Nov 2019 08:59:22 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-21.3 required=5.0 tests=AWL, BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3 autolearn=ham version=3.3.1 spammy=lingering, HContent-Transfer-Encoding:8bit X-HELO: mx1.osci.io X-Gerrit-PatchSet: 1 Date: Mon, 4 Nov 2019 03:59:18 -0500 From: "Florian Weimer (Code Review)" <gerrit@gnutoolchain-gerrit.osci.io> To: libc-alpha@sourceware.org Cc: Florian Weimer <fweimer@redhat.com> Message-ID: <gerrit.1572857958000.I37bc20f3449b9e358f32879ed231720c969965b4@gnutoolchain-gerrit.osci.io> Auto-Submitted: auto-generated X-Gerrit-MessageType: newchange Subject: [review] hurd: Remove lingering references to the time function X-Gerrit-Change-Id: I37bc20f3449b9e358f32879ed231720c969965b4 X-Gerrit-Change-Number: 505 X-Gerrit-ChangeURL: <https://gnutoolchain-gerrit.osci.io/r/c/glibc/+/505> X-Gerrit-Commit: 25bf44cd1bae6d4244078f5e043b3944881a3e37 References: <gerrit.1572857958000.I37bc20f3449b9e358f32879ed231720c969965b4@gnutoolchain-gerrit.osci.io> Reply-To: fweimer@redhat.com, fweimer@redhat.com, libc-alpha@sourceware.org MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Disposition: inline User-Agent: Gerrit/3.0.3-75-g9005159e5d Content-Type: text/plain; charset=UTF-8 |
Commit Message
Simon Marchi (Code Review)
Nov. 4, 2019, 8:59 a.m. UTC
Change URL: https://gnutoolchain-gerrit.osci.io/r/c/glibc/+/505 ...................................................................... hurd: Remove lingering references to the time function They cause a check-localplt failure after commit f9a7554009cf38f39. Fixes: f9a7554009cf38f390e74fcabc5b49f974f72382 Change-Id: I37bc20f3449b9e358f32879ed231720c969965b4 --- M sysdeps/mach/sleep.c 1 file changed, 2 insertions(+), 2 deletions(-)
Comments
Adhemerval Zanella has posted comments on this change. Change URL: https://gnutoolchain-gerrit.osci.io/r/c/glibc/+/505 ...................................................................... Patch Set 1: Code-Review+1 LGTM.
Adhemerval Zanella has posted comments on this change. Change URL: https://gnutoolchain-gerrit.osci.io/r/c/glibc/+/505 ...................................................................... Patch Set 1: Code-Review+2
Florian Weimer (Code Review), le lun. 04 nov. 2019 03:59:18 -0500, a ecrit: > Change URL: https://gnutoolchain-gerrit.osci.io/r/c/glibc/+/505 > ...................................................................... > > hurd: Remove lingering references to the time function > > They cause a check-localplt failure after commit f9a7554009cf38f39. > > Fixes: f9a7554009cf38f390e74fcabc5b49f974f72382 > Change-Id: I37bc20f3449b9e358f32879ed231720c969965b4 Reviewed-by: Samuel Thibault <samuel.thibault@ens-lyon.org> Thanks! > --- > M sysdeps/mach/sleep.c > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/sysdeps/mach/sleep.c b/sysdeps/mach/sleep.c > index b30ec7b..267928e 100644 > --- a/sysdeps/mach/sleep.c > +++ b/sysdeps/mach/sleep.c > @@ -33,10 +33,10 @@ > > recv = __mach_reply_port (); > > - before = time (NULL); > + before = time_now (); > (void) __mach_msg (NULL, MACH_RCV_MSG|MACH_RCV_TIMEOUT|MACH_RCV_INTERRUPT, > 0, 0, recv, seconds * 1000, MACH_PORT_NULL); > - after = time (NULL); > + after = time_now (); > __mach_port_destroy (__mach_task_self (), recv); > > return seconds - (after - before); > > -- > Gerrit-Project: glibc > Gerrit-Branch: master > Gerrit-Change-Id: I37bc20f3449b9e358f32879ed231720c969965b4 > Gerrit-Change-Number: 505 > Gerrit-PatchSet: 1 > Gerrit-Owner: Florian Weimer <fweimer@redhat.com> > Gerrit-MessageType: newchange >
Florian Weimer has posted comments on this change. Change URL: https://gnutoolchain-gerrit.osci.io/r/c/glibc/+/505 ...................................................................... Patch Set 2: Code-Review+2
* Florian Weimer: > Florian Weimer has posted comments on this change. > > Change URL: https://gnutoolchain-gerrit.osci.io/r/c/glibc/+/505 > ...................................................................... > > > Patch Set 2: Code-Review+2 Not sure what was going on here. The changeset did not show up as merged in Gerrit after Gerrit saw it pushed. Gerrit created a v2, and I had to +2 it so that it got merged. This did not happen with the other change, <https://gnutoolchain-gerrit.osci.io/r/c/glibc/+/506>, even though pushing that created a v2 as well. I added Reviewed-By: lines in both cases. Simon, would you please have a look? Is this an artifact of using Gerrit in non-push mode? Have you seen something similar with GDB? Thanks, Florian
On 2019-11-07 4:07 a.m., Florian Weimer wrote: > * Florian Weimer: > >> Florian Weimer has posted comments on this change. >> >> Change URL: https://gnutoolchain-gerrit.osci.io/r/c/glibc/+/505 >> ...................................................................... >> >> >> Patch Set 2: Code-Review+2 > > Not sure what was going on here. The changeset did not show up as > merged in Gerrit after Gerrit saw it pushed. Gerrit created a v2, and I > had to +2 it so that it got merged. > > This did not happen with the other change, > <https://gnutoolchain-gerrit.osci.io/r/c/glibc/+/506>, even though > pushing that created a v2 as well. > > I added Reviewed-By: lines in both cases. > > Simon, would you please have a look? Is this an artifact of using > Gerrit in non-push mode? Have you seen something similar with GDB? Yes, we have seen the same maybe 2-3 times (so it was more the exception than the norm). Gerrit marked the change as merged, but it did not close it. And as you said, doing some label change (like +2) seemed to make Gerrit re-evaluate the state of the change correctly. So it's a known bug. I haven't really looked into it because it doesn't happen often, so it's hard to reproduce. Simon
diff --git a/sysdeps/mach/sleep.c b/sysdeps/mach/sleep.c index b30ec7b..267928e 100644 --- a/sysdeps/mach/sleep.c +++ b/sysdeps/mach/sleep.c @@ -33,10 +33,10 @@ recv = __mach_reply_port (); - before = time (NULL); + before = time_now (); (void) __mach_msg (NULL, MACH_RCV_MSG|MACH_RCV_TIMEOUT|MACH_RCV_INTERRUPT, 0, 0, recv, seconds * 1000, MACH_PORT_NULL); - after = time (NULL); + after = time_now (); __mach_port_destroy (__mach_task_self (), recv); return seconds - (after - before);