[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 2/3] hw/s390x/css: Remove QEMU_PACKED from st
From: |
Cornelia Huck |
Subject: |
Re: [Qemu-devel] [PATCH v2 2/3] hw/s390x/css: Remove QEMU_PACKED from struct SenseId |
Date: |
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-devel] [PATCH v2 2/3] hw/s390x/css: Remove QEMU_PACKED from struct SenseId, Cornelia Huck, 2018/09/26
[Qemu-devel] [PATCH v2 3/3] hw/s390x/ioinst: Fix alignment problem in struct SubchDev, Thomas Huth, 2018/09/26