Re: [Qemu-devel] [PATCH master/stable-1.0] pci: fix corrupted pci conf index register by unaligned write

[Michael S. Tsirkin]

On Sun, Jan 08, 2012 at 12:48:05PM +0200, Avi Kivity wrote:
> On 01/08/2012 12:12 PM, Michael S. Tsirkin wrote:
> > > >
> > > > > And the check for unaligned writes is, well, unusual :-)
> > > 
> > > What's unusual?
> > > 
> > > 
> > > > This seems to be how memory API behaves ... right, Avi?
> > > > Maybe this should be documented somewhere.
> > > 
> > > Document what?
> >
> > That address passed to callbacks is in fact an offset
> > from start of the region.
> >
> 
> It's documented already (and it's not new - since 8da3ff18097, Dec 2008).

True, memory.h explicitly says:
@addr is relative to @mr

But now that's mentioned, I have a question:
we could also specify min access size and disable
unaligned access in ops.valid structure.
That would trigger machine checks on access errors
instead of ignoring them as we do now.
Anyone knows offhand what does real hardware do?

> -- 
> error compiling committee.c: too many arguments to function