Re: [Qemu-ppc] [Qemu-devel] [PATCH v2 1/2] ohw: correctly program BAR addresses of PCI devices

[Mark Cave-Ayland]

On 28/05/14 00:28, Andreas Färber wrote:

> Am 28.05.2014 01:22, schrieb Alexander Graf:
> > On 23.04.14 23:19, Hervé Poussineau wrote:
> > > Note that offsets are PReP ones, so this breaks compatibility with
> > > hardware which has different values.
> >
> > So that means that this patch breaks non-PReP? I don't think we want
> > that ;).
>
> Please note that these patches are for OHW, and -M prep is the only
> machine to still use OHW these days, with the QEMU counterparts touching
> only PReP AFAICS, so I see no reason not to fix OHW - would you rather
> guard that with some #ifdef to make it clearer?
>
> I'm not sure about the correctness of these patches, so I'm dependent on
> review and - more likely - need to finally carve some time for simply
> testing whether they break anything and, if not, apply them.

Given that Hervé has actually managed to boot real OSs with these 
patches, I'd be inclined to accept them.

Andreas, I remember asking before but can't remember if I got an answer 
or not as to what is in your PReP test image suite? If we can get the 
same images, then we can help you with testing :)


ATB,

Mark.