Skip to content

Commit

Permalink
Cygwin: find_fast_cwd: don't run assembler checking code on ARM64
Browse files Browse the repository at this point in the history
https://cygwin.com/pipermail/cygwin/2024-February/255397.html
reports a crash on ARM64 probably related to checking x86_64
code on the x86_64 emulator on AArch64.

At least for testing, pull the code checking the host HW
up to be called before trying to evaluate assembler code.

Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
  • Loading branch information
github-cygwin authored and jeremyd2019 committed Feb 16, 2024
1 parent 44b5e4e commit 3326e2f
Showing 1 changed file with 12 additions and 15 deletions.
27 changes: 12 additions & 15 deletions winsup/cygwin/path.cc
Original file line number Diff line number Diff line change
Expand Up @@ -4861,29 +4861,26 @@ find_fast_cwd_pointer ()
static fcwd_access_t **
find_fast_cwd ()
{
USHORT emulated, hosted;
fcwd_access_t **f_cwd_ptr;

/* First check if we're running in WOW64 on ARM64 emulating AMD64. Skip
fetching FAST_CWD pointer as long as there's no solution for finding
it on that system. */
if (IsWow64Process2 (GetCurrentProcess (), &emulated, &hosted)
&& hosted == IMAGE_FILE_MACHINE_ARM64)
f_cwd_ptr = NULL;
else
/* Fetch the pointer but don't set the global fast_cwd_ptr yet. First
we have to make sure we know the version of the FAST_CWD structure
used on the system. */
fcwd_access_t **f_cwd_ptr = find_fast_cwd_pointer ();
f_cwd_ptr = find_fast_cwd_pointer ();
if (!f_cwd_ptr)
{
bool warn = 1;
USHORT emulated, hosted;

/* Check if we're running in WOW64 on ARM64 emulating AMD64. Skip
warning as long as there's no solution for finding the FAST_CWD
pointer on that system. */
if (IsWow64Process2 (GetCurrentProcess (), &emulated, &hosted)
&& hosted == IMAGE_FILE_MACHINE_ARM64)
warn = 0;

if (warn)
small_printf ("Cygwin WARNING:\n"
small_printf ("Cygwin WARNING:\n"
" Couldn't compute FAST_CWD pointer. This typically occurs if you're using\n"
" an older Cygwin version on a newer Windows. Please update to the latest\n"
" available Cygwin version from https://cygwin.com/. If the problem persists,\n"
" please see https://cygwin.com/problems.html\n\n");
}
if (f_cwd_ptr && *f_cwd_ptr)
{
/* Just evaluate structure version. */
Expand Down

0 comments on commit 3326e2f

Please sign in to comment.