[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [qemu-s390x] [PATCH v2 2/3] hw/s390x/css: Remove QEMU_PACKED from st
Re: [qemu-s390x] [PATCH v2 2/3] hw/s390x/css: Remove QEMU_PACKED from struct SenseId
Wed, 26 Sep 2018 18:52:11 +0200
On Wed, 26 Sep 2018 12:05:37 +0200
Thomas Huth <address@hidden> wrote:
> On 2018-09-26 11:53, Cornelia Huck wrote:
> > It seems our only choice is which kind of ugly we prefer when fixing
> > this issue... at least the usage of stw_be_p makes this look a bit
> > better :)
> > But maybe add a comment
> > /*
> > * As the SenseId struct cannot be packed (would cause unaligned
> > * accesses), we have to copy the individual fields to an unstructured
> > * area using the correct layout.
> > */
> > so that we don't wonder why it looks like this in the future?
> Sure, could you add it when picking up the patch? Or shall I respin?
At that point in time, I'm confused enough to prefer a respin so that I
catch everything :)
Re: [qemu-s390x] [PATCH v2 2/3] hw/s390x/css: Remove QEMU_PACKED from struct SenseId, Cornelia Huck, 2018/09/26
[qemu-s390x] [PATCH v2 3/3] hw/s390x/ioinst: Fix alignment problem in struct SubchDev, Thomas Huth, 2018/09/26