public inbox for libc-alpha@sourceware.org
 help / color / mirror / Atom feed
* [PATCH] hppa: fix __O_SYNC to match the kernel
@ 2015-02-24  5:19 Mike Frysinger
  2015-02-24 14:50 ` John David Anglin
  2015-02-27  6:46 ` Mike Frysinger
  0 siblings, 2 replies; 17+ messages in thread
From: Mike Frysinger @ 2015-02-24  5:19 UTC (permalink / raw)
  To: libc-alpha; +Cc: carlos, John David Anglin

From: John David Anglin <dave.anglin@bell.net>

2015-02-24  John David Anglin <dave.anglin@bell.net>

	* sysdeps/unix/sysv/linux/hppa/bits/fcntl.h (__O_SYNC): Change
	to 00100000.
---
 sysdeps/unix/sysv/linux/hppa/bits/fcntl.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sysdeps/unix/sysv/linux/hppa/bits/fcntl.h b/sysdeps/unix/sysv/linux/hppa/bits/fcntl.h
index 9aad095..a800e28 100644
--- a/sysdeps/unix/sysv/linux/hppa/bits/fcntl.h
+++ b/sysdeps/unix/sysv/linux/hppa/bits/fcntl.h
@@ -27,7 +27,7 @@
 #define O_NONBLOCK	00200004 /* HPUX has separate NDELAY & NONBLOCK */
 #define __O_DSYNC	01000000
 #define __O_RSYNC	02000000 /* HPUX only */
-#define __O_SYNC	01000000
+#define __O_SYNC	00100000
 #define O_SYNC		(__O_SYNC|__O_DSYNC)
 
 #define O_BLKSEEK	00000100 /* HPUX only */
-- 
2.3.0

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] hppa: fix __O_SYNC to match the kernel
  2015-02-24  5:19 [PATCH] hppa: fix __O_SYNC to match the kernel Mike Frysinger
@ 2015-02-24 14:50 ` John David Anglin
  2015-02-24 16:20   ` Joseph Myers
  2015-02-27  6:53   ` Mike Frysinger
  2015-02-27  6:46 ` Mike Frysinger
  1 sibling, 2 replies; 17+ messages in thread
From: John David Anglin @ 2015-02-24 14:50 UTC (permalink / raw)
  To: Mike Frysinger, libc-alpha; +Cc: carlos

On 2015-02-24 12:19 AM, Mike Frysinger wrote:
> From: John David Anglin <dave.anglin@bell.net>
>
> 2015-02-24  John David Anglin <dave.anglin@bell.net>
>
> 	* sysdeps/unix/sysv/linux/hppa/bits/fcntl.h (__O_SYNC): Change
> 	to 00100000.
Mike, thanks for sending this change.  Reminds me I should go through 
the Debian glibc
patches and see what else needs sending.

Dave

-- 
John David Anglin  dave.anglin@bell.net

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] hppa: fix __O_SYNC to match the kernel
  2015-02-24 14:50 ` John David Anglin
@ 2015-02-24 16:20   ` Joseph Myers
  2015-02-27  5:06     ` Carlos O'Donell
  2015-02-27  6:53   ` Mike Frysinger
  1 sibling, 1 reply; 17+ messages in thread
From: Joseph Myers @ 2015-02-24 16:20 UTC (permalink / raw)
  To: John David Anglin; +Cc: Mike Frysinger, libc-alpha, carlos

On Tue, 24 Feb 2015, John David Anglin wrote:

> On 2015-02-24 12:19 AM, Mike Frysinger wrote:
> > From: John David Anglin <dave.anglin@bell.net>
> > 
> > 2015-02-24  John David Anglin <dave.anglin@bell.net>
> > 
> > 	* sysdeps/unix/sysv/linux/hppa/bits/fcntl.h (__O_SYNC): Change
> > 	to 00100000.
> Mike, thanks for sending this change.  Reminds me I should go through the
> Debian glibc
> patches and see what else needs sending.

See <https://sourceware.org/glibc/wiki/PortStatus> for a list of issues 
where the hppa port needs updating.  Also see Bugzilla for hppa bugs.  As 
usual, if the issue you're fixing was user-visible in a release, there 
should be a bug filed in Bugzilla for it.

-- 
Joseph S. Myers
joseph@codesourcery.com

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] hppa: fix __O_SYNC to match the kernel
  2015-02-24 16:20   ` Joseph Myers
@ 2015-02-27  5:06     ` Carlos O'Donell
  0 siblings, 0 replies; 17+ messages in thread
From: Carlos O'Donell @ 2015-02-27  5:06 UTC (permalink / raw)
  To: Joseph Myers, John David Anglin; +Cc: Mike Frysinger, libc-alpha, carlos

On 02/24/2015 11:20 AM, Joseph Myers wrote:
> On Tue, 24 Feb 2015, John David Anglin wrote:
> 
>> On 2015-02-24 12:19 AM, Mike Frysinger wrote:
>>> From: John David Anglin <dave.anglin@bell.net>
>>>
>>> 2015-02-24  John David Anglin <dave.anglin@bell.net>
>>>
>>> 	* sysdeps/unix/sysv/linux/hppa/bits/fcntl.h (__O_SYNC): Change
>>> 	to 00100000.
>> Mike, thanks for sending this change.  Reminds me I should go through the
>> Debian glibc
>> patches and see what else needs sending.
> 
> See <https://sourceware.org/glibc/wiki/PortStatus> for a list of issues 
> where the hppa port needs updating.  Also see Bugzilla for hppa bugs.  As 
> usual, if the issue you're fixing was user-visible in a release, there 
> should be a bug filed in Bugzilla for it.

Thanks Mike!

Cheers,
Carlos.
 

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] hppa: fix __O_SYNC to match the kernel
  2015-02-24  5:19 [PATCH] hppa: fix __O_SYNC to match the kernel Mike Frysinger
  2015-02-24 14:50 ` John David Anglin
@ 2015-02-27  6:46 ` Mike Frysinger
  1 sibling, 0 replies; 17+ messages in thread
From: Mike Frysinger @ 2015-02-27  6:46 UTC (permalink / raw)
  To: libc-alpha; +Cc: carlos, John David Anglin

[-- Attachment #1: Type: text/plain, Size: 49 bytes --]

i've pushed this now with Carlos' blessing
-mike

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] hppa: fix __O_SYNC to match the kernel
  2015-02-24 14:50 ` John David Anglin
  2015-02-24 16:20   ` Joseph Myers
@ 2015-02-27  6:53   ` Mike Frysinger
  2015-03-01 19:55     ` John David Anglin
  2015-03-07 17:13     ` John David Anglin
  1 sibling, 2 replies; 17+ messages in thread
From: Mike Frysinger @ 2015-02-27  6:53 UTC (permalink / raw)
  To: John David Anglin; +Cc: libc-alpha, carlos

[-- Attachment #1: Type: text/plain, Size: 383 bytes --]

On 24 Feb 2015 09:48, John David Anglin wrote:
> Mike, thanks for sending this change.  Reminds me I should go through 
> the Debian glibc
> patches and see what else needs sending.

this should be an easy one:
https://sourceware.org/git/?p=glibc.git;a=commitdiff;h=1c1d3f4f11b4a911a3b6ffab0aac61d5f8e02873

if you could provide context, i could shepherd it through.
-mike

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] hppa: fix __O_SYNC to match the kernel
  2015-02-27  6:53   ` Mike Frysinger
@ 2015-03-01 19:55     ` John David Anglin
  2015-03-01 22:24       ` Joseph Myers
  2015-03-08 19:52       ` Carlos O'Donell
  2015-03-07 17:13     ` John David Anglin
  1 sibling, 2 replies; 17+ messages in thread
From: John David Anglin @ 2015-03-01 19:55 UTC (permalink / raw)
  To: Mike Frysinger; +Cc: libc-alpha, carlos

Hi Mike,

On 2015-02-27, at 1:53 AM, Mike Frysinger wrote:

> On 24 Feb 2015 09:48, John David Anglin wrote:
>> Mike, thanks for sending this change.  Reminds me I should go through 
>> the Debian glibc
>> patches and see what else needs sending.
> 
> this should be an easy one:
> https://sourceware.org/git/?p=glibc.git;a=commitdiff;h=1c1d3f4f11b4a911a3b6ffab0aac61d5f8e02873

I started to write a ChangeLog entry and realized that there is a problem with the change
to feholdexcept.c.  I believe the second set of bufptr needs to be removed.

> 
> if you could provide context, i could shepherd it through.

The patch is intended to fix problems in accessing and updating the floating-point status register
and exception registers.

Originally, the routines were written to save and restore the status register and the entire set of 
(all seven) exception registers.  At some point, the code was changed to just manipulate the status
register and the first exception register.  A couple of bugs were introduced when this was done.

In fesetenv.c, the post increment of bufptr was retained in the first asm but the constraint for it
does not indicate that bufptr is modified.  As a result, GCC miscompiled fesetenv.  We get better
code by not modifying bufptr as GCC doesn't have to reload bufptr.

The main bug in feholdexcept is the second set of bufptr.  This existed to the restore the exception
registers in reverse order.  This statement should have been removed when the code was changed
to only update the status and first exception registers.  The offset used in the statement is also off
by a factor two, so it probably never worked correctly.  With the current patch, the code loads zero to
the status and exception register.  As a result, the T bit is not set properly.

I will test a revised change to feholdexcept and get back to you.

Dave
--
John David Anglin	dave.anglin@bell.net



^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] hppa: fix __O_SYNC to match the kernel
  2015-03-01 19:55     ` John David Anglin
@ 2015-03-01 22:24       ` Joseph Myers
  2015-03-02 13:18         ` John David Anglin
  2015-03-08 19:52       ` Carlos O'Donell
  1 sibling, 1 reply; 17+ messages in thread
From: Joseph Myers @ 2015-03-01 22:24 UTC (permalink / raw)
  To: John David Anglin; +Cc: Mike Frysinger, libc-alpha, carlos

On Sun, 1 Mar 2015, John David Anglin wrote:

> The main bug in feholdexcept is the second set of bufptr.  This existed 
> to the restore the exception registers in reverse order.  This statement 
> should have been removed when the code was changed to only update the 
> status and first exception registers.  The offset used in the statement 
> is also off by a factor two, so it probably never worked correctly.  
> With the current patch, the code loads zero to the status and exception 
> register.  As a result, the T bit is not set properly.

If this bug was user-visible in a release, there should be a bug filed in 
Bugzilla for it (this generally applies when fixing any bug that was 
user-visible in a release).

-- 
Joseph S. Myers
joseph@codesourcery.com

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] hppa: fix __O_SYNC to match the kernel
  2015-03-01 22:24       ` Joseph Myers
@ 2015-03-02 13:18         ` John David Anglin
  2015-03-02 14:09           ` Joseph Myers
  2015-03-11  7:33           ` Mike Frysinger
  0 siblings, 2 replies; 17+ messages in thread
From: John David Anglin @ 2015-03-02 13:18 UTC (permalink / raw)
  To: Joseph Myers; +Cc: Mike Frysinger, libc-alpha, carlos

On 2015-03-01, at 5:24 PM, Joseph Myers wrote:

> If this bug was user-visible in a release, there should be a bug filed in 
> Bugzilla for it (this generally applies when fixing any bug that was 
> user-visible in a release).

https://sourceware.org/bugzilla/show_bug.cgi?id=18068

Dave
--
John David Anglin	dave.anglin@bell.net



^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] hppa: fix __O_SYNC to match the kernel
  2015-03-02 13:18         ` John David Anglin
@ 2015-03-02 14:09           ` Joseph Myers
  2015-03-11  7:33           ` Mike Frysinger
  1 sibling, 0 replies; 17+ messages in thread
From: Joseph Myers @ 2015-03-02 14:09 UTC (permalink / raw)
  To: John David Anglin; +Cc: Mike Frysinger, libc-alpha, carlos

On Mon, 2 Mar 2015, John David Anglin wrote:

> On 2015-03-01, at 5:24 PM, Joseph Myers wrote:
> 
> > If this bug was user-visible in a release, there should be a bug filed in 
> > Bugzilla for it (this generally applies when fixing any bug that was 
> > user-visible in a release).
> 
> https://sourceware.org/bugzilla/show_bug.cgi?id=18068

The ports component should no longer be used (i.e. should only contain 
resolved bugs; I don't know if it's possible to disable filing new bugs in 
a component without affecting existing bugs in that component); bugs for 
all architectures should now go in components such as libc.

As the fix for this has been checked in, the other steps needed are 
putting the [BZ #18068] marker in the ChangeLog entry, adding the bug to 
the list of fixed bugs in NEWS, and closing the bug as fixed.

-- 
Joseph S. Myers
joseph@codesourcery.com

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] hppa: fix __O_SYNC to match the kernel
  2015-02-27  6:53   ` Mike Frysinger
  2015-03-01 19:55     ` John David Anglin
@ 2015-03-07 17:13     ` John David Anglin
  2015-03-08 19:53       ` Carlos O'Donell
  2015-03-11  6:29       ` Carlos O'Donell
  1 sibling, 2 replies; 17+ messages in thread
From: John David Anglin @ 2015-03-07 17:13 UTC (permalink / raw)
  To: Mike Frysinger; +Cc: libc-alpha, carlos

[-- Attachment #1: Type: text/plain, Size: 793 bytes --]

On 2015-02-27, at 1:53 AM, Mike Frysinger wrote:

> On 24 Feb 2015 09:48, John David Anglin wrote:
>> Mike, thanks for sending this change.  Reminds me I should go through 
>> the Debian glibc
>> patches and see what else needs sending.
> 
> this should be an easy one:
> https://sourceware.org/git/?p=glibc.git;a=commitdiff;h=1c1d3f4f11b4a911a3b6ffab0aac61d5f8e02873
> 
> if you could provide context, i could shepherd it through.
> -mike


Attached is an updated patch.  It fixes the saving and updating of the floating-point environment
in feholdexcept and fesetenv.  The glibc test "test-fenv" passes with this change.

The patch has been tested with Debian glibc 2.19-15+b1 with no observed regressions.

Regards,
Dave
--
John David Anglin	dave.anglin@bell.net



[-- Attachment #2: local-fpa-r2.diff.txt --]
[-- Type: text/plain, Size: 2212 bytes --]

2015-03-07  John David Anglin  <danglin@gcc.gnu.org>

	* ports/sysdeps/hppa/fpu/feholdexcpt.c (feholdexcept): Don't modify
	bufptr in asms.
	* ports/sysdeps/hppa/fpu/fesetenv.c (fesetenv): Likewise.

Index: glibc-2.19/ports/sysdeps/hppa/fpu/feholdexcpt.c
===================================================================
--- glibc-2.19.orig/ports/sysdeps/hppa/fpu/feholdexcpt.c
+++ glibc-2.19/ports/sysdeps/hppa/fpu/feholdexcpt.c
@@ -29,8 +29,8 @@ feholdexcept (fenv_t *envp)
   /* Store the environment.  */
   bufptr = clear.buf;
   __asm__ (
-	   "fstd,ma %%fr0,8(%1)\n"
-	   : "=m" (clear), "+r" (bufptr) : : "%r0");
+	   "fstd %%fr0,0(%1)\n"
+	   : "=m" (clear) : "r" (bufptr) : "%r0");
   memcpy (envp, &clear.env, sizeof (fenv_t));
 
   /* Clear exception queues */
@@ -40,11 +40,9 @@ feholdexcept (fenv_t *envp)
   /* Now clear all flags  */
   clear.env.__status_word &= ~(FE_ALL_EXCEPT << 27);
 
-  /* Load the new environment. Note: fr0 must load last to enable T-bit
-     Thus we start bufptr at the end and work backwards */
-  bufptr = (unsigned long long *)((unsigned int)(clear.buf) + sizeof(unsigned int)*4);
+  /* Load the new environment. Note: fr0 must load last to enable T-bit.  */
   __asm__ (
-	   "fldd,mb -8(%0),%%fr0\n"
+	   "fldd 0(%0),%%fr0\n"
 	   : : "r" (bufptr), "m" (clear) : "%r0");
 
   return 0;
Index: glibc-2.19/ports/sysdeps/hppa/fpu/fesetenv.c
===================================================================
--- glibc-2.19.orig/ports/sysdeps/hppa/fpu/fesetenv.c
+++ glibc-2.19/ports/sysdeps/hppa/fpu/fesetenv.c
@@ -33,7 +33,7 @@ fesetenv (const fenv_t *envp)
      we want to use from the environment specified by the parameter.  */
   bufptr = temp.buf;
   __asm__ (
-	   "fstd,ma %%fr0,8(%1)\n"
+	   "fstd %%fr0,0(%1)\n"
 	   : "=m" (temp) : "r" (bufptr) : "%r0");
 
   temp.env.__status_word &= ~(FE_ALL_EXCEPT
@@ -54,7 +54,7 @@ fesetenv (const fenv_t *envp)
      we take advantage of that to load in reverse order so fr0
      is loaded last and T-Bit is enabled. */
   __asm__ (
-	   "fldd,mb -8(%1),%%fr0\n"
+	   "fldd 0(%1),%%fr0\n"
 	   : : "m" (temp), "r" (bufptr) : "%r0" );
 
   /* Success.  */

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] hppa: fix __O_SYNC to match the kernel
  2015-03-01 19:55     ` John David Anglin
  2015-03-01 22:24       ` Joseph Myers
@ 2015-03-08 19:52       ` Carlos O'Donell
  2015-03-08 21:30         ` John David Anglin
  1 sibling, 1 reply; 17+ messages in thread
From: Carlos O'Donell @ 2015-03-08 19:52 UTC (permalink / raw)
  To: John David Anglin, Mike Frysinger; +Cc: libc-alpha, carlos

On 03/01/2015 02:55 PM, John David Anglin wrote:
> In fesetenv.c, the post increment of bufptr was retained in the first asm but the constraint for it
> does not indicate that bufptr is modified.  As a result, GCC miscompiled fesetenv.  We get better
> code by not modifying bufptr as GCC doesn't have to reload bufptr.

It uses "+r" which means the operand, the register, is read and modified.

The fact that we get better code, or work around a gcc bug, is a good reason
to make the change, but I don't see what's wrong with the original code.

> The main bug in feholdexcept is the second set of bufptr.  This existed to the restore the exception
> registers in reverse order.  This statement should have been removed when the code was changed
> to only update the status and first exception registers.  The offset used in the statement is also off
> by a factor two, so it probably never worked correctly.  With the current patch, the code loads zero to
> the status and exception register.  As a result, the T bit is not set properly.

This doesn't sound right either.

fstd,ma %%fr0,8(%1)

Results in a store to bufptr, with bubptr-=8 *after* the store.

fldd,mb -8(%0),%%fr0

Results in a bufptr-=8, followed by a load from bufptr to fr0.

Thus while the operations are left-over from previous iterations of
the code where we did save/load all of the registers, the above code
is idempotent with respect to your changes.

The one problem is that bufptr is not marked as changed in the *second*
assembly contstraint which just lists bufptr as input (which is wrong)
and an optimization might reorder things such that this breaks.

Cheers,
Carlos.

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] hppa: fix __O_SYNC to match the kernel
  2015-03-07 17:13     ` John David Anglin
@ 2015-03-08 19:53       ` Carlos O'Donell
  2015-03-11  6:29       ` Carlos O'Donell
  1 sibling, 0 replies; 17+ messages in thread
From: Carlos O'Donell @ 2015-03-08 19:53 UTC (permalink / raw)
  To: John David Anglin, Mike Frysinger; +Cc: libc-alpha, carlos

On 03/07/2015 12:13 PM, John David Anglin wrote:
> On 2015-02-27, at 1:53 AM, Mike Frysinger wrote:
> 
>> > On 24 Feb 2015 09:48, John David Anglin wrote:
>>> >> Mike, thanks for sending this change.  Reminds me I should go through 
>>> >> the Debian glibc
>>> >> patches and see what else needs sending.
>> > 
>> > this should be an easy one:
>> > https://sourceware.org/git/?p=glibc.git;a=commitdiff;h=1c1d3f4f11b4a911a3b6ffab0aac61d5f8e02873
>> > 
>> > if you could provide context, i could shepherd it through.
>> > -mike
> 
> Attached is an updated patch.  It fixes the saving and updating of the floating-point environment
> in feholdexcept and fesetenv.  The glibc test "test-fenv" passes with this change.
> 
> The patch has been tested with Debian glibc 2.19-15+b1 with no observed regressions.

Testing right now.

Will commit if it looks good.

c.

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] hppa: fix __O_SYNC to match the kernel
  2015-03-08 19:52       ` Carlos O'Donell
@ 2015-03-08 21:30         ` John David Anglin
  2015-03-09 12:36           ` Carlos O'Donell
  0 siblings, 1 reply; 17+ messages in thread
From: John David Anglin @ 2015-03-08 21:30 UTC (permalink / raw)
  To: Carlos O'Donell; +Cc: Mike Frysinger, libc-alpha, carlos


On 2015-03-08, at 3:52 PM, Carlos O'Donell wrote:

> On 03/01/2015 02:55 PM, John David Anglin wrote:
>> In fesetenv.c, the post increment of bufptr was retained in the first asm but the constraint for it
>> does not indicate that bufptr is modified.  As a result, GCC miscompiled fesetenv.  We get better
>> code by not modifying bufptr as GCC doesn't have to reload bufptr.
> 
> It uses "+r" which means the operand, the register, is read and modified.
> 
> The fact that we get better code, or work around a gcc bug, is a good reason
> to make the change, but I don't see what's wrong with the original code.
> 
>> The main bug in feholdexcept is the second set of bufptr.  This existed to the restore the exception
>> registers in reverse order.  This statement should have been removed when the code was changed
>> to only update the status and first exception registers.  The offset used in the statement is also off
>> by a factor two, so it probably never worked correctly.  With the current patch, the code loads zero to
>> the status and exception register.  As a result, the T bit is not set properly.
> 
> This doesn't sound right either.
> 
> fstd,ma %%fr0,8(%1)
> 
> Results in a store to bufptr, with bubptr-=8 *after* the store.
> 
> fldd,mb -8(%0),%%fr0
> 
> Results in a bufptr-=8, followed by a load from bufptr to fr0.
> 
> Thus while the operations are left-over from previous iterations of
> the code where we did save/load all of the registers, the above code
> is idempotent with respect to your changes.
> 
> The one problem is that bufptr is not marked as changed in the *second*
> assembly contstraint which just lists bufptr as input (which is wrong)
> and an optimization might reorder things such that this breaks.

Exactly.  The post and pre-increments could be used but one needs to ensure that the bufptr asm
arguments are marked as written.  I suspect instructions with post and pre-increment are slightly
slower but the difference may not be visible given the length of the pipeline.  We want to avoid
re-initializing bufptr.

Thus, I believe it's better to not modify bufptr in the asms in these two functions.

Dave
--
John David Anglin	dave.anglin@bell.net



^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] hppa: fix __O_SYNC to match the kernel
  2015-03-08 21:30         ` John David Anglin
@ 2015-03-09 12:36           ` Carlos O'Donell
  0 siblings, 0 replies; 17+ messages in thread
From: Carlos O'Donell @ 2015-03-09 12:36 UTC (permalink / raw)
  To: John David Anglin; +Cc: Mike Frysinger, libc-alpha, carlos

On 03/08/2015 05:30 PM, John David Anglin wrote:
> Thus, I believe it's better to not modify bufptr in the asms in these two functions.

Agreed. I'm running another test run right now and will commit your changes
if it makes things better (which I assume it does).

Cheers,
Carlos.

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] hppa: fix __O_SYNC to match the kernel
  2015-03-07 17:13     ` John David Anglin
  2015-03-08 19:53       ` Carlos O'Donell
@ 2015-03-11  6:29       ` Carlos O'Donell
  1 sibling, 0 replies; 17+ messages in thread
From: Carlos O'Donell @ 2015-03-11  6:29 UTC (permalink / raw)
  To: John David Anglin, Mike Frysinger; +Cc: libc-alpha, carlos

On 03/07/2015 12:13 PM, John David Anglin wrote:
> On 2015-02-27, at 1:53 AM, Mike Frysinger wrote:
> 
>> On 24 Feb 2015 09:48, John David Anglin wrote:
>>> Mike, thanks for sending this change.  Reminds me I should go through 
>>> the Debian glibc
>>> patches and see what else needs sending.
>>
>> this should be an easy one:
>> https://sourceware.org/git/?p=glibc.git;a=commitdiff;h=1c1d3f4f11b4a911a3b6ffab0aac61d5f8e02873
>>
>> if you could provide context, i could shepherd it through.
>> -mike
> 
> 
> Attached is an updated patch.  It fixes the saving and updating of the floating-point environment
> in feholdexcept and fesetenv.  The glibc test "test-fenv" passes with this change.
> 
> The patch has been tested with Debian glibc 2.19-15+b1 with no observed regressions.

Applied.

commit fae1aa8d226ce860124efd67ede03004b19b89e2
Author: John David Anglin <danglin@gcc.gnu.org>
Date:   Tue Mar 10 23:43:50 2015 -0400

    hppa: Fix feholdexcpt and fesetenv (Bug 18110).
    
    The constraints in the inline assembly in feholdexcept and fesetenv
    are incorrect. The assembly modifies the buffer pointer, but doesn't
    express that in the constraints. The simple fix is to remove the
    modification of the buffer pointer which is no longer required by
    the existing code, and adjust the one constraint that did express
    the modification of bufptr.
    
    The change fixes test-fenv when glibc is compiled with recent gcc.

Cheers,
Carlos.

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH] hppa: fix __O_SYNC to match the kernel
  2015-03-02 13:18         ` John David Anglin
  2015-03-02 14:09           ` Joseph Myers
@ 2015-03-11  7:33           ` Mike Frysinger
  1 sibling, 0 replies; 17+ messages in thread
From: Mike Frysinger @ 2015-03-11  7:33 UTC (permalink / raw)
  To: John David Anglin; +Cc: Joseph Myers, libc-alpha, carlos

[-- Attachment #1: Type: text/plain, Size: 1230 bytes --]

On 02 Mar 2015 08:18, John David Anglin wrote:
> On 2015-03-01, at 5:24 PM, Joseph Myers wrote:
> > If this bug was user-visible in a release, there should be a bug filed in 
> > Bugzilla for it (this generally applies when fixing any bug that was 
> > user-visible in a release).
> 
> https://sourceware.org/bugzilla/show_bug.cgi?id=18068

thanks, updated the source:

--- a/ChangeLog
+++ b/ChangeLog
@@ -351,6 +351,7 @@
 
 2015-02-27  John David Anglin  <dave.anglin@bell.net>
 
+	[BZ #18068]
 	* sysdeps/unix/sysv/linux/hppa/bits/fcntl.h (__O_SYNC): Change
 	to 00100000.
 
--- a/NEWS
+++ b/NEWS
@@ -13,8 +13,8 @@ Version 2.22
   16560, 16783, 17269, 17523, 17569, 17588, 17631, 17711, 17776, 17779,
   17792, 17836, 17912, 17916, 17932, 17944, 17949, 17964, 17965, 17967,
   17969, 17978, 17987, 17991, 17996, 17998, 17999, 18019, 18020, 18029,
-  18030, 18032, 18036, 18038, 18039, 18042, 18043, 18046, 18047, 18104,
-  18110, 18111.
+  18030, 18032, 18036, 18038, 18039, 18042, 18043, 18046, 18047, 18068,
+  18104, 18110, 18111.
 
 * Character encoding and ctype tables were updated to Unicode 7.0.0, using
   new generator scripts contributed by Pravin Satpute and Mike FABIAN (Red
-mike

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

^ permalink raw reply	[flat|nested] 17+ messages in thread

end of thread, other threads:[~2015-03-11  7:33 UTC | newest]

Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-02-24  5:19 [PATCH] hppa: fix __O_SYNC to match the kernel Mike Frysinger
2015-02-24 14:50 ` John David Anglin
2015-02-24 16:20   ` Joseph Myers
2015-02-27  5:06     ` Carlos O'Donell
2015-02-27  6:53   ` Mike Frysinger
2015-03-01 19:55     ` John David Anglin
2015-03-01 22:24       ` Joseph Myers
2015-03-02 13:18         ` John David Anglin
2015-03-02 14:09           ` Joseph Myers
2015-03-11  7:33           ` Mike Frysinger
2015-03-08 19:52       ` Carlos O'Donell
2015-03-08 21:30         ` John David Anglin
2015-03-09 12:36           ` Carlos O'Donell
2015-03-07 17:13     ` John David Anglin
2015-03-08 19:53       ` Carlos O'Donell
2015-03-11  6:29       ` Carlos O'Donell
2015-02-27  6:46 ` Mike Frysinger

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).