<html><head><meta http-equiv="content-type" content="text/html; charset=utf-8"></head><body dir="auto"><div>Hi all,</div><div><br></div><div>I just test on the Power8 system and It works !!!!!!!</div><div><br></div><div>Many thanks to all of you for your help; Especially to Mihai "the Endian bug killer" for his fantastic work. </div><div><br><div><span style="background-color: rgba(255, 255, 255, 0);">--- <br>Sébastien Chabrolles <br>Power Systems Benchmark Specialist<br>IBM Client Center, Montpellier <br><a href="x-apple-data-detectors://1/0" x-apple-data-detectors="true" x-apple-data-detectors-type="address" x-apple-data-detectors-result="1/0">1 rue vieille poste 34000 Montpellier FRANCE</a> <br>Tel <a href="tel:+33%204%2067%2034%2040%2095" x-apple-data-detectors="true" x-apple-data-detectors-type="telephone" x-apple-data-detectors-result="1/1">+33 4 67 34 40 95</a> <br></span><div><span style="background-color: rgba(255, 255, 255, 0);">Email : <a href="mailto:loic.fura@fr.ibm.com">s.chabrolles@fr.ibm.com</a></span></div></div></div><div><br>Le 21 juin 2014 à 22:30, "Mike Gabriel" <<a href="mailto:mike.gabriel@das-netzwerkteam.de">mike.gabriel@das-netzwerkteam.de</a>> a écrit :<br><br></div><blockquote type="cite"><div>
<pre>Hi Mihai,
thanks for digging into this. I wish you could work more on NX!!! Good work!!!
On Sa 21 Jun 2014 05:15:43 CEST, Mihai Moldovan wrote:
> An endiannes issue was setting incorrect event masks when creating
> X11 windows.
>
> This time, a smaller integer has been casted to a bigger one and
> passed to some
> function actually setting its value.
>
> This meant, that garbage from stack was attached to the smaller
> integer value,
> putting unknown memory into the lower bytes of the bigger integer.
>
> Fix this by creating a big, initialized temporary variable, let the
> function do
> its magic on that one and pass the value back to the smaller variable -- and
> cross your fingers the smaller variable can hold it without overrunning. (The
> last bit is a design issue we can't really fix and has been around
> even before
> this patch.)
I have a question on this patch. While integrating it into the
nx-libs.git repo, I realized, that it can be applied to Windows.c
twice. Once at the position that you provide in the patch file,
another (second) time around line 2895...
"""
mike@minobo:~/MyDocuments/4projects/x2go-upstream/nx-libs$ patch -p1 <
nxagent_Window.c-ppc64-create-windows.patch
patching file nx-X11/programs/Xserver/hw/nxagent/Window.c
Hunk #1 succeeded at 2894 (offset 2559 lines).
"""
Maybe the same patch is needed at that second position?
Mike
--
DAS-NETZWERKTEAM
mike gabriel, herweg 7, 24357 fleckeby
fon: +49 (1520) 1976 148
GnuPG Key ID 0x25771B31
mail: <a href="mailto:mike.gabriel@das-netzwerkteam.de">mike.gabriel@das-netzwerkteam.de</a>, <a href="http://das-netzwerkteam.de">http://das-netzwerkteam.de</a>
freeBusy:
<a href="https://mail.das-netzwerkteam.de/freebusy/m.gabriel%40das-netzwerkteam.de.xfb">https://mail.das-netzwerkteam.de/freebusy/m.gabriel%40das-netzwerkteam.de.xfb</a>
</pre><pre>_______________________________________________
x2go-dev mailing list
<a href="mailto:x2go-dev@lists.x2go.org">x2go-dev@lists.x2go.org</a>
<a href="http://lists.x2go.org/listinfo/x2go-dev">http://lists.x2go.org/listinfo/x2go-dev</a></pre></div></blockquote></body></html>