Re: [PATCH v7 1/1] Cygwin: pty: add pseudo console support.

[Corinna Vinschen <corinna-cygwin at cygwin dot com>]

Hi Takashi,

On Aug 16 01:09, Takashi Yano wrote:
> - Support pseudo console in PTY. Pseudo console is a new feature
>   in Windows 10 1809, which provides console APIs on virtual
>   terminal. With this patch, native console applications can work
>   in PTY such as mintty, ssh, gnu screen or tmux.
> ---
>  winsup/cygwin/dtable.cc               |  50 ++
>  winsup/cygwin/fhandler.h              |  42 +-
>  winsup/cygwin/fhandler_console.cc     |  32 +
>  winsup/cygwin/fhandler_tty.cc         | 843 +++++++++++++++++++++++++-
>  winsup/cygwin/fork.cc                 |  24 +
>  winsup/cygwin/select.cc               |  22 +-
>  winsup/cygwin/spawn.cc                |  55 ++
>  winsup/cygwin/strace.cc               |  24 +
>  winsup/cygwin/tty.cc                  |   8 +
>  winsup/cygwin/tty.h                   |  24 +-
>  winsup/utils/cygwin-console-helper.cc |  14 +-
>  11 files changed, 1093 insertions(+), 45 deletions(-)

It looks like there's some timing problem in terms of stderr output.  I
just made up the example for the Cygwin ML discussion about the mcmodel.
When I start the code built with -mcmodel=small I expected the output:

Cygwin runtime failure: /home/corinna/dll/main.exe: Invalid relocation.
Offset 0xfffffffd80348989 at address 0x40000103b doesn't fit into 32 bits

However, no output was generated at first run, and not at second:

---
$ ./main
$ ./main
$ 
---

Then I started ssh, and suddenly the messages appeared.  They
even appeared at the right spot:

---
$ ./main
Cygwin runtime failure: /home/corinna/dll/main.exe: Invalid relocation.
Offset 0xfffffffd80348989 at address 0x40000103b doesn't fit into 32 bits
$ ./main
Cygwin runtime failure: /home/corinna/dll/main.exe: Invalid relocation.
Offset 0xfffffffd80348989 at address 0x40000103b doesn't fit into 32 bits
$ ssh
[...]
----

Very puzzelling.


Corinna

-- 
Corinna Vinschen
Cygwin Maintainer

Attachment: signature.asc
Description: PGP signature