<div><div dir="auto">If the patch requires staging patch sets wouldn’t it make sense to carry the patch in staging for the time being?</div></div><div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Sep 1, 2021 at 10:43 AM Eric Pouech <<a href="mailto:eric.pouech@orange.fr" target="_blank">eric.pouech@orange.fr</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)">
<div>
<div>Le 31/08/2021 à 15:32, Alexandre
Julliard a écrit :<br>
</div>
<blockquote type="cite">
<pre style="font-family:monospace">Eric Pouech <a href="mailto:eric.pouech@orange.fr" style="font-family:monospace" target="_blank"><eric.pouech@orange.fr></a> writes:
</pre>
<blockquote type="cite">
<blockquote type="cite">
<pre style="font-family:monospace">Offsetting by 0x1000 is fine.
But I'd say that the ELF data should be stored in the 64-bit PEB. Also
note that once the new Wow64 architecture is enabled, all ELF data
inside 32-bit processes is going to be 64-bit.
</pre>
</blockquote>
<pre style="font-family:monospace">ok I'll resend the serie with just offsetting for now.
is there a way to test the new WoW64 arch before it's enabled?
</pre>
</blockquote>
<pre style="font-family:monospace">You can try the patch below. Note that you need to apply the
ntdll-NtAlertThreadByThreadId series from wine-staging first.
</pre>
</blockquote>
<p><font face="Helvetica, Arial, sans-serif" style="font-family:Helvetica,Arial,sans-serif;color:rgb(0,0,0)">thanks</font></p>
<p><font face="Helvetica, Arial, sans-serif" style="font-family:Helvetica,Arial,sans-serif;color:rgb(0,0,0)"><br>
</font></p>
<p><font face="Helvetica, Arial, sans-serif" style="font-family:Helvetica,Arial,sans-serif;color:rgb(0,0,0)">didn't run into
anything dramatic ;-)</font></p>
<p><font face="Helvetica, Arial, sans-serif" style="font-family:Helvetica,Arial,sans-serif;color:rgb(0,0,0)">no specific issues on
dbghelp testing on my side</font></p>
<p><font face="Helvetica, Arial, sans-serif" style="font-family:Helvetica,Arial,sans-serif;color:rgb(0,0,0)"><br>
</font></p>
<p><font face="Helvetica, Arial, sans-serif" style="font-family:Helvetica,Arial,sans-serif;color:rgb(0,0,0)">as I understand the new
DLL scheme (for a WOW64 DLL on 64bit cpu)</font></p>
<p><font face="Helvetica, Arial, sans-serif" style="font-family:Helvetica,Arial,sans-serif;color:rgb(0,0,0)">- wine loader will
always be a 64bit ELF<br>
</font></p>
<p><font face="Helvetica, Arial, sans-serif" style="font-family:Helvetica,Arial,sans-serif;color:rgb(0,0,0)">- all ELF required libs
will be loaded as 64bit ELF into that process</font></p>
<p><font face="Helvetica, Arial, sans-serif" style="font-family:Helvetica,Arial,sans-serif;color:rgb(0,0,0)">- PE module will be
32bit PE but with thunks to 64bit code for interaction with ELF
DLLs</font></p>
<p><font face="Helvetica, Arial, sans-serif" style="font-family:Helvetica,Arial,sans-serif;color:rgb(0,0,0)">one of the current
interest of winedbg was the ability to show both the PE modules
and the ELF modules and debug both</font></p>
<p><font face="Helvetica, Arial, sans-serif" style="font-family:Helvetica,Arial,sans-serif;color:rgb(0,0,0)">this will open a nice
can of worms as it would require multiple CPUs support for the
same process :-(</font></p>
<p><font face="Helvetica, Arial, sans-serif" style="font-family:Helvetica,Arial,sans-serif;color:rgb(0,0,0)"><br>
</font></p>
<font face="Helvetica, Arial, sans-serif" style="font-family:Helvetica,Arial,sans-serif;color:rgb(0,0,0)">MS debugger has the
ability to switch back and forth between the two worlds (ie set
CPU), but requires specific command to display a combined (mutli
CPU) backtrace<br>
</font>
<p><font face="Helvetica, Arial, sans-serif" style="font-family:Helvetica,Arial,sans-serif;color:rgb(0,0,0)"><br>
</font></p>
<p><font face="Helvetica, Arial, sans-serif" style="font-family:Helvetica,Arial,sans-serif;color:rgb(0,0,0)">note: your patch
doesn't apply cleanly on top of wine-staging/</font>ntdll-NtAlertThreadByThreadId</p>
<p>(some stuff in common on the wine-staging series and the patch ;
I hope I got the conflict resolution right)</p>
<p><br>
</p>
<p>A+<br>
</p>
<p><br>
</p>
</div>
</blockquote></div></div>
</div>