]> sourceware.org Git - newlib-cygwin.git/log
newlib-cygwin.git
4 years agoCygwin: raw disk I/O: lock floppys as well
Corinna Vinschen [Fri, 24 Apr 2020 14:22:26 +0000 (16:22 +0200)]
Cygwin: raw disk I/O: lock floppys as well

The workaround to access the full disk required since Vista
and described in http://support.microsoft.com/kb/942448
(NOT ACCESSIBLE at the time of writing this commit message)
is required on floppy drives as well.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: raw disk I/O: Fix return value in error case
Corinna Vinschen [Fri, 24 Apr 2020 14:19:09 +0000 (16:19 +0200)]
Cygwin: raw disk I/O: Fix return value in error case

The cast to generate the return value uses a DWORD variable
as test and set value.  The error case is the constant -1.
Given the type of the other half of the conditioal expression,
-1 is cast to DWORD as well.

On 64 bit, this results in the error case returning a 32 bit
-1 value which is equivalent to (ssize_t) 4294967295 rather
than (ssize_t) -1.

Add a fixing cast.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: file I/O: make sure to treat write return value as ssize_t
Corinna Vinschen [Fri, 24 Apr 2020 14:14:43 +0000 (16:14 +0200)]
Cygwin: file I/O: make sure to treat write return value as ssize_t

The return type of fhandler write methods is ssize_t.  Don't
use an int to store the return value, use ssize_t.  Use ptrdiff_t
for the buffer size.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: accounts: Report unrecognized db_* nsswitch.conf keywords
David Macek via Cygwin-patches [Mon, 20 Apr 2020 17:21:15 +0000 (19:21 +0200)]
Cygwin: accounts: Report unrecognized db_* nsswitch.conf keywords

Signed-off-by: David Macek <david.macek.0@gmail.com>
4 years agoCygwin: accounts: Don't keep old schemes when parsing nsswitch.conf
David Macek via Cygwin-patches [Mon, 20 Apr 2020 17:21:00 +0000 (19:21 +0200)]
Cygwin: accounts: Don't keep old schemes when parsing nsswitch.conf

The implicit assumption seemed to be that any subsequent occurence of
the same setting in nsswitch.conf is supposed to rewrite the previous
ones completely.  This was not the case if the third or any further
schema was previously defined and the last line defined less than that
(but at least 2), for example:

```
db_home: windows cygwin /myhome/%U
db_home: cygwin desc
```

Let's document this behavior as well.

Signed-off-by: David Macek <david.macek.0@gmail.com>
4 years agoCygwin: accounts: Unify nsswitch.conf db_* defaults
David Macek via Cygwin-patches [Tue, 21 Apr 2020 18:31:09 +0000 (20:31 +0200)]
Cygwin: accounts: Unify nsswitch.conf db_* defaults

Signed-off-by: David Macek <david.macek.0@gmail.com>
4 years agoCygwin: Add David Macek to CONTRIBUTORS
Corinna Vinschen [Tue, 21 Apr 2020 08:38:57 +0000 (10:38 +0200)]
Cygwin: Add David Macek to CONTRIBUTORS

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: symlinks: fix WSL symlink creation if cygdrive prefix is /
Corinna Vinschen [Tue, 21 Apr 2020 08:31:53 +0000 (10:31 +0200)]
Cygwin: symlinks: fix WSL symlink creation if cygdrive prefix is /

If the cygdrive prefix is /, then the following happens right now:

  $ ln -s /tmp/foo .
  $ ls -l foo
  lrwxrwxrwx 1 user group 12 Apr 15 23:44 foo -> /mnt/tmp/foo

Fix this by skipping cygdrive prefix conversion to WSL drive
prefix "/mnt", if the cygdrive prefix is just "/".  There's no
satisfying way to do the right thing all the time in this case
anyway.  For a description and the alternatives, see
https://cygwin.com/pipermail/cygwin-developers/2020-April/011859.html

Also, fix a typo in a comment.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agocygheap_pwdgrp: Handle invalid db_* entries correctly
David Macek via Cygwin-patches [Thu, 16 Apr 2020 21:09:07 +0000 (23:09 +0200)]
cygheap_pwdgrp: Handle invalid db_* entries correctly

If the first scheme in db_* was invalid, the code would think there
were no schemes specified and replace the second scheme with
NSS_SCHEME_DESC.

Signed-off-by: David Macek <david.macek.0@gmail.com>
4 years agoCygwin: faq: disable outdated "sshd in domain" faq
Corinna Vinschen [Tue, 14 Apr 2020 11:26:37 +0000 (13:26 +0200)]
Cygwin: faq: disable outdated "sshd in domain" faq

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoproc_cpuinfo: Add PPIN support for AMD
Brian Inglis [Sat, 11 Apr 2020 04:35:28 +0000 (22:35 -0600)]
proc_cpuinfo: Add PPIN support for AMD

Newer AMD CPUs support a feature called protected processor
identification number (PPIN). This feature can be detected via
CPUID_Fn80000008_EBX[23].

4 years agoCygwin: utils: override definition of PMEM_EXTENDED_PARAMETER
Corinna Vinschen [Tue, 7 Apr 2020 16:54:59 +0000 (18:54 +0200)]
Cygwin: utils: override definition of PMEM_EXTENDED_PARAMETER

PMEM_EXTENDED_PARAMETER is defined in the local winlean.h as long
as mingw-w64 doesn't define it (in winnt.h).  ntdll.h needs the
definition for declaring NtMapViewOfSectionEx.  cygpath.cc and ps.cc
both include ntdll.h but not winlean.h, so they complain about the
missing definition.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: mmap_alloc: fix comment to document using the extended memory API
Corinna Vinschen [Tue, 7 Apr 2020 12:17:04 +0000 (14:17 +0200)]
Cygwin: mmap_alloc: fix comment to document using the extended memory API

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: threads: use extended memory API if available
Corinna Vinschen [Tue, 7 Apr 2020 15:41:05 +0000 (17:41 +0200)]
Cygwin: threads: use extended memory API if available

So far Cygwin was jumping through hoops to restrict memory
allocation to specific regions.  With the advent of VirtualAlloc2
and MapViewOfFile3 (and it's NT counterpart NtMapViewOfSectionEx),
we can skip searching for free space in the specific regions
and just call these functions and let the OS do the job more
efficiently and less racy.

Use VirtualAlloc2 on W10 1803 and later in thread stack allocation.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: mmap: use extended memory API if available
Corinna Vinschen [Tue, 7 Apr 2020 12:16:09 +0000 (14:16 +0200)]
Cygwin: mmap: use extended memory API if available

So far Cygwin was jumping through hoops to restrict memory
allocation to specific regions.  With the advent of VirtualAlloc2
and MapViewOfFile3 (and it's NT counterpart NtMapViewOfSectionEx),
we can skip searching for free space in the specific regions
and just call these functions and let the OS do the job more
efficiently and less racy.

Use the new functions on W10 1803 and later in mmap.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: memory: declare extended memory API
Corinna Vinschen [Tue, 7 Apr 2020 12:13:50 +0000 (14:13 +0200)]
Cygwin: memory: declare extended memory API

Windows 10 1803 introduced an extended memory API allowing
to specify memory regions allocations are to be taken off.

In preparation of using this API, define the struct
MEM_EXTENDED_PARAMETER and friends.  Declare and allow to
autoload the functions VirtualAlloc2 and NtMapViewOfSectionEx.

Introduce a wincap flag has_extended_mem_api.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: threads: add missing include of mmap_alloc.h
Corinna Vinschen [Tue, 7 Apr 2020 15:40:38 +0000 (17:40 +0200)]
Cygwin: threads: add missing include of mmap_alloc.h

This is needed for using mmap_alloc.alloc()

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: threads: lower thread size from pool to 256 Megs
Corinna Vinschen [Tue, 7 Apr 2020 11:43:17 +0000 (13:43 +0200)]
Cygwin: threads: lower thread size from pool to 256 Megs

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: threads: use mmap area to fulfill requests for big stacks
Corinna Vinschen [Tue, 7 Apr 2020 12:09:45 +0000 (14:09 +0200)]
Cygwin: threads: use mmap area to fulfill requests for big stacks

Otherwise big stacks have a higher probability to collide with
randomized PEBs and TEBs after fork.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: symlinks: Allow traversing WSL symlinks
Corinna Vinschen [Sun, 5 Apr 2020 14:17:36 +0000 (16:17 +0200)]
Cygwin: symlinks: Allow traversing WSL symlinks

Unfortunately Windows doesn't understand WSL symlinks,
despite being a really easy job.  NT functions trying
to access paths traversing WSL symlinks return the status
code STATUS_IO_REPARSE_TAG_NOT_HANDLED.  Handle this
status code same as STATUS_OBJECT_PATH_NOT_FOUND in
symlink_info::check to align behaviour to traversing
paths with other non-NTFS type symlinks.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: docs: revamp docs explaining symlinks
Corinna Vinschen [Fri, 3 Apr 2020 19:44:00 +0000 (21:44 +0200)]
Cygwin: docs: revamp docs explaining symlinks

The descriptions of symlink handling are a bit dated, so
revamp them and add the new WSL symlink type.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: symlinks: create WSL symlinks on supporting filesystems
Corinna Vinschen [Fri, 3 Apr 2020 19:40:01 +0000 (21:40 +0200)]
Cygwin: symlinks: create WSL symlinks on supporting filesystems

WSL symlinks are reparse points containing a POSIX path in UTF-8.
On filesystems supporting reparse points, use this symlink type.
On other filesystems, or in case of error, fall back to the good
old plain SYSTEM file.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: symlinks: fix WSL symlinks pointing to /mnt
Corinna Vinschen [Thu, 2 Apr 2020 20:25:55 +0000 (22:25 +0200)]
Cygwin: symlinks: fix WSL symlinks pointing to /mnt

Commit 4a36897af392 allowed to convert /mnt/<drive> path
prefixes to Cygwin cygdrive prefixes on the fly.  However,
the patch neglected WSL symlinks pointing to the /mnt
directory.  Rearrange path conversion so /mnt is converted
to the cygdrive prefix path itself.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agowinsup/cygwin: remove defines added in mingw-w64 v7.0.0
Biswapriyo Nath [Wed, 1 Apr 2020 05:50:00 +0000 (11:20 +0530)]
winsup/cygwin: remove defines added in mingw-w64 v7.0.0

Signed-off-by: Biswapriyo Nath <nathbappai@gmail.com>
4 years agoCygwin: symlinks: support WSL symlinks
Corinna Vinschen [Fri, 27 Mar 2020 11:12:31 +0000 (12:12 +0100)]
Cygwin: symlinks: support WSL symlinks

Treat WSL symlinks just like other symlinks.  Convert
absolute paths pointing to Windows drives via
/mnt/<driveletter> to Windows-style paths <driveletter>:

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: serial: fix GCC warning
Corinna Vinschen [Thu, 26 Mar 2020 13:13:37 +0000 (14:13 +0100)]
Cygwin: serial: fix GCC warning

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agonewlib/libm/math: Make pow/powf return qnan for snan arg
Keith Packard via Newlib [Thu, 26 Mar 2020 00:18:21 +0000 (17:18 -0700)]
newlib/libm/math: Make pow/powf return qnan for snan arg

The IEEE spec for pow only has special case for x**0 and 1**y when x/y
are quiet NaN. For signaling NaN, the general case applies and these functions
should signal the invalid exception and return a quiet NaN.

Signed-off-by: Keith Packard <keithp@keithp.com>
4 years agonewlib/libm/common: Don't re-convert float to bits in modf/modff
Keith Packard via Newlib [Thu, 26 Mar 2020 00:18:20 +0000 (17:18 -0700)]
newlib/libm/common: Don't re-convert float to bits in modf/modff

These functions shared a pattern of re-converting the argument to bits
when returning +/-0. Skip that as the initial conversion still has the
sign bit.

Signed-off-by: Keith Packard <keithp@keithp.com>
4 years agonewlib/libm/common: Fix modf/modff returning snan
Keith Packard via Newlib [Thu, 26 Mar 2020 00:18:19 +0000 (17:18 -0700)]
newlib/libm/common: Fix modf/modff returning snan

Recent GCC appears to elide multiplication by 1, which causes snan
parameters to be returned unchanged through *iptr. Use the existing
conversion of snan to qnan to also set the correct result in *iptr
instead.

Signed-off-by: Keith Packard <keithp@keithp.com>
4 years agoFix spurious underflow exceptions for Bessel functions for double(from glibc bug...
Joseph S. Myers [Wed, 25 Mar 2020 18:18:44 +0000 (11:18 -0700)]
Fix spurious underflow exceptions for Bessel functions for double(from glibc bug 14155)

This fix comes from glibc, from files which originated from
the same place as the newlib files. Those files in glibc carry
the same license as the newlib files.

Bug 14155 is spurious underflow exceptions from Bessel functions for
large arguments.  (The correct results for large x are roughly
constant * sin or cos (x + constant) / sqrt (x), so no underflow
exceptions should occur based on the final result.)

There are various places underflows may occur in the intermediate
calculations that cause the failures listed in that bug.  This patch
fixes problems for the double version where underflows occur in
calculating the intermediate functions P and Q (in particular, x**-12
gets computed while calculating Q).  Appropriate approximations are
used for P and Q for arguments at least 0x1p28 and above to avoid the
underflows.

For sufficiently large x - 0x1p129 and above - the code already has a
cut-off to avoid calculating P and Q at all, which means the
approximations -0.125 / x and 0.375 / x can't themselves cause
underflows calculating Q.  This cut-off is heuristically reasonable
for the point beyond which Q can be neglected (based on expecting
around 0x1p-64 to be the least absolute value of sin or cos for large
arguments representable in double).

The float versions use a cut-off 0x1p17, which is less heuristically
justifiable but should still only affect values near zeroes of the
Bessel functions where these implementations are intrinsically
inaccurate anyway (bugs 14469-14472), and should serve to avoid
underflows (the float underflow for jn in bug 14155 probably comes
from the recurrence to compute jn).  ldbl-96 uses 0x1p129, which may
not really be enough heuristically (0x1p143 or so might be safer - 143
= 64 + 79, number of mantissa bits plus total number of significant
bits in representation) but again should avoid underflows and only
affect values where the code is substantially inaccurate anyway.
ldbl-128 and ldbl-128ibm share a completely different implementation
with no such cut-off, which I propose to fix separately.

Signed-off-by: Keith Packard <keithp@keithp.com>
4 years agoRevert "Cygwin: serial: read: if VMIN > 0, wait for VMIN chars in inbound queue" topic/serial_io github/topic/serial_io
Corinna Vinschen [Wed, 25 Mar 2020 20:01:29 +0000 (21:01 +0100)]
Revert "Cygwin: serial: read: if VMIN > 0, wait for VMIN chars in inbound queue"

This reverts commit 082f2513c721e942d0fd563c4dc9117eee3513ab.

Turns out, Linux as well as BSD really only wait for the smaller
number, MIN or # of requested bytes.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agofhandler_serial: fix comments
Corinna Vinschen [Wed, 25 Mar 2020 11:25:06 +0000 (12:25 +0100)]
fhandler_serial: fix comments

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: serial: read: if VMIN > 0, wait for VMIN chars in inbound queue
Corinna Vinschen [Wed, 25 Mar 2020 11:21:59 +0000 (12:21 +0100)]
Cygwin: serial: read: if VMIN > 0, wait for VMIN chars in inbound queue

Per termios, read waits for MIN chars even if the number of requested
bytes is less.  This requires to add WaitCommEvent to wait non-busily
for MIN chars prior to calling ReadFile, so, reintroduce it.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: serial: use per call OVERLAPPED structs
Corinna Vinschen [Mon, 23 Mar 2020 20:06:00 +0000 (21:06 +0100)]
Cygwin: serial: use per call OVERLAPPED structs

Sharing the OVERLAPPED struct and event object in there between
read and select calls in the fhandler might have been a nice
optimization way back when, but it is a dangerous, not thread-safe
approach.  Fix this by creating per-fhandler, per-call OVERLAPPED
structs and event objects on demand.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: serial: select: call ClearCommError prior to calling WaitCommEvent
Corinna Vinschen [Mon, 23 Mar 2020 16:23:19 +0000 (17:23 +0100)]
Cygwin: serial: select: call ClearCommError prior to calling WaitCommEvent

This (hopefully) eliminates a race condition

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: serial: select: fix WaitCommEvent request
Corinna Vinschen [Mon, 23 Mar 2020 12:08:32 +0000 (13:08 +0100)]
Cygwin: serial: select: fix WaitCommEvent request

- make sure event object is reset
- set read_ready to true if WaitCommEvent returns success
- improve debugging

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: serial: tcsetattr: set timeouts unconditionally
Corinna Vinschen [Mon, 23 Mar 2020 11:13:35 +0000 (12:13 +0100)]
Cygwin: serial: tcsetattr: set timeouts unconditionally

tcsetattr checks if the VTIME and VMIN values changed and only
calls SetCommTimeouts if so.  That's a problem if tcsetattr
is supposed to set VTIME and VIMN to 0, because these are the
start values anyway.  But this requires to set ReadIntervalTimeout
to MAXDWORD, which just doesn't happen.

Fix this by dropping the over-optimization of checking the old
values before calling SetCommTimeouts,

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: serial: avoid overrun of vtime
Corinna Vinschen [Sat, 21 Mar 2020 09:36:11 +0000 (10:36 +0100)]
Cygwin: serial: avoid overrun of vtime

After changing the type of fhandler_serial::vtime_ to cc_t, vtime_
must be stored in 10s of seconds, not in milliseconds.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: serial: select: fix previous revamp patch
Ã…ke Rehnman [Fri, 20 Mar 2020 09:37:17 +0000 (10:37 +0100)]
Cygwin: serial: select: fix previous revamp patch

- We need a verify function.
- The event object referenced in WaitCommEvent must not be a local var,
  move it into fhandler_serial.

4 years agoCygwin: serial: revamp overlapped IO in read and select
Corinna Vinschen [Wed, 18 Mar 2020 20:14:06 +0000 (21:14 +0100)]
Cygwin: serial: revamp overlapped IO in read and select

Get rid of WaitCommEvent and using overlapped_armed to share the
same overlapped operation between read and select.  Rather, make
sure to cancel the overlapped IO before leaving any of these functions.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: serial: read: revamp raw_read, change vmin_ and vtime_ to cc_t
Corinna Vinschen [Tue, 17 Mar 2020 16:45:05 +0000 (17:45 +0100)]
Cygwin: serial: read: revamp raw_read, change vmin_ and vtime_ to cc_t

- Datatypes were incorrect, especially vmin_ and vtime_.
  Change them to cc_t, as in user space.

- Error checking had a gap or two.  Debug output used the
  wrong formatting.

- Don't use ev member for ClearCommError and WaitCommEvent.
  Both returned values are different (error value vs. event
  code).  The values are not used elsewhere so it doesn't make
  sense to store them in the object.  Therefore, drop ev member.

- Some variable names were not very helpful.  Especially using
  n as lpNumberOfBytesTransferred from GetOverlappedResult and
  then actually printing it as if it makes sense was quite
  puzzeling.

- Rework the loop and the definition of minchars so that it
  still makes sense when looping.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: serial: select: simplify peek_serial
Corinna Vinschen [Tue, 17 Mar 2020 16:24:56 +0000 (17:24 +0100)]
Cygwin: serial: select: simplify peek_serial

- Don't use ev member for ClearCommError and WaitCommEvent.
  Both returned values are different (error value vs. event
  code).  The values are not used elsewhere so it doesn't make
  sense to store them in the object.

- Drop local variable ready which is used inconsequentially.

- Since WFSO already waits 10 ms, don't wait again if no char
  is in the inbound queue.

- Avoid else if chains.

- Only print one line of debug output on error.

- Drop overlapped_armed < 0 check.  This value is only set in
  fhandler_serial::raw_read if VTIME > 0, and even then it's only
  set to be immediately reset to 0 before calling ReadFile.  So
  overlapped_armed is never actually < 0 when calling select.

- Fix a screwed up statement order.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: Use a separate Start Menu folder for WoW64 installs
Jon Turney [Thu, 19 Mar 2020 13:54:10 +0000 (13:54 +0000)]
Cygwin: Use a separate Start Menu folder for WoW64 installs

This aligns the shortcuts to documentation with the setup changes in
https://sourceware.org/pipermail/cygwin-apps/2020-March/039873.html

v2:
Create/remove the Start Menu directory as needed/possible
Correctly use that directory when making shortcuts

4 years agoFix hypotf missing mask in hi+lo decomposition
Fabian Schriever [Thu, 19 Mar 2020 15:34:08 +0000 (16:34 +0100)]
Fix hypotf missing mask in hi+lo decomposition

Add the missing mask for the decomposition of hi+lo which caused some
errors of 1-2 ULP.

This change is taken over from FreeBSD:
https://github.com/freebsd/freebsd/commit/95436ce20dab5a34ba46373410b96411b1734578

Additionally I've removed some variable assignments which were never
read before being overwritten again in the next 2 lines.

4 years agoFix modf/f for NaN input
Fabian Schriever [Wed, 18 Mar 2020 13:18:20 +0000 (14:18 +0100)]
Fix modf/f for NaN input

For NaN input the modf/f procedures should return NaN instead of zero
with the sign of the input.

4 years agoFix for k_tan.c specific inputs
Fabian Schriever [Tue, 17 Mar 2020 14:48:44 +0000 (15:48 +0100)]
Fix for k_tan.c specific inputs

This fix for k_tan.c is a copy from fdlibm version 5.3 (see also
http://www.netlib.org/fdlibm/readme), adjusted to use the macros
available in newlib (SET_LOW_WORD).

This fix reduces the ULP error of the value shown in the fdlibm readme
(tan(1.7765241907548024E+269)) to 0.45 (thereby reducing the error by
1).

This issue only happens for large numbers that get reduced by the range
reduction to a value smaller in magnitude than 2^-28, that is also
reduced an uneven number of times. This seems rather unlikely given that
one ULP is (much) larger than 2^-28 for the values that may cause an
issue.  Although given the sheer number of values a double can
represent, it is still possible that there are more affected values,
finding them however will be quite hard, if not impossible.

We also took a look at how another library (libm in FreeBSD) handles the
issue: In FreeBSD the complete if branch which checks for values smaller
than 2^-28 (or rather 2^-27, another change done by FreeBSD) is moved
out of the kernel function and into the external function. This means
that the value that gets checked for this condition is the unreduced
value. Therefore the input value which caused a problem in the
fdlibm/newlib kernel tan will run through the full polynomial, including
the careful calculation of -1/(x+r). So the difference is really whether
r or y is used. r = y + p with p being the result of the polynomial with
1/3*x^3 being the largest (and magnitude defining) value. With x being
<2^-27 we therefore know that p is smaller than y (y has to be at least
the size of the value of x last mantissa bit divided by 2, which is at
least x*2^-51 for doubles) by enough to warrant saying that r ~ y.  So
we can conclude that the general implementation of this special case is
the same, FreeBSD simply has a different philosophy on when to handle
especially small numbers.

4 years agoCygwin: serial: wait for CancelIo completion
Corinna Vinschen [Mon, 16 Mar 2020 09:20:16 +0000 (10:20 +0100)]
Cygwin: serial: wait for CancelIo completion

Per https://devblogs.microsoft.com/oldnewthing/20110202-00/?p=11613
GetOverlappedResult must be called blocking, waiting for the overlapped
operation to complete.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoRTEMS: Include missing header and fix stub
Sebastian Huber [Fri, 13 Mar 2020 17:21:36 +0000 (18:21 +0100)]
RTEMS: Include missing header and fix stub

Signed-off-by: Sebastian Huber <sebastian.huber@embedded-brains.de>
4 years agoCygwin: pty: Add FreeConsole to destructor of pty slave.
Takashi Yano via Cygwin-patches [Fri, 13 Mar 2020 03:06:49 +0000 (12:06 +0900)]
Cygwin: pty: Add FreeConsole to destructor of pty slave.

- When pseudo console is closed, all the processes attched to the
  pseudo console are terminated. This causes the problem reported
  in https://sourceware.org/pipermail/cygwin/2020-March/244046.html.
  This patch fixes the issue.

4 years agoCygwin: serial: try fix o_nonblock
Corinna Vinschen [Thu, 12 Mar 2020 15:07:01 +0000 (16:07 +0100)]
Cygwin: serial: try fix o_nonblock

4 years agoCygwin: belatedly add Hans-Bernhard to CONTRIBUTORS file
Corinna Vinschen [Wed, 11 Mar 2020 16:40:03 +0000 (17:40 +0100)]
Cygwin: belatedly add Hans-Bernhard to CONTRIBUTORS file

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: fix formatting: drop trailing whitespace
Corinna Vinschen [Wed, 11 Mar 2020 12:36:41 +0000 (13:36 +0100)]
Cygwin: fix formatting: drop trailing whitespace

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: fix formatting: collapse whitespace-only lines
Corinna Vinschen [Wed, 11 Mar 2020 12:28:27 +0000 (13:28 +0100)]
Cygwin: fix formatting: collapse whitespace-only lines

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: fix formatting: drop spaces leading tabs
Corinna Vinschen [Wed, 11 Mar 2020 12:23:55 +0000 (13:23 +0100)]
Cygwin: fix formatting: drop spaces leading tabs

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: fix formatting: replace TAB char with \t in string constant
Corinna Vinschen [Wed, 11 Mar 2020 12:44:05 +0000 (13:44 +0100)]
Cygwin: fix formatting: replace TAB char with \t in string constant

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoFix truncf for sNaN input
Fabian Schriever [Wed, 11 Mar 2020 09:58:05 +0000 (10:58 +0100)]
Fix truncf for sNaN input

Make line 47 in sf_trunc.c reachable. While converting the double
precision function trunc to the single precision version truncf an error
was introduced into the special case. This special case is meant to
catch both NaNs and infinities, however qNaNs and infinities work just
fine with the simple return of x (line 51). The only error occurs for
sNaNs where the same sNaN is returned and no invalid exception is
raised.

4 years agoi386/fenv.c: Include fenv.c implementation shared with x86_64, not stub
Joel Sherrill [Tue, 10 Mar 2020 15:03:44 +0000 (10:03 -0500)]
i386/fenv.c: Include fenv.c implementation shared with x86_64, not stub

4 years agoFix error in fdim/f for infinities
Fabian Schriever [Tue, 10 Mar 2020 10:24:12 +0000 (11:24 +0100)]
Fix error in fdim/f for infinities

The comparison c == FP_INFINITE causes the function to return +inf as it
expects x = +inf to always be larger than y. This shortcut causes
several issues as it also returns +inf for the following cases:
 - fdim(+inf, +inf), expected (as per C99): +0.0
 - fdim(-inf, any non NaN), expected: +0.0

I don't see a reason to keep the comparison as all the infinity cases
return the correct result using just the ternary operation.

4 years agoFix error in exp in magnitude [2e-32,2e-28]
Fabian Schriever [Fri, 6 Mar 2020 14:46:33 +0000 (15:46 +0100)]
Fix error in exp in magnitude [2e-32,2e-28]

While testing the exp function we noticed some errors at the specified
magnitude. Within this range the exp function returns the input value +1
as an output. We chose to run a test of 1m exponentially spaced values
in the ranges [-2^-27,-2^-32] and [2^-32,2^-27] which showed 7603 and
3912 results with an error of >=0.5 ULP (compared with MPFR in 128 bit)
with the highest being 0.56 ULP and 0.53 ULP.

It's easy to fix by changing the magnitude at which the input value +1
is returned from <2^-28 to <2^-32 and using the polynomial instead. This
reduces the number of results with an error of >=0.5 ULP to 485 and 479
in above tests, all of which are exactly 0.5 ULP.

As we were already checking on exp we also took a look at expf. For expf
the magnitude where the input value +1 is returned can be increased from
<2^-28 to <2^-23 without accuracy loss for a slight performance
improvement. To ensure this was the correct value we tested all values
in the ranges [-2^-17,-2^-28] and [2^-28,2^-17] (~92.3m values each).

4 years agoCygwin: console: Fix behaviour of "ESC 8" after reset.
Takashi Yano [Mon, 9 Mar 2020 01:38:36 +0000 (10:38 +0900)]
Cygwin: console: Fix behaviour of "ESC 8" after reset.

- This patch matches the behaviour of "ESC 8" (DECRC) to the real
  xterm after full reset (RIS), soft reset (DECSTR) and "CSI 3 J".

4 years agoDo not bother passing optional argument to WriteConsoleA.
Hans-Bernhard Broeker [Sun, 8 Mar 2020 20:41:14 +0000 (21:41 +0100)]
Do not bother passing optional argument to WriteConsoleA.

Passing a pointer to a local variable to WriteConsoleA is
not actually needed if we're not going to do anything with
what WriteConsoleA would put in there.

For the wpbuf class the pointer argument was made optional,
so it can be just left out; other call places now pass a
NULL pointer instead.  The local variables `wn' and `n'
are no unused, so they go away.

4 years agoCollect handling of wpixput and wpbuf into a helper class.
Hans-Bernhard Broeker [Sun, 8 Mar 2020 20:41:13 +0000 (21:41 +0100)]
Collect handling of wpixput and wpbuf into a helper class.

Replace direct access to a pair of co-dependent variables
by calls to methods of a class that encapsulates their relation.

Also replace C #define by C++ class constant.

4 years agoFix error in float trig. function range reduction
Fabian Schriever [Tue, 3 Mar 2020 13:49:06 +0000 (14:49 +0100)]
Fix error in float trig. function range reduction

The single-precision trigonometric functions show rather high errors in
specific ranges starting at about 30000 radians. For example the sinf
procedure produces an error of 7626.55 ULP with the input
5.195880078125e+04 (0x474AF6CD) (compared with MPFR in 128bit
precision). For the test we used 100k values evenly spaced in the range
of [30k, 70k]. The issues are periodic at higher ranges.

This error was introduced when the double precision range reduction was
first converted to float. The shift by 8 bits always returns 0 as iq is
never higher than 255.

The fix reduces the error of the example above to 0.45 ULP, highest
error within the test set fell to 1.31 ULP, which is not perfect, but
still a significant improvement. Testing other previously erroneous
ranges no longer show particularly large accuracy errors.

4 years agoCygwin: console: convert wpbuf_put to inline function
Corinna Vinschen [Mon, 2 Mar 2020 19:30:09 +0000 (20:30 +0100)]
Cygwin: console: convert wpbuf_put to inline function

fix potential buffer overrun while at it

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: console: Add a workaround for "ESC 7" and "ESC 8".
Takashi Yano [Mon, 2 Mar 2020 01:12:57 +0000 (10:12 +0900)]
Cygwin: console: Add a workaround for "ESC 7" and "ESC 8".

- In xterm compatible mode, "ESC 7" and "ESC 8" do not work properly
  in the senario:
   1) Execute /bin/ls /bin to fill screen.
   2) Sned CSI?1049h to alternate screen.
   3) Reduce window size.
   4) Send CSI?1049l to resume screen.
   5) Send "ESC 7" and "ESC 8".
  After sending "ESC 8", the cursor goes to incorrect position. This
  patch adds a workaround for this issue.

4 years agoCygwin: console: Prevent buffer overrun.
Takashi Yano [Mon, 2 Mar 2020 01:12:56 +0000 (10:12 +0900)]
Cygwin: console: Prevent buffer overrun.

- This patch prevent potential buffer overrun in the code handling
  escape sequences.

4 years agoCygwin: console: Fix setting/unsetting xterm mode for input.
Takashi Yano [Mon, 2 Mar 2020 01:12:55 +0000 (10:12 +0900)]
Cygwin: console: Fix setting/unsetting xterm mode for input.

- This patch fixes the issue that xterm compatible mode for input
  is not correctly set/unset in some situation such as:
   1) cat is stopped by ctrl-c.
   2) The window size is changed in less.
  In case 1), request_xterm_mode_input(true) is called in read(),
  however, cat is stopped without request_xterm_mode_input(false).
  In case 2), less uses longjmp in signal handler, therefore,
  corresponding request_xterm_mode_input(false) is not called if
  the SIGWINCH signal is sent within read(). With this patch,
  InterlockedExchange() is used instead of InterlockedIncrement/
  Decrement().

4 years agoCygwin: console: Revise the code to fix tab position.
Takashi Yano [Mon, 2 Mar 2020 01:12:54 +0000 (10:12 +0900)]
Cygwin: console: Revise the code to fix tab position.

- This patch fixes the issue that the cursor position is broken if
  window size is changed while executing vim, less etc.

4 years agoCygwin: ioctl: TIOCINQ: always return number of chars in the inbound queue
Corinna Vinschen [Mon, 2 Mar 2020 16:02:51 +0000 (17:02 +0100)]
Cygwin: ioctl: TIOCINQ: always return number of chars in the inbound queue

So far ioctl(TIOCINQ) could end up returning -1 with errno set to EINVAL
if a non-zero device error mask has been returned by ClearCommError.
This doesn't reflect Linux behaviour, which always returns the number of
chars in the inbound queue, independent of any I/O error condition.
EINVAL was a pretty weird error code to use in this scenario, too.

Fix this by dropping all checking for device errors in the TIOCINQ
case.  Just return the number of chars in the inbound queue.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoFix error in powf for (-1.0, NaN) input
Fabian Schriever [Mon, 2 Mar 2020 14:40:23 +0000 (15:40 +0100)]
Fix error in powf for (-1.0, NaN) input

Prevent confusion between -1.0 and 1.0 in powf. The corresponding
similar error was previously fixed for pow (see commit bb25dd1b)

4 years agoarm: Finish moving newlib to unified syntax for Thumb1
Richard Earnshaw [Mon, 2 Mar 2020 13:33:11 +0000 (13:33 +0000)]
arm: Finish moving newlib to unified syntax for Thumb1

Most code in newlib already uses unified syntax, but just a couple of
laggards remain.  This patch removes these and means the the entire
code base has now been converted.

4 years agoCygwin: console: Adjust the detailed behaviour of ESC sequences.
Takashi Yano [Thu, 27 Feb 2020 02:33:50 +0000 (11:33 +0900)]
Cygwin: console: Adjust the detailed behaviour of ESC sequences.

- This patch makes some detailed behaviour of ESC sequences such as
  "CSI Ps L" (IL), "CSI Ps M" (DL) and "ESC M" (RI) in xterm mode
  match with real xterm.

4 years agoCygwin: AF_UNIX: rework fixup_after_exec
Corinna Vinschen [Fri, 28 Feb 2020 12:21:05 +0000 (13:21 +0100)]
Cygwin: AF_UNIX: rework fixup_after_exec

fhandler_socket_unix::fixup_after_exec incorrectly calls
fhandler_socket_unix::fixup_after_fork with a NULL parent process
handle.  Not only that calling DuplicateHandle with a NULL parent
handle fails, but it's utterly wrong trying to duplicate the handles
at all here.

Rather just set some important values to NULL and reopen the shared
memory region.  Create a fixup_helper method to call common code from
fixup_after_fork and fixup_after_exec.

Add comments to other invocations of fixup_after_fork with NULL
handle to mark them as correct this way.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: 32 bit: remove old code to 16 bit align stack
Corinna Vinschen [Fri, 28 Feb 2020 13:31:56 +0000 (14:31 +0100)]
Cygwin: 32 bit: remove old code to 16 bit align stack

Aligning the stack pointer using an asm statement isn't any longer
supported.  gcc-9.2.0 generates the following warning:

  init.cc:33:46: error: listing the stack pointer register '%esp'
  in a clobber list is deprecated [-Werror=deprecated]
  [...]
  init.cc:33:46: note: the value of the stack pointer after an
  'asm' statement must be the same as it was before the statement

Replace the asm expression with the gcc function attribute
`force_align_arg_pointer'.  This aligns the stack exactly as
required.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: AF_UNIX: use Nt functions within Nt functions
Corinna Vinschen [Fri, 28 Feb 2020 11:40:49 +0000 (12:40 +0100)]
Cygwin: AF_UNIX: use Nt functions within Nt functions

Functionaly equivalent, but makes for cleaner code

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: AF_UNIX: fix creating shared mem region in dup
Corinna Vinschen [Fri, 28 Feb 2020 11:39:41 +0000 (12:39 +0100)]
Cygwin: AF_UNIX: fix creating shared mem region in dup

reopen_shmem is accidentally called on the parent fhandler
rather than the child fhandler, and it's called too early.
Make sure to call it on the child and only after its shmem_handle
is valid.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: Update dumper for bfd API changes
Jon Turney [Wed, 26 Feb 2020 18:48:51 +0000 (18:48 +0000)]
Cygwin: Update dumper for bfd API changes

Update dumper for bfd API changes in binutils 2.34

libbfd doesn't guarantee API stability, so we've just been lucky this
hasn't broken more often.

See binutils commit fd361982.

4 years agoCygwin: console: Add emulation of CSI3J on Win10 1809.
Takashi Yano [Wed, 26 Feb 2020 15:33:02 +0000 (00:33 +0900)]
Cygwin: console: Add emulation of CSI3J on Win10 1809.

- This patch add emulation of CSI3J, which is broken in Win10 1809,
  rather than ignoring it as before.

4 years agoCygwin: console: Add support for REP escape sequence to xterm mode.
Takashi Yano [Wed, 26 Feb 2020 15:33:01 +0000 (00:33 +0900)]
Cygwin: console: Add support for REP escape sequence to xterm mode.

- In Win10 upto 1809, xterm compatible mode does not have REP
  escape sequence which terminfo declares. This patch adds support
  for "CSI Ps b" (REP). With this patch, bvi (binary editor) works
  normally in Win10 1809. Also, xterm compatible mode does not have
  "CSI Pm `" (HPA), "CSI Pm a" (HPR) and "CSI Ps e" (VPR). However,
  they do not appear to be declared by terminfo. Therefore, these
  have been pending.

4 years agoCygwin: console: Unify workaround code for CSI3J and CSI?1049h/l.
Takashi Yano [Wed, 26 Feb 2020 15:33:00 +0000 (00:33 +0900)]
Cygwin: console: Unify workaround code for CSI3J and CSI?1049h/l.

- This patch unifies workaround code for CSI3J and CSI?1049h/l into
  the code handling other escape sequences in xterm mode.

4 years agoCygwin: console: Add workaround for broken IL/DL in xterm mode.
Takashi Yano [Wed, 26 Feb 2020 15:32:59 +0000 (00:32 +0900)]
Cygwin: console: Add workaround for broken IL/DL in xterm mode.

- Cygwin console with xterm compatible mode causes problem reported
  in https://www.cygwin.com/ml/cygwin-patches/2020-q1/msg00212.html
  if background/foreground colors are set to gray/black respectively
  in Win10 1903/1909. This is caused by "CSI Ps L" (IL), "CSI Ps M"
  (DL) and "ESC M" (RI) control sequences which are broken. This
  patch adds a workaround for the issue.

4 years agoCygwin: ps: fix compiler warning in ttynam
Corinna Vinschen [Wed, 26 Feb 2020 20:08:51 +0000 (21:08 +0100)]
Cygwin: ps: fix compiler warning in ttynam

The helper function ttynam creates a tty name by using sprintf wrongly
on a pretty short buffer.  The foramt string only specifies a minimum
field length, not a maximum field length, so gcc-9.2.0 complains:

  ps.cc:101:23: warning: 'sprintf' may write a terminating nul past the
  end of the destination [-Wformat-overflow=]

Fix this thoroughly by specifying a maximum field width as well as by
using snprintf with a fixed buffer length.  Also, drop using a static
buffer in favor of using a buffer in the caller.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: cygserver: drop useless packed attribute
Corinna Vinschen [Wed, 26 Feb 2020 19:52:55 +0000 (20:52 +0100)]
Cygwin: cygserver: drop useless packed attribute

...from structs used for data exchange between clients and cygserver.
All of the structs have the same size and member offsets, packed or
unpacked.  Keeping the packed attribute results in ominous warnings
from gcc-9.2.0:

  cygserver.cc:259:10: warning: taking address of packed member of
  'client_request_attach_tty::request_attach_tty' may result in an
  unaligned pointer value [-Waddress-of-packed-member]

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: Makefile.in: add -fno-builtin-execve CFLAG when building exec.o
Corinna Vinschen [Wed, 26 Feb 2020 16:02:01 +0000 (17:02 +0100)]
Cygwin: Makefile.in: add -fno-builtin-execve CFLAG when building exec.o

gcc-9.2.0 has an execve builtin which uses the nothrow attribute.
This results in an error when aliasing execve to _execve for newlib:

exec.cc:88:23: error: 'int _execve(const char*, char* const*, char*
const*)' specifies less restrictive attribute than its target
'int execve(const char*, char* const*, char* const*)': 'nothrow'
[-Werror=missing-attributes]
   88 | EXPORT_ALIAS (execve, _execve) /* For newlib */

Add the -fno-builtin-execve CFLAGS when building exec.o to override
the gcc builtin.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: posix timers: fix uninitialized variable
Corinna Vinschen [Wed, 26 Feb 2020 15:50:34 +0000 (16:50 +0100)]
Cygwin: posix timers: fix uninitialized variable

The variable returning the overrun count from the tracker object after
disarming the overrun counter was not correctly initialized.  For some
reason this has only been noticed by gcc-9.2.0, not by the formerly used
gcc-7.4.0.

This problem should not have had any runtime impact.  The method
timer_tracker::disarm_overrun_event is supposed to be called in
lock-step with timer_tracker::arm_overrun_event, which in turn
results in the variable getting a valid value.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agocpuinfo:power management: add proc_feedback, acc_power
Brian Inglis [Tue, 25 Feb 2020 23:44:16 +0000 (16:44 -0700)]
cpuinfo:power management: add proc_feedback, acc_power

linux 4.6 x86/cpu: Add advanced power management bits
Bit 11 of CPUID 8000_0007 edx is processor feedback interface.
Bit 12 of CPUID 8000_0007 edx is accumulated power.

Print proper names in /proc/cpuinfo

[missed enabling this 2016 change during previous major cpuinfo update
as no power related changes were made to the Linux files since then]

4 years agox86_64/i386 fenv: Replace symlink with include fenv_stub.c
Joel Sherrill [Tue, 25 Feb 2020 13:38:33 +0000 (07:38 -0600)]
x86_64/i386 fenv: Replace symlink with include fenv_stub.c

Having symlinks for these files led to an issue reported to the RTEMS
Project that showed up using some tar for native Windows to unpack the
newlib sources.  It creates symlinks in the tar file as copies of the
files the symlinks point to.  If the links appear in the tar file before
the source exists, it cannot copy the file.

The solution in this patch is to convert the files that are symbolic
links into simple files which include the file they were linked to.
This should be more portable and avoids the symbolinc link problem.

4 years agoCygwin: console: Fix segfault on shared_console_info access.
Takashi Yano [Mon, 24 Feb 2020 16:12:16 +0000 (01:12 +0900)]
Cygwin: console: Fix segfault on shared_console_info access.

- Accessing shared_console_info before initialization causes access
  violation because it is a NULL pointer. The cause of the problem
  reported in https://cygwin.com/ml/cygwin/2020-02/msg00197.html is
  this NULL pointer access in request_xterm_mode_output() when it is
  called from close(). This patch makes sure that shared_console_info
  is not NULL before calling request_xterm_mode_output().

4 years agofhandler_proc/cpuinfo: support fast short REP MOVSB
Brian Inglis [Fri, 21 Feb 2020 21:28:08 +0000 (14:28 -0700)]
fhandler_proc/cpuinfo: support fast short REP MOVSB

Added in Linux 5.6:
Check FSRM and use REP MOVSB for short copies on systems that have it.

>From the Intel Optimization Reference Manual:

3.7.6.1 Fast Short REP MOVSB
Beginning with processors based on Ice Lake Client microarchitecture,
REP MOVSB performance is enhanced with string lengths up to 128 bytes.
Support for fast-short REP MOVSB is indicated by the CPUID feature flag:
CPUID [EAX=7H, ECX=0H).EDX.FAST_SHORT_REP_MOVSB[bit 4] = 1.
There is no change in the REP STOS performance.

4 years agoCygwin: add release message for NUL character bugs
Corinna Vinschen [Fri, 21 Feb 2020 09:57:10 +0000 (10:57 +0100)]
Cygwin: add release message for NUL character bugs

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: don't move cursor on NUL char at all
Corinna Vinschen [Fri, 21 Feb 2020 09:26:25 +0000 (10:26 +0100)]
Cygwin: don't move cursor on NUL char at all

Add a comment instead to explain that this behaviour contradicts
the terminfo entry.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: fhandler_console.cc: fix minor style issues
Corinna Vinschen [Thu, 20 Feb 2020 13:57:26 +0000 (14:57 +0100)]
Cygwin: fhandler_console.cc: fix minor style issues

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: console: ignore NUL byte on write in xterm emulation mode as well
Corinna Vinschen [Thu, 20 Feb 2020 13:48:03 +0000 (14:48 +0100)]
Cygwin: console: ignore NUL byte on write in xterm emulation mode as well

A NUL byte in the output stream got accidentally not handled as IGN char
in xterm console mode.  The internal mbtowc conversion doesn't handle
embedded NUL values gracefully, it always stops converting at NUL bytes.
This broke the output of strings with embedded NUL bytes.

Fix this by always skipping IGN chars in the "normal char output loop"
and make sure not to move the cursor one position to the right, as in
legacy console mode.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agonewlib/libc/include/devctl.h: Add extern "C" wrapper
Joel Sherrill [Wed, 19 Feb 2020 14:03:40 +0000 (08:03 -0600)]
newlib/libc/include/devctl.h: Add extern "C" wrapper

Adding this was necessary to allow posix_devctl() from C++.

4 years agoBump DLL version to 3.1.5
Corinna Vinschen [Wed, 19 Feb 2020 12:38:42 +0000 (13:38 +0100)]
Bump DLL version to 3.1.5

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: console: Add guard for set/unset xterm compatible mode. cygwin-3_1_4-release
Takashi Yano [Tue, 18 Feb 2020 09:12:54 +0000 (18:12 +0900)]
Cygwin: console: Add guard for set/unset xterm compatible mode.

- Setting / unsetting xterm compatible mode may cause race issue
  between multiple processes. This patch adds guard for that.

4 years agoCygwin: console: Fix ioctl() FIONREAD.
Takashi Yano [Tue, 18 Feb 2020 04:05:07 +0000 (13:05 +0900)]
Cygwin: console: Fix ioctl() FIONREAD.

- ioctl() FIONREAD for console does not return correct value since
  commit cfb517f39a8bcf2d995a732d250563917600408a. This patch fixes
  the issue.

4 years agoCygwin: add @cjksingle to release docs
Corinna Vinschen [Tue, 18 Feb 2020 10:32:47 +0000 (11:32 +0100)]
Cygwin: add @cjksingle to release docs

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
4 years agoCygwin: describe new locale modifier @cjksingle for user guide
Thomas Wolff [Sun, 16 Feb 2020 23:00:00 +0000 (00:00 +0100)]
Cygwin: describe new locale modifier @cjksingle for user guide

4 years agoLocale modifier "@cjksingle" to enforce single-width CJK width.
Thomas Wolff [Sun, 16 Feb 2020 23:00:00 +0000 (00:00 +0100)]
Locale modifier "@cjksingle" to enforce single-width CJK width.

This option follows a proposal in the Terminals Working Group Specifications
(https://gitlab.freedesktop.org/terminal-wg/specifications/issues/9#note_406682).
It makes locale width consistent with the corresponding mintty feature.

This page took 0.0735170000000001 seconds and 5 git commands to generate.