]> sourceware.org Git - newlib-cygwin.git/log
newlib-cygwin.git
3 years agoCygwin: console: Fix SIGWINCH handling in Win7.
Takashi Yano via Cygwin-patches [Thu, 18 Feb 2021 09:01:27 +0000 (18:01 +0900)]
Cygwin: console: Fix SIGWINCH handling in Win7.

- If ENABLE_VIRTUAL_TERMINAL_INPUT is not set, changing window height
  does not generate WINDOW_BUFFER_SIZE_EVENT. This happens if console
  is in the legacy mode. Therefore, with this patch, the windows size
  is checked every time in cons_master_thread() if the cosole is in
  the legacy mode.

3 years agocpuinfo: add AVX features; move SME, SEV/_ES features
Brian Inglis [Wed, 17 Feb 2021 16:28:36 +0000 (09:28 -0700)]
cpuinfo: add AVX features; move SME, SEV/_ES features

Linux 5.11 ðŸ’• Valentine's Day Edition ðŸ’• added features and changes:
add Intel 0x00000007 EDX:23 avx512_fp16 and 0x00000007:1 EAX:4 avx_vnni;
group scattered AMD 0x8000001f EAX Secure Mem/Encrypted Virt features at end:
0 sme, 1 sev, 3 sev_es (more to come not yet displayed)

3 years agocpuinfo: fix check for cpuid 0x80000007 support
Brian Inglis [Wed, 17 Feb 2021 16:28:35 +0000 (09:28 -0700)]
cpuinfo: fix check for cpuid 0x80000007 support

3 years agoInclude malloc.h in libc/stdlib/aligned_alloc.c
Hans-Peter Nilsson [Tue, 16 Feb 2021 16:42:01 +0000 (17:42 +0100)]
Include malloc.h in libc/stdlib/aligned_alloc.c

Without this, for a bare-iron/simulator target such as cris-elf,
you'll see, at newlib build time:

/x/gccobj/./gcc/xgcc -B/x/gccobj/./gcc/ <many options elided> -c -o lib_a-aligned_alloc.o \
 `test -f 'aligned_alloc.c' || echo '/y/newlib/libc/stdlib/'`aligned_alloc.c
/y/newlib/libc/stdlib/aligned_alloc.c: In function 'aligned_alloc':
/y/newlib/libc/stdlib/aligned_alloc.c:35:10: warning: implicit declaration of function \
 '_memalign_r' [-Wimplicit-function-declaration]
   35 |   return _memalign_r (_REENT, align, size);
      |          ^~~~~~~~~~~

3 years agoCygwin: console: Introduce new thread which handles input signal.
Takashi Yano via Cygwin-patches [Tue, 16 Feb 2021 11:37:05 +0000 (20:37 +0900)]
Cygwin: console: Introduce new thread which handles input signal.

- Currently, Ctrl-Z, Ctrl-\ and SIGWINCH does not work in console
  if the process does not call read() or select(). This is because
  these are processed in process_input_message() which is called
  from read() or select(). This is a long standing issue of console.
  Addresses:
    https://cygwin.com/pipermail/cygwin/2020-May/244898.html
    https://cygwin.com/pipermail/cygwin/2021-February/247779.html

  With this patch, new thread which handles only input signals is
  introduced so that Crtl-Z, etc. work without calling read() or
  select(). Ctrl-S and Ctrl-Q are also handled in this thread.

3 years agoComplete revert of 2019-08-19, st_atime in libc/include/sys/stat.h
Hans-Peter Nilsson [Mon, 15 Feb 2021 20:20:35 +0000 (21:20 +0100)]
Complete revert of 2019-08-19, st_atime in libc/include/sys/stat.h

The revert-part of the revert-and-fix commit, b99887c4283f a.k.a.
"Revert previous change to sys/stat.h and fix cris libgloss",
apparently intending to revert f75aa6785151 a.k.a. "Fix regression in
cris-elf caused by sys/stat.h change" and fix it in another way,
wasn't complete.  Although the fix-part added the prerequisite "#undef
st_atime" (et al) to gensyscalls, the revert-part didn't revert the
"&& !defined(__cris__)" in sys/stat.h, stopping st_atime (et al) from
being defined.

The effect of the unreverted change is that accessing the struct stat
compatibility member names "st_atime" (et al) as in "struct stat
mystat; mystat.st_atime;" yields errors, observable for example when
building libgfortran in gcc:

/x/gcc/libgfortran/intrinsics/stat.c:114:42: error: 'struct stat' has \
no member named 'st_atime'; did you mean 'st_atim'?
  114 |       sarray->base_addr[8 * stride] = sb.st_atime;
      |                                          ^~~~~~~~
      |                                          st_atim
(etc.)

Trivially fixed by completing the reversion, removing the "&&
!defined(__cris__)" in sys/stat.h.

Beware: the net effect of the earlier related change to struct stat in
sys/stat.h, leading up to the fix, *does* change its definition as a
type.  Thankfully, replacing members like "time_t st_atime; long
st_spare1;" by "struct timespec st_atim;", ditto st_mtim and st_ctim,
is layout-compatible.  To wit, that change is "binary compatible".

Incidentally, related to the simulator / Linux ABI, there's a
transitional stage (see gensyscalls), reloading between "struct stat"
(sys/stat.h) and "struct new_stat" (kernel/simulator) as necessary.

Tested by a cris-elf gcc build (including libgfortran).

3 years agowinsup/doc/posix.xml: add note for getrlimit, setrlimit, xrefs to notes
Brian Inglis [Mon, 15 Feb 2021 22:35:39 +0000 (15:35 -0700)]
winsup/doc/posix.xml: add note for getrlimit, setrlimit, xrefs to notes

change notes to see "Implementation Notes" to xref to std-notes;
add xref to std-notes to getrlimit, setrlimit;
add note to document limitations of getrlimit, setrlimit resources support

3 years agoCygwin: console: Abort read() on signal if SA_RESTART is not set.
Takashi Yano via Cygwin-patches [Sun, 14 Feb 2021 09:42:50 +0000 (18:42 +0900)]
Cygwin: console: Abort read() on signal if SA_RESTART is not set.

- Currently, console read() keeps reading after SIGWINCH is sent
  even if SA_RESTART flag is not set. With this patch, read()
  returns EINTR on SIGWINCH if SA_RESTART flag is not set.
  The same problem for SIGQUIT and SIGTSTP has also been fixed.

3 years agoCygwin: pty: Fix a bug in input transfer for GDB.
Takashi Yano via Cygwin-patches [Sun, 14 Feb 2021 18:47:51 +0000 (03:47 +0900)]
Cygwin: pty: Fix a bug in input transfer for GDB.

- With this patch, not only NL but also CR is treated as a line end
  in the code checking if input transfer is necessary.

3 years agoCygwin: pty: Reduce unecessary input transfer.
Takashi Yano via Cygwin-patches [Thu, 11 Feb 2021 09:09:42 +0000 (18:09 +0900)]
Cygwin: pty: Reduce unecessary input transfer.

- Currently, input transfer is performed every time one line is read(),
  if the non-cygwin app is running in the background. With this patch,
  transfer is triggered by setpgid() rather than read() so that the
  unnecessary input transfer can be reduced much in that situation.

3 years agoCygwin: only export tmpfile64 on 32 bit
Corinna Vinschen [Fri, 12 Feb 2021 09:24:54 +0000 (10:24 +0100)]
Cygwin: only export tmpfile64 on 32 bit

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
3 years agoCygwin: Have tmpfile(3) use O_TMPFILE
Mark Geisert [Thu, 11 Feb 2021 06:53:05 +0000 (22:53 -0800)]
Cygwin: Have tmpfile(3) use O_TMPFILE

Per discussion on cygwin-developers, a Cygwin tmpfile(3) implementation
has been added to syscalls.cc.  This overrides the one supplied by
newlib.  Then the open(2) flag O_TMPFILE was added to the open call that
tmpfile internally makes.

This v2 patch removes O_CREAT from open() call as O_TMPFILE obviates it.
Note that open() takes a directory's path but returns an fd to a file.

3 years agoCygwin: fhandler: clean up 'copyto' logic
Corinna Vinschen [Wed, 10 Feb 2021 09:42:05 +0000 (10:42 +0100)]
Cygwin: fhandler: clean up 'copyto' logic

Analyzing the fhandler::copyto logic shows that the fhandler_base::reset
method was only called from copyto anyway.

Trying to convert reset to a protected method uncovered that the copyto
method is actually thought upside down from an object oriented POV.

Rather than calling copyto, manipulating the object given as parameter,
rename the method to copy_from, which manipulates the calling object
itself with data from the object given as parameter.

Eventually make reset a protected method and rename it to
_copy_from_reset_helper to clarify it's only called from copy_from.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
3 years agoCygwin: check path_conv_handle for NULL before trying to dup it
Corinna Vinschen [Wed, 10 Feb 2021 09:39:03 +0000 (10:39 +0100)]
Cygwin: check path_conv_handle for NULL before trying to dup it

path_conv_handle::dup calls DuplicateHandle unconditionally,
but we only have a handle in some cases.  Check handle for being
non-NULL before calling DuplicateHandle.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
3 years agoCygwin: drop path_conv::reset_conv_handle
Corinna Vinschen [Wed, 10 Feb 2021 09:37:13 +0000 (10:37 +0100)]
Cygwin: drop path_conv::reset_conv_handle

path_conv::reset_conv_handle is only called after fhandler::copyto
has been called.  This duplicated the path_conv_handle if there was
one, so just setting the conv handle to NULL potentially produces a
handle leak.  Replace reset_conv_handle calls with calls to
close_conv_handle and drop the reset_conv_handle method.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
3 years agoCygwin: don't copy path_conv in fhandler_base::reset
Corinna Vinschen [Wed, 10 Feb 2021 09:33:36 +0000 (10:33 +0100)]
Cygwin: don't copy path_conv in fhandler_base::reset

There's a slim chance that duplicating fhandlers may end up duplicating
path_conv_handle handles twice ending up with a handle leak, due to
fhandler_base::reset calling path_conv::operator<< after the only
caller, fhandler::copyto, already called path_conv::operator=.

Just drop the call which basically duplicates what path_conv::operator=
already did.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
3 years agoRevert "Cygwin: fstat_helper: always use handle in call to get_file_attribute"
Ken Brown [Mon, 8 Feb 2021 20:00:24 +0000 (15:00 -0500)]
Revert "Cygwin: fstat_helper: always use handle in call to get_file_attribute"

This reverts commit 76dca77f049271e2529c25de8a396e65dbce615d.  That
commit was based on the incorrect assumption that get_stat_handle,
when called on a FIFO in fstat_helper, would always return a handle
that is safe to use for getting the file information.

That assumption is true in many cases but not all.  For example, if
the call to fstat_helper arises from a call to fstat(2) on a FIFO that
has been opened for writing, then get_stat_handle will return a pipe
handle instead of a file handle.

3 years agoCygwin: drop ftw.h in favor of new newlib ftw.h
Corinna Vinschen [Tue, 9 Feb 2021 10:31:56 +0000 (11:31 +0100)]
Cygwin: drop ftw.h in favor of new newlib ftw.h

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
3 years agoFTW Port for Newlib
Eshan dhawan [Wed, 20 Jan 2021 22:14:27 +0000 (03:44 +0530)]
FTW Port for Newlib

Signed-off-by: Eshan Dhawan <eshandhawan51@gmail.com>
3 years agoRemoved Soft float from MIPS
Eshan dhawan [Wed, 3 Feb 2021 22:04:06 +0000 (03:34 +0530)]
Removed Soft float from MIPS

This Patch removes Soft Float code from MIPS.
Instead It adds the soft float code from RISCV

The code came from FreeBSD and assumes the FreeBSD softfp
implementation not the one with GCC. That was an overlooked and
fixed in the other fenv code already.

Signed-off-by: Eshan Dhawan <eshandhawan51@gmail.com>
3 years agoRISC-V: Use __bss_start for the starting point of .bss.
Yeting Kuo [Thu, 4 Feb 2021 07:34:00 +0000 (15:34 +0800)]
RISC-V: Use __bss_start for the starting point of .bss.

It's more flexible for the positions of .bss and .data.

3 years agofhandler_serial.cc: MARK and SPACE parity for serial port
Marek Smetana [Mon, 1 Feb 2021 21:02:14 +0000 (22:02 +0100)]
fhandler_serial.cc: MARK and SPACE parity for serial port

3 years agoCygwin: recognize native Windows AF_UNIX sockets as reparse points
Ken Brown [Sat, 30 Jan 2021 15:28:06 +0000 (10:28 -0500)]
Cygwin: recognize native Windows AF_UNIX sockets as reparse points

Allow check_reparse_point_target to recognize reparse points with
reparse tag IO_REPARSE_TAG_AF_UNIX.  These are used in recent versions
of Windows 10 to represent AF_UNIX sockets.

check_reparse_point_target now returns PATH_REP on files of this type,
so that they are treated as known reparse points (but not as sockets).
This allows tools like 'rm', 'ls', etc. to operate on these files.

Addresses: https://cygwin.com/pipermail/cygwin/2020-September/246362.html
   https://cygwin.com/pipermail/cygwin/2021-January/247666.html

3 years agoCygwin: include/cygwin/limits.h: new header
Ken Brown [Fri, 29 Jan 2021 16:30:44 +0000 (11:30 -0500)]
Cygwin: include/cygwin/limits.h: new header

The new header defines some Cygwin-specific limits, using private
names.  It is included by include/limits.h.

For example, we now have

  #define __OPEN_MAX 3200

in include/cygwin/limits.h and

  #define OPEN_MAX __OPEN_MAX

in include/limits.h.  The purpose is to hide implementation details
from users who view <limits.h>.

3 years agoCygwin: remove the OPEN_MAX_MAX macro
Ken Brown [Fri, 29 Jan 2021 14:14:35 +0000 (09:14 -0500)]
Cygwin: remove the OPEN_MAX_MAX macro

Replace all occurrences of OPEN_MAX_MAX by OPEN_MAX, and define the
latter to be 3200, which was the value of the former.  In view of the
recent change to getdtablesize, there is no longer a need to
distinguish between these two macros.

3 years agoCygwin: sysconf, getrlimit: don't call getdtablesize
Ken Brown [Fri, 29 Jan 2021 14:02:15 +0000 (09:02 -0500)]
Cygwin: sysconf, getrlimit: don't call getdtablesize

Now that getdtablesize always returns OPEN_MAX_MAX, we can simplify
sysconf(_SC_OPEN_MAX) and getrlimit(RLIMIT_NOFILE) to just use that
same constant instead of calling getdtablesize.

3 years agoCygwin: getdtablesize: always return OPEN_MAX_MAX
Ken Brown [Thu, 28 Jan 2021 01:06:22 +0000 (20:06 -0500)]
Cygwin: getdtablesize: always return OPEN_MAX_MAX

According to the Linux man page for getdtablesize(3), the latter is
supposed to return "the maximum number of files a process can have
open, one more than the largest possible value for a file descriptor."
The constant OPEN_MAX_MAX is the only limit enforced by Cygwin, so we
now return that.

Previously getdtablesize returned the current size of cygheap->fdtab,
Cygwin's internal file descriptor table.  But this is a dynamically
growing table, and its current size does not reflect an actual limit
on the number of open files.

With this change, gnulib now reports that getdtablesize and
fcntl(F_DUPFD) work on Cygwin.  Packages like GNU tar that use the
corresponding gnulib modules will no longer use gnulib replacements on
Cygwin.

3 years agoCygwin: exceptions.cc: Suspend all threads in sig_handle_tty_stop().
Takashi Yano via Cygwin-patches [Fri, 29 Jan 2021 03:46:26 +0000 (12:46 +0900)]
Cygwin: exceptions.cc: Suspend all threads in sig_handle_tty_stop().

- Currently, thread created by pthread_create() is not suspended by
  the signal SIGTSTP. For example, even if a process with a thread
  is suspended by Ctrl-Z, the thread continues running. This patch
  fixes the issue.

3 years agoCygwin: console: Align the behaviour against signal with pty.
Takashi Yano via Cygwin-patches [Fri, 29 Jan 2021 03:45:44 +0000 (12:45 +0900)]
Cygwin: console: Align the behaviour against signal with pty.

- Currently, read() returns -1 with EINTR if the process is suspended
  by Ctrl-Z and resumed by fg command, while pty continues to read.
  For example, xxd command stops with error "Interrupted system call"
  after Ctrl-Z and fg. This patch aligns the behaviour with pty (and
  Linux).

3 years agoCygwin: pty: Make slave read() thread-safe.
Takashi Yano via Cygwin-patches [Thu, 28 Jan 2021 14:11:33 +0000 (23:11 +0900)]
Cygwin: pty: Make slave read() thread-safe.

- Currently slave read() is somehow not thread-safe. This patch
  fixes the issue.

3 years agoCygwin: console: Make read() thread-safe.
Takashi Yano via Cygwin-patches [Thu, 28 Jan 2021 14:11:32 +0000 (23:11 +0900)]
Cygwin: console: Make read() thread-safe.

- Currently read() is somehow not thread-safe. This patch fixes
  the issue.

3 years agoCygwin: fchmodat: add limited support for AT_SYMLINK_NOFOLLOW
Ken Brown [Tue, 26 Jan 2021 20:54:05 +0000 (15:54 -0500)]
Cygwin: fchmodat: add limited support for AT_SYMLINK_NOFOLLOW

Allow fchmodat with the AT_SYMLINK_NOFOLLOW flag to succeed on
non-symlinks.  Previously it always failed, as it does on Linux.  But
POSIX permits it to succeed on non-symlinks even if it fails on
symlinks.

The reason for following POSIX rather than Linux is to make gnulib
report that fchmodat works on Cygwin.  This improves the efficiency of
packages like GNU tar that use gnulib's fchmodat module.  Previously
such packages would use a gnulib replacement for fchmodat on Cygwin.

3 years agoCygwin: pty: Allow multiple apps to enable pseudo console simultaneously.
Takashi Yano via Cygwin-patches [Thu, 28 Jan 2021 03:26:14 +0000 (12:26 +0900)]
Cygwin: pty: Allow multiple apps to enable pseudo console simultaneously.

- After commit bb428520, there has been the disadvantage:
  7) Pseudo console cannot be activated if it is already activated for
     another process on same pty.
  This patch clears this disadvantage.

3 years agoCygwin: pty: Make apps using console APIs be able to debug with gdb.
Takashi Yano via Cygwin-patches [Thu, 28 Jan 2021 03:26:13 +0000 (12:26 +0900)]
Cygwin: pty: Make apps using console APIs be able to debug with gdb.

- After commit bb428520, there has been the disadvantage:
  2) The apps which use console API cannot be debugged with gdb. This
     is because pseudo console is not activated since gdb uses
     CreateProcess() rather than exec(). Even with this limitation,
     attaching gdb to native app, in which pseudo console is already
     activated, works.
  This patch clears this disadvantage.

3 years agoCygwin: pty: Keep code page between non-cygwin apps.
Takashi Yano via Cygwin-patches [Thu, 28 Jan 2021 03:26:12 +0000 (12:26 +0900)]
Cygwin: pty: Keep code page between non-cygwin apps.

- After commit bb428520, there has been the disadvantage:
  4) Code page cannot be changed by chcp.com. Acctually, chcp works
     itself and changes code page of its own pseudo console.  However,
     since pseudo console is recreated for another process, it cannot
     inherit the code page.
  This patch clears this disadvantage.

3 years agoCygwin: pty: Inherit typeahead data between two input pipes.
Takashi Yano via Cygwin-patches [Thu, 28 Jan 2021 03:26:11 +0000 (12:26 +0900)]
Cygwin: pty: Inherit typeahead data between two input pipes.

- PTY has a problem that the key input, which is typed during windows
  native app is running, disappears when it returns to shell. This is
  beacuse pty has two input pipes, one is for cygwin apps and the other
  one is for native windows apps. The key input during windows native
  program is running is sent to the second input pipe while cygwin
  shell reads input from the first input pipe. This issue had been
  fixed once by commit 29431fcb, however, the new implementation of
  pseudo console support by commit bb428520 could not inherit this
  feature. This patch realize transfering input data between these
  two pipes bidirectionally by utilizing cygwin-console-helper process.
  The helper process is launched prior to starting the non-cygwin app,
  however, exits immediately unlike previous implementation.

3 years agoCygwin: Align *utime*() with POSIX/glibc
Corinna Vinschen [Tue, 26 Jan 2021 16:26:35 +0000 (17:26 +0100)]
Cygwin: Align *utime*() with POSIX/glibc

Followup to previous patch, this time matching definitions in Cygwin

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
3 years agoAlign *utime*() with POSIX/glibc
Sebastian Huber [Tue, 26 Jan 2021 14:16:31 +0000 (15:16 +0100)]
Align *utime*() with POSIX/glibc

Change the prototypes to be in line with POSIX/glibc.  This may fix
issues with new warnings produced by GCC 11.

Signed-off-by: Sebastian Huber <sebastian.huber@embedded-brains.de>
3 years agoCygwin: chown: make sure ctime gets updated when necessary
Ken Brown [Tue, 26 Jan 2021 02:05:37 +0000 (21:05 -0500)]
Cygwin: chown: make sure ctime gets updated when necessary

Following POSIX, ensure that ctime is updated if chown succeeds,
unless the new owner is specified as (uid_t)-1 and the new group is
specified as (gid_t)-1.  Previously, ctime was unchanged whenever the
owner and group were both unchanged.

Aside from POSIX compliance, this fix makes gnulib report that chown
works on Cygwin.  This improves the efficiency of packages like GNU
tar that use gnulib's chown module.  Previously such packages would
use a gnulib replacement for chown on Cygwin.

3 years agosyscalls.cc: Deduplicate remove
Ben Wijen [Fri, 22 Jan 2021 15:47:12 +0000 (16:47 +0100)]
syscalls.cc: Deduplicate remove

The remove code is already in the _remove_r function.
So, just call the _remove_r function.

3 years agoCygwin: console: Add missing guard regarding attach_mutex.
Takashi Yano via Cygwin-patches [Mon, 25 Jan 2021 09:18:11 +0000 (18:18 +0900)]
Cygwin: console: Add missing guard regarding attach_mutex.

- The commit a5333345 did not fix the problem enough. This patch
  provides additional guard for the issue.

3 years agosyscalls.cc: unlink_nt: Try FILE_DISPOSITION_IGNORE_READONLY_ATTRIBUTE
Ben Wijen [Fri, 22 Jan 2021 15:47:11 +0000 (16:47 +0100)]
syscalls.cc: unlink_nt: Try FILE_DISPOSITION_IGNORE_READONLY_ATTRIBUTE

I think we don't need an extra flag as we can utilize: access & FILE_WRITE_ATTRIBUTES
What do you think?

Ben Wijen (1):
  syscalls.cc: unlink_nt: Try FILE_DISPOSITION_IGNORE_READONLY_ATTRIBUTE

 winsup/cygwin/ntdll.h     |  3 ++-
 winsup/cygwin/syscalls.cc | 22 +++++++--------
 winsup/cygwin/wincap.cc   | 11 ++++++++
 winsup/cygwin/wincap.h    | 56 ++++++++++++++++++++-------------------
 4 files changed, 53 insertions(+), 39 deletions(-)

--
2.30.0

>From 2d0ff6fec10d03c24d11c747852018b7bc1136ac Mon Sep 17 00:00:00 2001
In-Reply-To: <20210122105201.GD810271@calimero.vinschen.de>
References: <20210122105201.GD810271@calimero.vinschen.de>
From: Ben Wijen <ben@wijen.net>
Date: Tue, 17 Dec 2019 15:15:25 +0100
Subject: [PATCH v3 1/8] syscalls.cc: unlink_nt: Try
 FILE_DISPOSITION_IGNORE_READONLY_ATTRIBUTE

Implement wincap.has_posix_unlink_semantics_with_ignore_readonly and when set
skip setting/clearing of READONLY attribute and instead use
FILE_DISPOSITION_IGNORE_READONLY_ATTRIBUTE

3 years agoCygwin: ptsname_r: always return an error number on failure
Ken Brown [Wed, 20 Jan 2021 17:01:13 +0000 (12:01 -0500)]
Cygwin: ptsname_r: always return an error number on failure

Return EBADF on a bad file descriptor.  Previously 0 was returned, in
violation of the requirement in
https://man7.org/linux/man-pages/man3/ptsname_r.3.html that an error
number should be returned on failure.

We are intentionally deviating from Linux, on which ENOTTY is
returned.

Addresses: https://lists.gnu.org/archive/html/bug-gnulib/2021-01/msg00245.html

3 years agoCygwin: normalize_posix_path: fix error handling when .. is encountered
Ken Brown [Wed, 20 Jan 2021 15:12:58 +0000 (10:12 -0500)]
Cygwin: normalize_posix_path: fix error handling when .. is encountered

When .. is in the source path and the path prefix exists but is not a
directory, return ENOTDIR instead of ENOENT.  This fixes a POSIX
compliance issue for realpath(3):

  https://pubs.opengroup.org/onlinepubs/9699919799/functions/realpath.html

Addresses: https://lists.gnu.org/archive/html/bug-gnulib/2021-01/msg00214.html

3 years agoCygwin: Move post-dir unlink check
Ben Wijen [Wed, 20 Jan 2021 16:10:51 +0000 (17:10 +0100)]
Cygwin: Move post-dir unlink check

Move post-dir unlink check from fhandler_disk_file::rmdir to
_unlink_nt_post_dir_check

If a directory is not removed through fhandler_disk_file::rmdir
we can now make sure the post dir check is performed.

3 years agoCygwin: console: Fix "Bad file descriptor" error in script command.
Takashi Yano via Cygwin-patches [Wed, 20 Jan 2021 09:16:20 +0000 (18:16 +0900)]
Cygwin: console: Fix "Bad file descriptor" error in script command.

- After the commit 72770148, script command exits occasionally with
  the error "Bad file descriptor" if it is started in console on Win7
  and non-cygwin process is executed. This patch fixes the issue.

3 years agoCygwin: pty: Reduce buffer size in get_console_process_id().
Takashi Yano via Cygwin-patches [Wed, 20 Jan 2021 00:57:00 +0000 (09:57 +0900)]
Cygwin: pty: Reduce buffer size in get_console_process_id().

- The buffer used in get_console_process_id(), introduced by commit
  72770148, is too large and ERROR_NOT_ENOUGH_MEMORY occurs in Win7.
  Therefore, the buffer size has been reduced.

3 years agoCygwin: pty: Lessen the side effect of workaround for rlwarp.
Takashi Yano via Cygwin-patches [Tue, 19 Jan 2021 09:27:02 +0000 (18:27 +0900)]
Cygwin: pty: Lessen the side effect of workaround for rlwarp.

- This patch lessens the side effect of the workaround for rlwrap
  introduced by commit 4e16b033.

3 years agoCygwin: spawn.cc: Fix typo in comment by commit 974e6d76.
Takashi Yano via Cygwin-patches [Mon, 18 Jan 2021 18:45:24 +0000 (03:45 +0900)]
Cygwin: spawn.cc: Fix typo in comment by commit 974e6d76.

3 years agoCygwin: rmdir: handle /dev in fhandler_dev::rmdir
Corinna Vinschen [Tue, 19 Jan 2021 09:50:36 +0000 (10:50 +0100)]
Cygwin: rmdir: handle /dev in fhandler_dev::rmdir

The isdev_dev check in rmdir is unclean.  Create a virtual method
fhandler_dev::rmdir to handle this transparently.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
3 years agoCygwin: document a recent bug fix
Ken Brown [Fri, 15 Jan 2021 17:37:57 +0000 (12:37 -0500)]
Cygwin: document a recent bug fix

This documents commit aec64798, "Cygwin: add flag to indicate reparse
points unknown to WinAPI".

3 years agoCygwin: pty: Set input_available_event only for cygwin pipe.
Takashi Yano via Cygwin-patches [Fri, 15 Jan 2021 09:26:31 +0000 (18:26 +0900)]
Cygwin: pty: Set input_available_event only for cygwin pipe.

- cat exits immediately in the following senario.
    1) Execute env CYGWIN=disable_pcon script
    2) Execute cmd.exe
    3) Execute cat in cmd.exe.
  This is caused by setting input_available_event for the pipe for
  non-cygwin app. This patch fixes the issue.

3 years agoCygwin: pty: Make master thread functions be static.
Takashi Yano via Cygwin-patches [Fri, 15 Jan 2021 08:32:13 +0000 (17:32 +0900)]
Cygwin: pty: Make master thread functions be static.

- The functions pty_master_thread() and pty_master_fwd_thread()
  should be static (i.e. should not access class member) because
  the instance is deleted if the master is dup()'ed and the first
  master is closed. In this case, because the dup()'ed instance
  still exists, these master threads are also still alive even
  though the instance has been deleted. As a result, accesing
  class members in these functions causes accessi violation.

  Addresses:
  https://cygwin.com/pipermail/cygwin-developers/2021-January/012030.html

3 years agoCygwin: pty: Prevent pty from changing code page of parent console.
Takashi Yano via Cygwin-patches [Mon, 18 Jan 2021 13:10:57 +0000 (22:10 +0900)]
Cygwin: pty: Prevent pty from changing code page of parent console.

- After commit 232fde0e, pty changes console code page when the first
  non-cygwin app is executed. If pty is started in real console device,
  pty changes the code page of root console. This causes very annoying
  result because changing code page changes the font of command prompt
  if console is in legacy mode. This patch avoids this by creating a
  new invisible console for the first pty started in console device.

3 years agocxx.cc: Fix dynamic initialization for static local variables
Ben Wijen [Fri, 15 Jan 2021 13:45:29 +0000 (14:45 +0100)]
cxx.cc: Fix dynamic initialization for static local variables

The old implementation for __cxa_guard_acquire did not return 1,
therefore dynamic initialization was never performed.

If concurrent-safe dynamic initialisation is ever needed, CXX ABI
must be followed when re-implementing __cxa_guard_acquire (et al.)

3 years agosyscalls.cc: Use EISDIR
Ben Wijen [Fri, 15 Jan 2021 13:45:27 +0000 (14:45 +0100)]
syscalls.cc: Use EISDIR

This is the non-POSIX value returned by Linux since 2.1.132.

3 years agosyscalls.cc: Fix num_links
Ben Wijen [Fri, 15 Jan 2021 13:45:26 +0000 (14:45 +0100)]
syscalls.cc: Fix num_links

NtQueryInformationFile on fh_ro needs FILE_READ_ATTRIBUTES
to succeed.

3 years agoCygwin: Add Ben Wijen to list of contributors
Corinna Vinschen [Mon, 18 Jan 2021 11:01:11 +0000 (12:01 +0100)]
Cygwin: Add Ben Wijen to list of contributors

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
3 years agoCygwin: pty: Make close_pseudoconsole() be a static member function.
Takashi Yano via Cygwin-patches [Fri, 15 Jan 2021 08:32:11 +0000 (17:32 +0900)]
Cygwin: pty: Make close_pseudoconsole() be a static member function.

- The function close_pseudoconsole() should be static so that it
  can be safely called in spawn.cc even after the fhandler_pty_slave
  instance has been deleted. That is, there is a problem with the
  current code. This patch fixes the issue.

3 years agoCygwin: console: Revise the code to switch xterm mode.
Takashi Yano via Cygwin-patches [Fri, 15 Jan 2021 08:32:10 +0000 (17:32 +0900)]
Cygwin: console: Revise the code to switch xterm mode.

- If application changes the console mode, mode management introduced
  by commit 10d8c278 will be corrupted. For example, stdout of jansi
  v2.0.1 or later is piped to less, jansi resets the xterm mode flag
  ENABLE_VIRTUAL_TERMINA_PROCESSING when jansi is terminated. This
  causes garbled output in less because less needs this flag enabled.
  This patch fixes the issue.

3 years agoCygwin: pty: Add workaround for rlwrap 0.40 or later.
Takashi Yano via Cygwin-patches [Fri, 15 Jan 2021 08:32:09 +0000 (17:32 +0900)]
Cygwin: pty: Add workaround for rlwrap 0.40 or later.

- The workaround for rlwrap introduced by commit 8199b0cc does not
  take effect for rlwrap 0.40 or later. This patch add a workaround
  for rlwrap 0.40 or later as well.

3 years agoCygwin: fstatat: call fstat64 instead of fstat
Ken Brown [Tue, 12 Jan 2021 19:18:28 +0000 (14:18 -0500)]
Cygwin: fstatat: call fstat64 instead of fstat

This fixes a bug on 32-bit Cygwin that was introduced in commit
84252946, "Cygwin: fstatat, fchownat: support the AT_EMPTY_PATH flag".

Add a comment explaining why fstat should not be called.

Addresses: https://cygwin.com/pipermail/cygwin/2021-January/247399.html

3 years agoRTEMS: Add <poll.h> and <sys/poll.h>
Sebastian Huber [Mon, 4 Jan 2021 18:41:43 +0000 (19:41 +0100)]
RTEMS: Add <poll.h> and <sys/poll.h>

Add the POSIX header file <poll.h> which is used by the GCC 11 Ada
runtime support.

Signed-off-by: Sebastian Huber <sebastian.huber@embedded-brains.de>
3 years agoBump up newlib version to 4.1.0 newlib-4.1.0
Jeff Johnston [Fri, 18 Dec 2020 23:50:49 +0000 (18:50 -0500)]
Bump up newlib version to 4.1.0

3 years agofixes to make compilation succeeds
Paul Zimmermann [Fri, 18 Dec 2020 08:52:40 +0000 (09:52 +0100)]
fixes to make compilation succeeds

3 years agoUpdate gamma functions from code in picolibc
Jeff Johnston [Thu, 17 Dec 2020 20:58:49 +0000 (15:58 -0500)]
Update gamma functions from code in picolibc

- fixes issue with inf sign when x is -0

3 years agofhandler_proc.cc(format_proc_cpuinfo): report Intel SGX bits
Brian Inglis [Thu, 17 Dec 2020 07:11:28 +0000 (00:11 -0700)]
fhandler_proc.cc(format_proc_cpuinfo): report Intel SGX bits

Update to Linux next 5.10 cpuinfo flags for Intel SDM 36.7.1 Software
Guard Extensions, and 38.1.4 SGX Launch Control Configuration.
Launch control restricts what software can run with enclave protections,
which helps protect the system from bad enclaves.

3 years agoRISC-V: Add semihosting support
Craig Blackmore [Tue, 15 Dec 2020 12:00:27 +0000 (12:00 +0000)]
RISC-V: Add semihosting support

3 years agoAdd declarations for __ieee754_tgamma functions to fdlibm.h
Jeff Johnston [Wed, 16 Dec 2020 20:25:25 +0000 (15:25 -0500)]
Add declarations for __ieee754_tgamma functions to fdlibm.h

3 years agoCygwin: pty: Revise the workaround for rlwrap.
Takashi Yano via Cygwin-patches [Wed, 16 Dec 2020 09:10:58 +0000 (18:10 +0900)]
Cygwin: pty: Revise the workaround for rlwrap.

- Previous workaround has a problem that screen is distorted if up
  arrow key is pressed at the first line after running "rlwrap cmd".
  This patch fixes the issue.

3 years agoCygwin: pty: Check response for CSI6n more strictly.
Takashi Yano via Cygwin-patches [Wed, 16 Dec 2020 09:10:15 +0000 (18:10 +0900)]
Cygwin: pty: Check response for CSI6n more strictly.

- Previous code to read response for CSI6n allows invalid response
  such as "CSI Pl; Pc H" other than correct response "CSI Pl; Pc R".
  With this patch, the response is checked more strictly.

3 years agoCygwin: Make sure newer apps get uname_x even when loading uname dynamically
Corinna Vinschen [Mon, 14 Dec 2020 11:29:23 +0000 (12:29 +0100)]
Cygwin: Make sure newer apps get uname_x even when loading uname dynamically

if an application built after API version 334 loads uname dynamically,
it actually gets the old uname, rather than the new uname_x.  Fix this by
checking the apps API version in uname and call uname_x instead, if it's
a newer app.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
3 years agoarm: Fix memchr() for Armv8-R
Sebastian Huber [Fri, 4 Dec 2020 07:42:04 +0000 (08:42 +0100)]
arm: Fix memchr() for Armv8-R

The Cortex-R52 processor is an Armv8-R processor with a NEON unit.  This
fix prevents conflicting architecture profiles A/R errors issued by the
linker.

Signed-off-by: Sebastian Huber <sebastian.huber@embedded-brains.de>
3 years agoCygwin: pty: Add a workaround for rlwrap.
Takashi Yano via Cygwin-patches [Mon, 14 Dec 2020 08:26:47 +0000 (17:26 +0900)]
Cygwin: pty: Add a workaround for rlwrap.

- If rlwrap is used with non-cygwin apps, it fails to setup pseudo
  console. This patch adds a workaround for this issue.

3 years agoCygwin: pty: Revise the code for timeout in term_has_pcon_cap().
Takashi Yano via Cygwin-patches [Mon, 14 Dec 2020 08:26:09 +0000 (17:26 +0900)]
Cygwin: pty: Revise the code for timeout in term_has_pcon_cap().

- Sometimes timeout period in term_has_pcon_cap() may not be enough
  when the machine slows down for some reason. This patch eases the
  issue. In the new code, effective timeout period is expected to be
  extended as a result due to slowing-down the wait loop as well when
  the machine gets into busy.

3 years agoCygwin: pty: Skip term_has_pcon_cap() if pseudo console is disabled.
Takashi Yano via Cygwin-patches [Mon, 14 Dec 2020 08:25:33 +0000 (17:25 +0900)]
Cygwin: pty: Skip term_has_pcon_cap() if pseudo console is disabled.

- This patch skips unnecessary term_has_pcon_cap() call if pseudo
  console is disabled.

3 years agoFix error in powf for x close to 1 and large y
Fabian Schriever [Fri, 11 Dec 2020 15:57:35 +0000 (16:57 +0100)]
Fix error in powf for x close to 1 and large y

This patch fixes the error found by Paul Zimmermann (see
https://homepages.loria.fr/PZimmermann/papers/#accuracy) regarding x
close to 1 and rather large y (specifically he found the case
powf(0x1.ffffeep-1,-0x1.000002p+27) which returns +Inf instead of the
correct value). We found 2 more values for x which show the same faulty
behaviour, and all 3 are fixed with this patch. We have tested all
combinations for x in [+1.fffdfp-1, +1.00020p+0] and y in
[-1.000007p+27, -1.000002p+27] and [1.000002p+27,1.000007p+27].

3 years agoBump newlib release to 4.0.0
Jeff Johnston [Tue, 17 Nov 2020 21:47:35 +0000 (16:47 -0500)]
Bump newlib release to 4.0.0

3 years agocygwin: use CREATE_DEFAULT_ERROR_MODE in spawn
Jeremy Drake via Cygwin-patches [Wed, 9 Dec 2020 19:27:11 +0000 (11:27 -0800)]
cygwin: use CREATE_DEFAULT_ERROR_MODE in spawn

This allows native processes to get Windows-default error handling
behavior (such as invoking the registered JIT debugger).

3 years agoCygwin: dtable::dup_worker: update comment and debug output
Ken Brown [Mon, 7 Dec 2020 21:19:57 +0000 (16:19 -0500)]
Cygwin: dtable::dup_worker: update comment and debug output

The comment and debug output became obsolete in commit 23771fa1f7 when
dup_worker started calling fhandler_base::clone instead of build_fh_pc
and fhandler_base::operator=.

3 years agoCygwin: Allow to set SO_PEERCRED zero (v2)
Mark Geisert [Mon, 7 Dec 2020 10:29:36 +0000 (02:29 -0800)]
Cygwin: Allow to set SO_PEERCRED zero (v2)

The existing code errors as EINVAL any attempt to set a value for
SO_PEERCRED via setsockopt() on an AF_UNIX/AF_LOCAL socket.  But to
enable the workaround set_no_getpeereid behavior for Python one has
to be able to set SO_PEERCRED to zero.  Ergo, this patch.  Python has
no way to specify a NULL pointer for 'optval'.

This v2 of patch allows the original working (i.e., allow NULL,0 for
optval,optlen to mean turn off SO_PEERCRED) in addition to the new
working described above.  The sense of the 'if' stmt is reversed for
readability.

3 years agoCygwin: Launch cygmagic with bash, not sh
Mark Geisert [Mon, 7 Dec 2020 06:17:15 +0000 (22:17 -0800)]
Cygwin: Launch cygmagic with bash, not sh

On some systems /bin/sh is not /bin/bash and cygmagic has bash-isms in
it.  So even though cygmagic has a /bin/bash shebang, it also needs to be
launched with bash from within Makefile.in.

3 years agoFix trace output for getdomainname()
Anton Lavrentiev via Cygwin-patches [Fri, 4 Dec 2020 22:58:01 +0000 (17:58 -0500)]
Fix trace output for getdomainname()

3 years agowinsup/doc/Makefile.in: create man5 dir and install proc.5
Brian Inglis [Wed, 25 Nov 2020 06:49:31 +0000 (23:49 -0700)]
winsup/doc/Makefile.in: create man5 dir and install proc.5

3 years agospecialnames.xml: add proc(5) Cygwin man page
Brian Inglis [Wed, 25 Nov 2020 06:49:30 +0000 (23:49 -0700)]
specialnames.xml: add proc(5) Cygwin man page

3 years agoCygwin: Fix remaining warnings building path testsuite
Corinna Vinschen [Tue, 1 Dec 2020 09:10:40 +0000 (10:10 +0100)]
Cygwin: Fix remaining warnings building path testsuite

3 years agoCygwin: Fix building of utils testsuite
Jon Turney [Mon, 30 Nov 2020 16:58:02 +0000 (16:58 +0000)]
Cygwin: Fix building of utils testsuite

Avoid referencing undefined max_mount_entry.

3 years agoCygwin: add flag to indicate reparse points unknown to WinAPI
Corinna Vinschen [Wed, 2 Dec 2020 15:12:58 +0000 (16:12 +0100)]
Cygwin: add flag to indicate reparse points unknown to WinAPI

https://cygwin.com/pipermail/cygwin/2020-December/246938.html
reports a problem where, when adding a Cygwin default symlink
to $PATH since Cygwin 3.1.5, $PATH handling appears to be broken.

3.1.5 switched to WSL symlinks as Cygwin default symlinks.

A piece of code in path handling skips resolving reparse points
if they are the last component in the path.  Thus a reparse point
in $PATH is not resolved but converted to Windows path syntax
verbatim.

If you do this with a WSL symlink, certain WinAPI functions fail.
The underlying $PATH handling fails to recognize the reparse
point in $PATH and returns with STATUS_IO_REPARSE_TAG_NOT_HANDLED.
As a result, the calling WinAPI function fails, most prominently
so CreateProcess.

Fix this problem by adding a PATH_REP_NOAPI bit to path_types
and a matching method path_conv::is_winapi_reparse_point().

Right now this flag is set for WSL symlinks and Cygwin AF_UNIX
sockets (new type implemented as reparse points).

The aforementioned code skipping repare point path resolution calls
is_winapi_reparse_point() rather than is_known_reparse_point(),
so now path resolution is only skipped for reparse points known
to WinAPI.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
3 years agoCygwin: path.h: add comments to briefly explain path_types
Corinna Vinschen [Wed, 2 Dec 2020 14:57:39 +0000 (15:57 +0100)]
Cygwin: path.h: add comments to briefly explain path_types

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
3 years agoCygwin: testsuite: libltp: fix warnings showing up with -Wall
Corinna Vinschen [Wed, 2 Dec 2020 13:12:24 +0000 (14:12 +0100)]
Cygwin: testsuite: libltp: fix warnings showing up with -Wall

This libltp is old as old dirt and still using K&R style.
If it's really to be used again at all, it needs a serious
refresh.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
3 years agoCygwin: /proc/sys FS: don't export NFS and DFS as block devices
Corinna Vinschen [Tue, 1 Dec 2020 08:37:07 +0000 (09:37 +0100)]
Cygwin: /proc/sys FS: don't export NFS and DFS as block devices

Network filesystems are not block devices.  Apparently this code
hasn't been executed anyway, given how network filesystems are
hidden behind \Device\Mup.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
3 years agoCygwin: Fix access to block devices below /proc/sys.
Christian Franke [Sat, 28 Nov 2020 21:09:23 +0000 (22:09 +0100)]
Cygwin: Fix access to block devices below /proc/sys.

Use fhandler_dev_floppy instead of fhandler_procsys for such devices.
The read()/write() functions from fhandler_procsys do not ensure
sector aligned transfers and lseek() fails always.

Signed-off-by: Christian Franke <franke@computer.org>
3 years agoCygwin: Have cygmagic not create output if an error occurs
Jon Turney [Thu, 12 Nov 2020 17:58:58 +0000 (17:58 +0000)]
Cygwin: Have cygmagic not create output if an error occurs

Improve the 'cygmagic' script, so it doesn't create the output file if
an error occurs, even in one of the backtick-enclosed pipelines it runs.

3 years agoCygwin: Use standard CXXFLAGS when compiling localtime_wrapper.c
Jon Turney [Thu, 19 Nov 2020 21:38:47 +0000 (21:38 +0000)]
Cygwin: Use standard CXXFLAGS when compiling localtime_wrapper.c

This has an separate, explicit compilation rule which omits CXXFLAGS, so
expected flags like '-g -O2' aren't being used.

3 years agoCygwin: Drop libgmon.a build dependency on gcrt0.o
Jon Turney [Thu, 19 Nov 2020 21:34:44 +0000 (21:34 +0000)]
Cygwin: Drop libgmon.a build dependency on gcrt0.o

libgmon.a depends on gcrt0.o, but doesn't include it.

3 years agoCygwin: pty: Fix minor style issue.
Takashi Yano via Cygwin-patches [Mon, 23 Nov 2020 11:03:04 +0000 (20:03 +0900)]
Cygwin: pty: Fix minor style issue.

3 years agoCygwin: pty: Discard "OSC Ps; ? BEL/ST" in pseudo console output.
Takashi Yano via Cygwin-patches [Mon, 23 Nov 2020 11:03:03 +0000 (20:03 +0900)]
Cygwin: pty: Discard "OSC Ps; ? BEL/ST" in pseudo console output.

- If vim is executed in WSL in mintty, some garbage string caused
  by "OSC Ps;? BEL/ST" will be shown in some situations. This patch
  fixes the issue by removing "OSC Ps;? BEL/ST" from pseudo console
  output.

3 years agoCygwin: pty: Fix a bug in the code removing "CSI > Pm m".
Takashi Yano via Cygwin-patches [Mon, 23 Nov 2020 11:03:02 +0000 (20:03 +0900)]
Cygwin: pty: Fix a bug in the code removing "CSI > Pm m".

- The code added by 8121b606e843c001d5ca5213d24099e04ebc62ca has a
  bug which fails to remove multiple "CSI > Pm m" sequences. This
  patch fixes the bug.

3 years agoCygwin: Remove surplus autoconf auxiliary files
Jon Turney [Wed, 21 Oct 2020 19:41:58 +0000 (20:41 +0100)]
Cygwin: Remove surplus autoconf auxiliary files

Since we are now only configuring once, in winsup, with
AC_CONFIG_AUX_DIR(..), the auxiliary files are taken from the top-level.

(Previously we had a random assorment of AC_CONFIG_AUX_DIR(..) and
AC_CONFIG_AUX_DIR(../..) in winsup subdirectories, so auxiliary files
would be taken from winsup or the top-level.)

3 years agoCygwin: Remove recursive configure
Jon Turney [Tue, 6 Oct 2020 15:22:02 +0000 (16:22 +0100)]
Cygwin: Remove recursive configure

There's doesn't seem to be much use in independently distributing these
subdirectories, so allowing them to be independently configured seems
pointless and overcomplicated.

The order in which the subdirectories are built is still a little odd,
as cygwin is linked with libcygserver, and cygserver is then linked with
cygwin. So, we build the cygwin directory first, which invokes a build
of libcygserver in the cygserver directory, and then build in the
cygserver directory to build the cygserver executable.

Drop AC_CONFIGURE_ARGS, since we don't need to recursively call
configure with the same arguments anymore.

Slightly refine when we build utils: Previously we didn't build any
utils if MinGW compiler use was avoided, now we just avoid building
those utils which require that compiler.

Greatly simplify how winsup_srcdir and target_builddir are set, since
we're only configuring from one directory.  (These are still kept
absolute, since we don't adjust them where used for being used in a
subdirectory).

Remove configure.cygwin and put it's (greatly reduced) contents inline
in the one place it's used now.

Remove generated configure and aclocal.m4 in subdirectories.

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