Re: [PATCH spice-protocol] qxl_dev: Align QXLRam to 4 bytes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> 
> On Tue, Jul 03, 2018 at 11:38:56AM -0400, Frediano Ziglio wrote:
> > > 
> > > On Thu, Jun 28, 2018 at 09:43:47PM +0100, Frediano Ziglio wrote:
> > > > This avoids compilers detect misaligned access.
> > > 
> > > Maybe "This helps some compilers to realize QXLRam is correctly aligned"?
> > > 
> > 
> > Well, the current code tells the compiler that this structure
> > is NOT aligned.
> > But we actually know the opposite
> 
> Well, we know that in QEMU and Xspice, this is correctly aligned (but
> one need to look at how ram offset is computed in order to see that), so
> we add the annotation to spice-protocol. Since this is quite odd, the
> commit log would indeed need to be quite descriptive.
> 
> Christophe
> 

Suggestions?

Frediano
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux Virtualization]     [Linux Virtualization]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]