September 2020 - Wine-Commits

Jacek Caban : conhost: Terminate current read request when input when reading tty input fails.
by Alexandre Julliard Sept. 21, 2020

Sept. 21, 2020

Module: wine Branch: master Commit: c6909e52543b62dfcd5b183014e6eae27b69f3fc URL: https://source.winehq.org/git/wine.git/?a=commit;h=c6909e52543b62dfcd5b1830… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Sep 21 17:05:32 2020 +0200 conhost: Terminate current read request when input when reading tty input fails. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- programs/conhost/conhost.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/programs/conhost/conhost.c b/programs/conhost/conhost.c index a9ad38eb99..1d48194206 100644 --- a/programs/conhost/conhost.c +++ b/programs/conhost/conhost.c @@ -1637,6 +1637,11 @@ static DWORD WINAPI tty_input( void *param ) } TRACE( "NtReadFile failed: %#x\n", status ); + + EnterCriticalSection( &console_section ); + if (console->read_ioctl) read_complete( console, status, NULL, 0, FALSE ); + LeaveCriticalSection( &console_section ); + return 0; }

1 0

Jacek Caban : ntdll: Fill console size in get_initial_console.
by Alexandre Julliard Sept. 21, 2020

Sept. 21, 2020

Module: wine Branch: master Commit: 0957f9302ca40bd85e092ed335d3fd10ab9bf4da URL: https://source.winehq.org/git/wine.git/?a=commit;h=0957f9302ca40bd85e092ed3… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Sep 21 17:07:15 2020 +0200 ntdll: Fill console size in get_initial_console. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/unix/env.c | 16 +++++++++++++++- 1 file changed, 15 insertions(+), 1 deletion(-) diff --git a/dlls/ntdll/unix/env.c b/dlls/ntdll/unix/env.c index 1684f5dcc9..f4d9ef15fc 100644 --- a/dlls/ntdll/unix/env.c +++ b/dlls/ntdll/unix/env.c @@ -32,6 +32,7 @@ #include <fcntl.h> #include <stdarg.h> #include <sys/types.h> +#include <sys/ioctl.h> #ifdef HAVE_SYS_STAT_H # include <sys/stat.h> #endif @@ -1154,10 +1155,23 @@ NTSTATUS CDECL get_dynamic_environment( WCHAR *env, SIZE_T *size ) */ void CDECL get_initial_console( RTL_USER_PROCESS_PARAMETERS *params ) { + int output_fd = -1; if (isatty(0) || isatty(1) || isatty(2)) params->ConsoleHandle = CONSOLE_HANDLE_SHELL; if (!isatty(0)) wine_server_fd_to_handle( 0, GENERIC_READ|SYNCHRONIZE, OBJ_INHERIT, &params->hStdInput ); - if (!isatty(1)) wine_server_fd_to_handle( 1, GENERIC_WRITE|SYNCHRONIZE, OBJ_INHERIT, &params->hStdOutput ); if (!isatty(2)) wine_server_fd_to_handle( 2, GENERIC_WRITE|SYNCHRONIZE, OBJ_INHERIT, &params->hStdError ); + else output_fd = 2; + if (!isatty(1)) wine_server_fd_to_handle( 1, GENERIC_WRITE|SYNCHRONIZE, OBJ_INHERIT, &params->hStdOutput ); + else output_fd = 1; + + if (output_fd != -1) + { + struct winsize size; + if (!ioctl( output_fd, TIOCGWINSZ, &size )) + { + params->dwXCountChars = size.ws_col; + params->dwYCountChars = size.ws_row; + } + } }

1 0

Jacek Caban : conhost: Delay creating input thread in Unix mode.
by Alexandre Julliard Sept. 21, 2020

Sept. 21, 2020

Module: wine Branch: master Commit: 158471d676e799d3d23a356b442ecc4afd2ade57 URL: https://source.winehq.org/git/wine.git/?a=commit;h=158471d676e799d3d23a356b… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Sep 21 17:06:11 2020 +0200 conhost: Delay creating input thread in Unix mode. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- programs/conhost/conhost.c | 19 ++++++++++++++----- 1 file changed, 14 insertions(+), 5 deletions(-) diff --git a/programs/conhost/conhost.c b/programs/conhost/conhost.c index 3f7b5fb914..0de55af14c 100644 --- a/programs/conhost/conhost.c +++ b/programs/conhost/conhost.c @@ -99,6 +99,7 @@ struct console unsigned int input_cp; /* console input codepage */ unsigned int output_cp; /* console output codepage */ unsigned int win; /* window handle if backend supports it */ + HANDLE input_thread; /* input thread handle */ HANDLE tty_input; /* handle to tty input stream */ HANDLE tty_output; /* handle to tty output stream */ char tty_buffer[4096]; /* tty output buffer */ @@ -1681,11 +1682,20 @@ static DWORD WINAPI tty_input( void *param ) EnterCriticalSection( &console_section ); if (console->read_ioctl) read_complete( console, status, NULL, 0, FALSE ); + CloseHandle( console->input_thread ); + console->input_thread = NULL; LeaveCriticalSection( &console_section ); return 0; } +static BOOL ensure_tty_input_thread( struct console *console ) +{ + if (!console->input_thread) + console->input_thread = CreateThread( NULL, 0, tty_input, console, 0, NULL ); + return console->input_thread != NULL; +} + static NTSTATUS screen_buffer_activate( struct screen_buffer *screen_buffer ) { RECT update_rect; @@ -2424,6 +2434,7 @@ static NTSTATUS console_input_ioctl( struct console *console, unsigned int code, case IOCTL_CONDRV_READ_CONSOLE: if (in_size || *out_size % sizeof(WCHAR)) return STATUS_INVALID_PARAMETER; + ensure_tty_input_thread( console ); return read_console( console, *out_size ); case IOCTL_CONDRV_READ_INPUT: @@ -2431,6 +2442,7 @@ static NTSTATUS console_input_ioctl( struct console *console, unsigned int code, unsigned int blocking; NTSTATUS status; if (in_size && in_size != sizeof(blocking)) return STATUS_INVALID_PARAMETER; + ensure_tty_input_thread( console ); blocking = in_size && *(unsigned int *)in_data; if (blocking && !console->record_count && *out_size) { @@ -2683,7 +2695,7 @@ static int main_loop( struct console *console, HANDLE signal ) int __cdecl wmain(int argc, WCHAR *argv[]) { int headless = 0, i, width = 0, height = 0; - HANDLE signal = NULL, input_thread; + HANDLE signal = NULL; WCHAR *end; static struct console console; @@ -2764,10 +2776,7 @@ int __cdecl wmain(int argc, WCHAR *argv[]) console.tty_input = GetStdHandle( STD_INPUT_HANDLE ); console.tty_output = GetStdHandle( STD_OUTPUT_HANDLE ); init_tty_output( &console ); - - if (!(input_thread = CreateThread( NULL, 0, tty_input, &console, 0, NULL ))) - return 1; - CloseHandle( input_thread ); + if (!console.is_unix && !ensure_tty_input_thread( &console )) return 1; } return main_loop( &console, signal );

1 0

Jacek Caban : kernelbase: Use conhost to handle Unix consoles.
by Alexandre Julliard Sept. 21, 2020

Sept. 21, 2020

Module: wine Branch: master Commit: 54e117018cd4cc58c258da92686bfad13946bde2 URL: https://source.winehq.org/git/wine.git/?a=commit;h=54e117018cd4cc58c258da92… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Sep 21 17:07:29 2020 +0200 kernelbase: Use conhost to handle Unix consoles. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernelbase/console.c | 60 ++++++++++++++++++++++++++++++++++++++++++----- dlls/ntdll/unix/env.c | 29 ++++++++++++++++++----- 2 files changed, 77 insertions(+), 12 deletions(-) diff --git a/dlls/kernelbase/console.c b/dlls/kernelbase/console.c index 2289d7ea5f..0d105199fc 100644 --- a/dlls/kernelbase/console.c +++ b/dlls/kernelbase/console.c @@ -1656,11 +1656,18 @@ static HANDLE create_pseudo_console( COORD size, HANDLE input, HANDLE output, HA si.StartupInfo.hStdError = output; si.StartupInfo.dwFlags = STARTF_USESTDHANDLES; swprintf( conhost_path, ARRAY_SIZE(conhost_path), L"%s\\conhost.exe", system_dir ); - swprintf( cmd, ARRAY_SIZE(cmd), - L"\"%s\" --headless %s--width %u --height %u --signal 0x%x --server 0x%x", - conhost_path, (flags & PSEUDOCONSOLE_INHERIT_CURSOR) ? L"--inheritcursor " : L"", - size.X, size.Y, signal, server ); - + if (signal) + { + swprintf( cmd, ARRAY_SIZE(cmd), + L"\"%s\" --headless %s--width %u --height %u --signal 0x%x --server 0x%x", + conhost_path, (flags & PSEUDOCONSOLE_INHERIT_CURSOR) ? L"--inheritcursor " : L"", + size.X, size.Y, signal, server ); + } + else + { + swprintf( cmd, ARRAY_SIZE(cmd), L"\"%s\" --unix --width %u --height %u --server 0x%x", + conhost_path, size.X, size.Y, server ); + } Wow64DisableWow64FsRedirection( &redir ); res = CreateProcessW( conhost_path, cmd, NULL, NULL, TRUE, DETACHED_PROCESS, NULL, NULL, &si.StartupInfo, &pi ); @@ -1745,11 +1752,52 @@ HRESULT WINAPI ResizePseudoConsole( HPCON handle, COORD size ) return E_NOTIMPL; } +static BOOL is_tty_handle( HANDLE handle ) +{ + return ((UINT_PTR)handle & 3) == 1; +} + void init_console( void ) { RTL_USER_PROCESS_PARAMETERS *params = RtlGetCurrentPeb()->ProcessParameters; - if (params->ConsoleHandle == CONSOLE_HANDLE_ALLOC) + if (params->ConsoleHandle == CONSOLE_HANDLE_SHELL) + { + HANDLE tty_in = NULL, tty_out = NULL, process = NULL; + COORD size; + + if (is_tty_handle( params->hStdInput )) + { + tty_in = params->hStdInput; + params->hStdInput = NULL; + } + if (is_tty_handle( params->hStdOutput )) + { + tty_out = params->hStdOutput; + params->hStdOutput = NULL; + } + if (is_tty_handle( params->hStdError )) + { + if (tty_out) CloseHandle( params->hStdError ); + else tty_out = params->hStdError; + params->hStdError = NULL; + } + + size.X = params->dwXCountChars; + size.Y = params->dwYCountChars; + TRACE( "creating unix console (size %u %u)\n", size.X, size.Y ); + params->ConsoleHandle = create_pseudo_console( size, tty_in, tty_out, NULL, 0, &process ); + CloseHandle( process ); + CloseHandle( tty_in ); + CloseHandle( tty_out ); + + if (params->ConsoleHandle && create_console_connection( params->ConsoleHandle )) + { + init_console_std_handles( FALSE ); + console_flags = 0; + } + } + else if (params->ConsoleHandle == CONSOLE_HANDLE_ALLOC) { HMODULE mod = GetModuleHandleW( NULL ); params->ConsoleHandle = NULL; diff --git a/dlls/ntdll/unix/env.c b/dlls/ntdll/unix/env.c index f4d9ef15fc..876dcf0192 100644 --- a/dlls/ntdll/unix/env.c +++ b/dlls/ntdll/unix/env.c @@ -1156,12 +1156,29 @@ NTSTATUS CDECL get_dynamic_environment( WCHAR *env, SIZE_T *size ) void CDECL get_initial_console( RTL_USER_PROCESS_PARAMETERS *params ) { int output_fd = -1; - if (isatty(0) || isatty(1) || isatty(2)) params->ConsoleHandle = CONSOLE_HANDLE_SHELL; - if (!isatty(0)) wine_server_fd_to_handle( 0, GENERIC_READ|SYNCHRONIZE, OBJ_INHERIT, &params->hStdInput ); - if (!isatty(2)) wine_server_fd_to_handle( 2, GENERIC_WRITE|SYNCHRONIZE, OBJ_INHERIT, &params->hStdError ); - else output_fd = 2; - if (!isatty(1)) wine_server_fd_to_handle( 1, GENERIC_WRITE|SYNCHRONIZE, OBJ_INHERIT, &params->hStdOutput ); - else output_fd = 1; + + wine_server_fd_to_handle( 0, GENERIC_READ|SYNCHRONIZE, OBJ_INHERIT, &params->hStdInput ); + wine_server_fd_to_handle( 1, GENERIC_WRITE|SYNCHRONIZE, OBJ_INHERIT, &params->hStdOutput ); + wine_server_fd_to_handle( 2, GENERIC_WRITE|SYNCHRONIZE, OBJ_INHERIT, &params->hStdError ); + + /* mark tty handles for kernelbase, see init_console */ + if (params->hStdInput && isatty(0)) + { + params->ConsoleHandle = CONSOLE_HANDLE_SHELL; + params->hStdInput = (HANDLE)((UINT_PTR)params->hStdInput | 1); + } + if (params->hStdError && isatty(2)) + { + params->ConsoleHandle = CONSOLE_HANDLE_SHELL; + params->hStdError = (HANDLE)((UINT_PTR)params->hStdError | 1); + output_fd = 2; + } + if (params->hStdOutput && isatty(1)) + { + params->ConsoleHandle = CONSOLE_HANDLE_SHELL; + params->hStdOutput = (HANDLE)((UINT_PTR)params->hStdOutput | 1); + output_fd = 1; + } if (output_fd != -1) {

1 0

Jacek Caban : kernelbase: Move force override logic to init_console_std_handles callers.
by Alexandre Julliard Sept. 21, 2020

Sept. 21, 2020

Module: wine Branch: master Commit: ef57a8df8b53fcacf769a28bb9f4e77317ce7167 URL: https://source.winehq.org/git/wine.git/?a=commit;h=ef57a8df8b53fcacf769a28b… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Sep 21 17:06:50 2020 +0200 kernelbase: Move force override logic to init_console_std_handles callers. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernelbase/console.c | 18 +++++++++++------- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git a/dlls/kernelbase/console.c b/dlls/kernelbase/console.c index 8c39cde30a..2289d7ea5f 100644 --- a/dlls/kernelbase/console.c +++ b/dlls/kernelbase/console.c @@ -224,20 +224,18 @@ static BOOL create_console_connection( HANDLE root ) return set_ntstatus( status ); } -static BOOL init_console_std_handles(void) +static BOOL init_console_std_handles( BOOL override_all ) { HANDLE std_out = NULL, std_err = NULL, handle; OBJECT_ATTRIBUTES attr = {sizeof(attr)}; IO_STATUS_BLOCK iosb; UNICODE_STRING name; - STARTUPINFOW si; NTSTATUS status; - GetStartupInfoW( &si ); attr.ObjectName = &name; attr.Attributes = OBJ_INHERIT; - if (!(si.dwFlags & STARTF_USESTDHANDLES) || !GetStdHandle( STD_INPUT_HANDLE )) + if (override_all || !GetStdHandle( STD_INPUT_HANDLE )) { /* FIXME: Use unbound console handle */ RtlInitUnicodeString( &name, L"\\Device\\ConDrv\\CurrentIn" ); @@ -250,7 +248,7 @@ static BOOL init_console_std_handles(void) SetStdHandle( STD_INPUT_HANDLE, console_handle_map( handle )); } - if (si.dwFlags & STARTF_USESTDHANDLES) + if (!override_all) { std_out = GetStdHandle( STD_OUTPUT_HANDLE ); std_err = GetStdHandle( STD_ERROR_HANDLE ); @@ -306,7 +304,13 @@ BOOL WINAPI DECLSPEC_HOTPATCH AttachConsole( DWORD pid ) if (ret) { RtlGetCurrentPeb()->ProcessParameters->ConsoleHandle = create_console_reference( console_connection ); - ret = RtlGetCurrentPeb()->ProcessParameters->ConsoleHandle && init_console_std_handles(); + if (RtlGetCurrentPeb()->ProcessParameters->ConsoleHandle) + { + STARTUPINFOW si; + GetStartupInfoW( &si ); + init_console_std_handles( !(si.dwFlags & STARTF_USESTDHANDLES) ); + } + else ret = FALSE; } if (!ret) FreeConsole(); @@ -379,7 +383,7 @@ BOOL WINAPI AllocConsole(void) CloseHandle( pi.hProcess ); } CloseHandle( event ); - if (!ret || !init_console_std_handles()) goto error; + if (!ret || !init_console_std_handles( !(app_si.dwFlags & STARTF_USESTDHANDLES) )) goto error; console = CreateFileW( L"CONIN$", GENERIC_READ | GENERIC_WRITE | SYNCHRONIZE, 0, NULL, OPEN_EXISTING, 0, 0 ); if (console == INVALID_HANDLE_VALUE) goto error; RtlGetCurrentPeb()->ProcessParameters->ConsoleHandle = console;

1 0

Jacek Caban : conhost: Use IOCTL_CONDRV_SETUP_INPUT to initialize input in Unix mode.
by Alexandre Julliard Sept. 21, 2020

Sept. 21, 2020

Module: wine Branch: master Commit: c47f64bd3e5ca6e10096dc9b39d3df2cc9be95a0 URL: https://source.winehq.org/git/wine.git/?a=commit;h=c47f64bd3e5ca6e10096dc9b… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Sep 21 17:06:37 2020 +0200 conhost: Use IOCTL_CONDRV_SETUP_INPUT to initialize input in Unix mode. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- programs/conhost/conhost.c | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/programs/conhost/conhost.c b/programs/conhost/conhost.c index 0de55af14c..c637b16379 100644 --- a/programs/conhost/conhost.c +++ b/programs/conhost/conhost.c @@ -1625,6 +1625,14 @@ static DWORD WINAPI tty_input( void *param ) BOOL signaled; NTSTATUS status; + if (console->is_unix) + { + unsigned int h = condrv_handle( console->tty_input ); + status = NtDeviceIoControlFile( console->server, NULL, NULL, NULL, &io, IOCTL_CONDRV_SETUP_INPUT, + &h, sizeof(h), NULL, 0 ); + if (status) ERR( "input setup failed: %#x\n", status ); + } + event = CreateEventW( NULL, TRUE, FALSE, NULL ); for (;;) @@ -1682,6 +1690,13 @@ static DWORD WINAPI tty_input( void *param ) EnterCriticalSection( &console_section ); if (console->read_ioctl) read_complete( console, status, NULL, 0, FALSE ); + if (console->is_unix) + { + unsigned int h = 0; + status = NtDeviceIoControlFile( console->server, NULL, NULL, NULL, &io, IOCTL_CONDRV_SETUP_INPUT, + &h, sizeof(h), NULL, 0 ); + if (status) ERR( "input restore failed: %#x\n", status ); + } CloseHandle( console->input_thread ); console->input_thread = NULL; LeaveCriticalSection( &console_section );

1 0

Jacek Caban : conhost: Force using relative cursor positioning in update_read_output.
by Alexandre Julliard Sept. 21, 2020

Sept. 21, 2020

Module: wine Branch: master Commit: fc408f3d4376722dbe173495be10ef2ce34874d1 URL: https://source.winehq.org/git/wine.git/?a=commit;h=fc408f3d4376722dbe173495… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Sep 21 17:06:03 2020 +0200 conhost: Force using relative cursor positioning in update_read_output. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- programs/conhost/conhost.c | 32 +++++++++++++++++++++++++++++++- 1 file changed, 31 insertions(+), 1 deletion(-) diff --git a/programs/conhost/conhost.c b/programs/conhost/conhost.c index 97855e5e32..3f7b5fb914 100644 --- a/programs/conhost/conhost.c +++ b/programs/conhost/conhost.c @@ -309,6 +309,27 @@ static void set_tty_cursor( struct console *console, unsigned int x, unsigned in tty_write( console, buf, strlen(buf) ); } +static void set_tty_cursor_relative( struct console *console, unsigned int x, unsigned int y ) +{ + if (y < console->tty_cursor_y) + { + char buf[64]; + sprintf( buf, "\x1b[%uA", console->tty_cursor_y - y ); + tty_write( console, buf, strlen(buf) ); + console->tty_cursor_y = y; + } + else + { + while (console->tty_cursor_y < y) + { + console->tty_cursor_x = 0; + console->tty_cursor_y++; + tty_write( console, "\r\n", 2 ); + } + } + set_tty_cursor( console, x, y ); +} + static void set_tty_attr( struct console *console, unsigned int attr ) { char buf[8]; @@ -1216,7 +1237,16 @@ static void update_read_output( struct console *console ) } } - update_output( screen_buffer, &update_rect ); + /* always try to use relative cursor positions in UNIX mode so that it works even if cursor + * position is out of sync */ + if (update_rect.left <= update_rect.right && update_rect.top <= update_rect.bottom) + { + if (console->is_unix) + set_tty_cursor_relative( screen_buffer->console, update_rect.left, update_rect.top ); + update_output( screen_buffer, &update_rect ); + } + if (console->is_unix) + set_tty_cursor_relative( screen_buffer->console, screen_buffer->cursor_x, screen_buffer->cursor_y ); tty_sync( screen_buffer->console ); }

1 0

Jacek Caban : conhost: Introduce Unix mode.
by Alexandre Julliard Sept. 21, 2020

Sept. 21, 2020

Module: wine Branch: master Commit: ef5af87f6cc951b29800bcaf97bc1d9c86283b4a URL: https://source.winehq.org/git/wine.git/?a=commit;h=ef5af87f6cc951b29800bcaf… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Sep 21 17:05:41 2020 +0200 conhost: Introduce Unix mode. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- programs/conhost/conhost.c | 28 +++++++++++++++++++++------- 1 file changed, 21 insertions(+), 7 deletions(-) diff --git a/programs/conhost/conhost.c b/programs/conhost/conhost.c index 1d48194206..5df6848ba5 100644 --- a/programs/conhost/conhost.c +++ b/programs/conhost/conhost.c @@ -79,6 +79,7 @@ struct console HANDLE server; /* console server handle */ unsigned int mode; /* input mode */ struct screen_buffer *active; /* active screen buffer */ + int is_unix; /* UNIX terminal mode */ INPUT_RECORD *records; /* input records */ unsigned int record_count; /* number of input records */ unsigned int record_size; /* size of input records buffer */ @@ -356,10 +357,14 @@ static void tty_sync( struct console *console ) static void init_tty_output( struct console *console ) { - /* initialize tty output, but don't flush */ - tty_write( console, "\x1b[2J", 4 ); /* clear screen */ - set_tty_attr( console, console->active->attr ); - tty_write( console, "\x1b[H", 3 ); /* move cursor to (0,0) */ + if (!console->is_unix) + { + /* initialize tty output, but don't flush */ + tty_write( console, "\x1b[2J", 4 ); /* clear screen */ + set_tty_attr( console, console->active->attr ); + tty_write( console, "\x1b[H", 3 ); /* move cursor to (0,0) */ + } + else console->tty_attr = empty_char_info.attr; console->tty_cursor_visible = TRUE; } @@ -2641,7 +2646,7 @@ static int main_loop( struct console *console, HANDLE signal ) int __cdecl wmain(int argc, WCHAR *argv[]) { - int headless = 0, i, width = 80, height = 150; + int headless = 0, i, width = 0, height = 0; HANDLE signal = NULL, input_thread; WCHAR *end; @@ -2664,18 +2669,24 @@ int __cdecl wmain(int argc, WCHAR *argv[]) headless = 1; continue; } + if (!wcscmp( argv[i], L"--unix")) + { + console.is_unix = 1; + headless = 1; + continue; + } if (!wcscmp( argv[i], L"--width" )) { if (++i == argc) return 1; width = wcstol( argv[i], &end, 0 ); - if (!width || width > 0xffff || *end) return 1; + if ((!width && !console.is_unix) || width > 0xffff || *end) return 1; continue; } if (!wcscmp( argv[i], L"--height" )) { if (++i == argc) return 1; height = wcstol( argv[i], &end, 0 ); - if (!height || height > 0xffff || *end) return 1; + if ((!height && !console.is_unix) || height > 0xffff || *end) return 1; continue; } if (!wcscmp( argv[i], L"--signal" )) @@ -2708,6 +2719,9 @@ int __cdecl wmain(int argc, WCHAR *argv[]) return 1; } + if (!width) width = 80; + if (!height) height = 150; + if (!(console.active = create_screen_buffer( &console, 1, width, height ))) return 1; if (headless) {

1 0

Jacek Caban : conhost: Signal console in tty_input if needed.
by Alexandre Julliard Sept. 21, 2020

Sept. 21, 2020

Module: wine Branch: master Commit: a33594a0483be2fe30d6cff8e523a9bb208b7240 URL: https://source.winehq.org/git/wine.git/?a=commit;h=a33594a0483be2fe30d6cff8… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Sep 21 17:05:26 2020 +0200 conhost: Signal console in tty_input if needed. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- programs/conhost/conhost.c | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/programs/conhost/conhost.c b/programs/conhost/conhost.c index 5f6065eab5..a9ad38eb99 100644 --- a/programs/conhost/conhost.c +++ b/programs/conhost/conhost.c @@ -473,7 +473,7 @@ static NTSTATUS read_complete( struct console *console, NTSTATUS status, const v status = wine_server_call( req ); } SERVER_END_REQ; - if (status) ERR( "failed: %#x\n", status ); + if (status && (console->read_ioctl || status != STATUS_INVALID_HANDLE)) ERR( "failed: %#x\n", status ); console->read_ioctl = 0; console->pending_read = 0; return status; @@ -1580,6 +1580,7 @@ static DWORD WINAPI tty_input( void *param ) char read_buf[4096]; WCHAR buf[4096]; DWORD count, i; + BOOL signaled; NTSTATUS status; event = CreateEventW( NULL, TRUE, FALSE, NULL ); @@ -1595,6 +1596,7 @@ static DWORD WINAPI tty_input( void *param ) if (status) break; EnterCriticalSection( &console_section ); + signaled = console->record_count != 0; /* FIXME: Handle partial char read */ count = MultiByteToWideChar(CP_UTF8, 0, read_buf, io.Information, buf, ARRAY_SIZE(buf)); @@ -1626,6 +1628,11 @@ static DWORD WINAPI tty_input( void *param ) } process_console_input( console ); + if (!signaled && console->record_count) + { + assert( !console->read_ioctl ); + read_complete( console, STATUS_SUCCESS, NULL, 0, TRUE ); /* signal console */ + } LeaveCriticalSection( &console_section ); }

1 0

Jacek Caban : conhost: Explicitly set tty cursor position after updating output in write_console.
by Alexandre Julliard Sept. 21, 2020

Sept. 21, 2020

Module: wine Branch: master Commit: da3bd4cb4a1c9644717547e0453ccd3498a3ceb9 URL: https://source.winehq.org/git/wine.git/?a=commit;h=da3bd4cb4a1c9644717547e0… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Sep 21 17:05:21 2020 +0200 conhost: Explicitly set tty cursor position after updating output in write_console. This avoids absolute cursor position jumps. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- programs/conhost/conhost.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/programs/conhost/conhost.c b/programs/conhost/conhost.c index 422a966eb2..5f6065eab5 100644 --- a/programs/conhost/conhost.c +++ b/programs/conhost/conhost.c @@ -1876,7 +1876,10 @@ static NTSTATUS write_console( struct screen_buffer *screen_buffer, const WCHAR if (++screen_buffer->cursor_y == screen_buffer->height) new_line( screen_buffer, &update_rect ); else if (screen_buffer->mode & ENABLE_WRAP_AT_EOL_OUTPUT) + { update_output( screen_buffer, &update_rect ); + set_tty_cursor( screen_buffer->console, screen_buffer->cursor_x, screen_buffer->cursor_y ); + } continue; case '\a': FIXME( "beep\n" );

1 0