<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>