<div dir="ltr"><div dir="ltr">Em seg., 15 de ago. de 2022 às 05:05, David Holmes <<a href="mailto:david.holmes@oracle.com">david.holmes@oracle.com</a>> escreveu:<br></div><div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
On 13/08/2022 2:05 am, Maurizio Cimadamore wrote:<br>
> If this is a problem with the debugger accidentally overwriting <br>
> LastError on some platforms, I think it would be perhaps better to fix <br>
> it at that level (given that our JVM has some code to defend against <br>
> similar accidental overwrites, I'm assuming that it is possible for the <br>
> JVM code/debugger code to act in a "transparent" fashion).<br>
<br>
PreserveLastError is just a helper for logging, so that the I/O from the <br>
logging doesn't overwrite any error from the real native operation.<br></blockquote><div><br></div><div>I think this is the correct general solution.</div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Fixing this in general would require that we define the required <br>
thread-local state to be preserved, and then save that after each native <br>
method invocation and restore it before the next. But that in itself <br>
would limit the effectiveness as it can't distinguish between <br>
application level native methods and library level (including things <br>
triggered from the VM). </blockquote><div><br></div><div>Consider this code:</div><div><br></div><div>write(...)</div><div>call-into-some-library-function(...)</div><div>log(last-error())</div><div><br></div><div>This is not the generally correct way to observe the status of write.</div><div>It is not, generally, a library's responsibility to preserve last-error.</div><div><br></div><div>The generally correct way is to observe the last-error immediately.</div><div>What matters is that the code below observes the status of write.</div><div><br></div><div><div>write(...)</div><div>log(last-error())</div><div><br></div><div>If there is some code running somewhere with the power of affecting last-error on this thread, that code must be very careful to preserve it.</div></div><div>It has been supposed that the OP's debugger is doing I/O from inside the application's thread;</div><div>if that is the case, then this debugger must do something like PreserveLastError.</div><div><br></div><div>--<br></div></div><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Pedro Lamarão</div></div></div></div>