summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Cygwin: fix formatting: drop trailing whitespaceCorinna Vinschen2020-03-1199-117/+117
| | | | Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
* Cygwin: fix formatting: collapse whitespace-only linesCorinna Vinschen2020-03-1122-26/+26
| | | | Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
* Cygwin: fix formatting: drop spaces leading tabsCorinna Vinschen2020-03-1147-137/+137
| | | | Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
* Cygwin: fix formatting: replace TAB char with \t in string constantCorinna Vinschen2020-03-111-1/+1
| | | | Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
* Fix truncf for sNaN inputFabian Schriever2020-03-111-1/+1
| | | | | | | | | | 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.
* i386/fenv.c: Include fenv.c implementation shared with x86_64, not stubJoel Sherrill2020-03-101-1/+1
|
* Fix error in fdim/f for infinitiesFabian Schriever2020-03-102-8/+2
| | | | | | | | | | | 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.
* Fix error in exp in magnitude [2e-32,2e-28]Fabian Schriever2020-03-092-2/+2
| | | | | | | | | | | | | | | | | | | | 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).
* Cygwin: console: Fix behaviour of "ESC 8" after reset.Takashi Yano2020-03-091-0/+7
| | | | | - This patch matches the behaviour of "ESC 8" (DECRC) to the real xterm after full reset (RIS), soft reset (DECSTR) and "CSI 3 J".
* Do not bother passing optional argument to WriteConsoleA.Hans-Bernhard Broeker2020-03-091-28/+23
| | | | | | | | | | | 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.
* Collect handling of wpixput and wpbuf into a helper class.Hans-Bernhard Broeker2020-03-091-65/+76
| | | | | | | 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.
* Fix error in float trig. function range reductionFabian Schriever2020-03-031-1/+1
| | | | | | | | | | | | | | | | | | 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.
* Cygwin: console: convert wpbuf_put to inline functionCorinna Vinschen2020-03-021-4/+7
| | | | | | fix potential buffer overrun while at it Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
* Cygwin: console: Add a workaround for "ESC 7" and "ESC 8".Takashi Yano2020-03-022-13/+41
| | | | | | | | | | | | - 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.
* Cygwin: console: Prevent buffer overrun.Takashi Yano2020-03-021-10/+8
| | | | | - This patch prevent potential buffer overrun in the code handling escape sequences.
* Cygwin: console: Fix setting/unsetting xterm mode for input.Takashi Yano2020-03-021-2/+3
| | | | | | | | | | | | | | - 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().
* Cygwin: console: Revise the code to fix tab position.Takashi Yano2020-03-021-12/+3
| | | | | - This patch fixes the issue that the cursor position is broken if window size is changed while executing vim, less etc.
* Cygwin: ioctl: TIOCINQ: always return number of chars in the inbound queueCorinna Vinschen2020-03-022-8/+4
| | | | | | | | | | | | | 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>
* Fix error in powf for (-1.0, NaN) inputFabian Schriever2020-03-021-1/+1
| | | | | Prevent confusion between -1.0 and 1.0 in powf. The corresponding similar error was previously fixed for pow (see commit bb25dd1b)
* arm: Finish moving newlib to unified syntax for Thumb1Richard Earnshaw2020-03-022-7/+19
| | | | | | 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.
* Cygwin: console: Adjust the detailed behaviour of ESC sequences.Takashi Yano2020-02-282-7/+45
| | | | | | - 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.
* Cygwin: AF_UNIX: rework fixup_after_execCorinna Vinschen2020-02-284-10/+16
| | | | | | | | | | | | | | | | | 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>
* Cygwin: 32 bit: remove old code to 16 bit align stackCorinna Vinschen2020-02-282-21/+6
| | | | | | | | | | | | | | | | | 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>
* Cygwin: AF_UNIX: use Nt functions within Nt functionsCorinna Vinschen2020-02-281-3/+3
| | | | | | Functionaly equivalent, but makes for cleaner code Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
* Cygwin: AF_UNIX: fix creating shared mem region in dupCorinna Vinschen2020-02-281-6/+6
| | | | | | | | | 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>
* Cygwin: Update dumper for bfd API changesJon Turney2020-02-272-8/+26
| | | | | | | | | 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.
* Cygwin: console: Add emulation of CSI3J on Win10 1809.Takashi Yano2020-02-261-2/+17
| | | | | - This patch add emulation of CSI3J, which is broken in Win10 1809, rather than ignoring it as before.
* Cygwin: console: Add support for REP escape sequence to xterm mode.Takashi Yano2020-02-263-0/+45
| | | | | | | | | | - 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.
* Cygwin: console: Unify workaround code for CSI3J and CSI?1049h/l.Takashi Yano2020-02-261-21/+22
| | | | | - This patch unifies workaround code for CSI3J and CSI?1049h/l into the code handling other escape sequences in xterm mode.
* Cygwin: console: Add workaround for broken IL/DL in xterm mode.Takashi Yano2020-02-263-2/+166
| | | | | | | | | - 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.
* Cygwin: ps: fix compiler warning in ttynamCorinna Vinschen2020-02-261-9/+10
| | | | | | | | | | | | | | | 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>
* Cygwin: cygserver: drop useless packed attributeCorinna Vinschen2020-02-261-9/+3
| | | | | | | | | | | | | ...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>
* Cygwin: Makefile.in: add -fno-builtin-execve CFLAG when building exec.oCorinna Vinschen2020-02-261-0/+2
| | | | | | | | | | | | | | | | 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>
* Cygwin: posix timers: fix uninitialized variableCorinna Vinschen2020-02-261-1/+1
| | | | | | | | | | | | | | 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>
* cpuinfo:power management: add proc_feedback, acc_powerBrian Inglis2020-02-261-2/+2
| | | | | | | | | | | 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]
* x86_64/i386 fenv: Replace symlink with include fenv_stub.cJoel Sherrill2020-02-2523-23/+161
| | | | | | | | | | | | 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.
* Cygwin: console: Fix segfault on shared_console_info access.Takashi Yano2020-02-242-12/+15
| | | | | | | | | - 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().
* fhandler_proc/cpuinfo: support fast short REP MOVSBBrian Inglis2020-02-241-0/+1
| | | | | | | | | | | | | | 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.
* Cygwin: add release message for NUL character bugsCorinna Vinschen2020-02-211-0/+7
| | | | Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
* Cygwin: don't move cursor on NUL char at allCorinna Vinschen2020-02-211-2/+3
| | | | | | | Add a comment instead to explain that this behaviour contradicts the terminfo entry. Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
* Cygwin: fhandler_console.cc: fix minor style issuesCorinna Vinschen2020-02-201-3/+3
| | | | Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
* Cygwin: console: ignore NUL byte on write in xterm emulation mode as wellCorinna Vinschen2020-02-201-1/+3
| | | | | | | | | | | | | 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>
* newlib/libc/include/devctl.h: Add extern "C" wrapperJoel Sherrill2020-02-201-0/+8
| | | | Adding this was necessary to allow posix_devctl() from C++.
* Bump DLL version to 3.1.5Corinna Vinschen2020-02-191-1/+1
| | | | Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
* Cygwin: console: Add guard for set/unset xterm compatible mode.cygwin-3_1_4-releaseTakashi Yano2020-02-184-58/+103
| | | | | - Setting / unsetting xterm compatible mode may cause race issue between multiple processes. This patch adds guard for that.
* Cygwin: console: Fix ioctl() FIONREAD.Takashi Yano2020-02-181-4/+33
| | | | | | - ioctl() FIONREAD for console does not return correct value since commit cfb517f39a8bcf2d995a732d250563917600408a. This patch fixes the issue.
* Cygwin: add @cjksingle to release docsCorinna Vinschen2020-02-182-0/+12
| | | | Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
* Cygwin: describe new locale modifier @cjksingle for user guideThomas Wolff2020-02-181-0/+11
|
* Locale modifier "@cjksingle" to enforce single-width CJK width.Thomas Wolff2020-02-182-4/+17
| | | | | | 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.
* Cygwin: rename NSIG to _NSIG, change visibility of NSIG to MISCCorinna Vinschen2020-02-187-29/+37
| | | | | | | | | | | NSIG is a deprecated symbol only visible under MISC visibility. _NSIG is used widely instead, and on most systems NSIG is defined in terms of _NSIG. Follow suit: Change NSIG to _NSIG throughout and change visiblity of NSIG to be defined only in __MISC_VISIBLE case. Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
generated by cgit v1.2.3 (git 2.25.1) at 2025-07-16 19:32:38 +0000