Re: [Spice-devel] [PATCH] demarshall: Add safe method for unaligned byte access.

2015-12-24 Thread Frediano Ziglio
Yes, I posted here http://lists.freedesktop.org/archives/spice-devel/2015-December/024882.html Frediano > > Hello, Frediano. > > Have you a chance to make such patch? > > Thank you. > > 10.12.2015, 23:30, "Frediano Ziglio" : > > Hi, > >   sorry, I looked more carefully at generated code and t

Re: [Spice-devel] [PATCH] demarshall: Add safe method for unaligned byte access.

2015-12-23 Thread Anton D . Kachalov
Hello, Frediano. Have you a chance to make such patch? Thank you. 10.12.2015, 23:30, "Frediano Ziglio" : > Hi, >   sorry, I looked more carefully at generated code and the current code is not > enough to make the -mno-unaligned-access option works as expected. > It does not globally enable align

Re: [Spice-devel] [PATCH] demarshall: Add safe method for unaligned byte access.

2015-12-10 Thread Frediano Ziglio
> > Hello, Frediano. > > Option "-mno-unaligned-access" doesn't help. Got a lot of "Alignment trap". > No additional code such as __aeabi_uread4() has added by the compiler. No > runtime checks. Looks like it just ignored. GCC 4.8.2. > > > http://sourceforge.net/p/tianocore/edk2/ci/68085d37f

Re: [Spice-devel] [PATCH] demarshall: Add safe method for unaligned byte access.

2015-12-03 Thread Anton D . Kachalov
Hello, Frediano. Option "-mno-unaligned-access" doesn't help. Got a lot of "Alignment trap". No additional code such as __aeabi_uread4() has added by the compiler. No runtime checks. Looks like it just ignored. GCC 4.8.2. http://sourceforge.net/p/tianocore/edk2/ci/68085d37f88ad97f071e2d

Re: [Spice-devel] [PATCH] demarshall: Add safe method for unaligned byte access.

2015-11-27 Thread Frediano Ziglio
> > Usefull for ARM. Enable with "USE_MEMCPY" define. > > Should be better to add "--use-memcpy" to configure.ac > > > > Signed-off-by: Anton D. Kachalov > > --- > > python_modules/demarshal.py | 29 +++-- > > 1 file changed, 19 insertions(+), 10 deletions(-) > > > > d

Re: [Spice-devel] [PATCH] demarshall: Add safe method for unaligned byte access.

2015-11-27 Thread Frediano Ziglio
> > Usefull for ARM. Enable with "USE_MEMCPY" define. > Should be better to add "--use-memcpy" to configure.ac > > Signed-off-by: Anton D. Kachalov > --- > python_modules/demarshal.py | 29 +++-- > 1 file changed, 19 insertions(+), 10 deletions(-) > > diff --git a/pyth

Re: [Spice-devel] [PATCH] demarshall: Add safe method for unaligned byte access.

2015-11-27 Thread Frediano Ziglio
> > On Fri, Nov 27, 2015 at 07:04:55AM -0500, Frediano Ziglio wrote: > > > > > > On Wed, Nov 25, 2015 at 09:04:58AM -0500, Frediano Ziglio wrote: > > > > > > > > > > Usefull for ARM. Enable with "USE_MEMCPY" define. > > > > > Should be better to add "--use-memcpy" to configure.ac > > > > > > >

Re: [Spice-devel] [PATCH] demarshall: Add safe method for unaligned byte access.

2015-11-27 Thread Christophe Fergeau
On Fri, Nov 27, 2015 at 07:04:55AM -0500, Frediano Ziglio wrote: > > > > On Wed, Nov 25, 2015 at 09:04:58AM -0500, Frediano Ziglio wrote: > > > > > > > > Usefull for ARM. Enable with "USE_MEMCPY" define. > > > > Should be better to add "--use-memcpy" to configure.ac > > > > > > > > Signed-off-b

Re: [Spice-devel] [PATCH] demarshall: Add safe method for unaligned byte access.

2015-11-27 Thread Frediano Ziglio
> > On Wed, Nov 25, 2015 at 09:04:58AM -0500, Frediano Ziglio wrote: > > > > > > Usefull for ARM. Enable with "USE_MEMCPY" define. > > > Should be better to add "--use-memcpy" to configure.ac > > > > > > Signed-off-by: Anton D. Kachalov > > > > Are you sure there are not other way except memc

Re: [Spice-devel] [PATCH] demarshall: Add safe method for unaligned byte access.

2015-11-27 Thread Christophe Fergeau
On Wed, Nov 25, 2015 at 09:04:58AM -0500, Frediano Ziglio wrote: > > > > Usefull for ARM. Enable with "USE_MEMCPY" define. > > Should be better to add "--use-memcpy" to configure.ac > > > > Signed-off-by: Anton D. Kachalov > > Are you sure there are not other way except memcpy? An alternate w

Re: [Spice-devel] [PATCH] demarshall: Add safe method for unaligned byte access.

2015-11-25 Thread Frediano Ziglio
> > Usefull for ARM. Enable with "USE_MEMCPY" define. > Should be better to add "--use-memcpy" to configure.ac > > Signed-off-by: Anton D. Kachalov Are you sure there are not other way except memcpy? > --- > python_modules/demarshal.py | 29 +++-- > 1 file changed, 19

[Spice-devel] [PATCH] demarshall: Add safe method for unaligned byte access.

2015-11-25 Thread Anton D . Kachalov
Usefull for ARM. Enable with "USE_MEMCPY" define. Should be better to add "--use-memcpy" to configure.ac Signed-off-by: Anton D. Kachalov --- python_modules/demarshal.py | 29 +++-- 1 file changed, 19 insertions(+), 10 deletions(-) diff --git a/python_modules/demarshal