Re: [Qemu-devel] [PATCH v8] hw/arm/virt: Add high MMIO PCI region, 512G in size

[Pavel Fedin]

 Hi!

> Something in your patch submission path seems to be wrapping long
> lines. It would be nice if you could fix it -- I couldn't apply this
> patch without manually editing it first.

 Damn, stupid Outlook... I already set maximum width to ~100, ok, will increase 
it further.

> We should add a note to QEMU's changelog to mention that if
> they have a 32-bit kernel on the virt board and the PCI has
> stopped working then they need to use this option.
> (http://wiki.qemu.org/ChangeLog/2.5)

 I tried to, but "log in" page seems to be missing "register" link, so i cannot 
create an account.

> Did you report the bug where the pci controller driver
> fails to start if the second region is out of its range
> to the kernel mailing list? (It would be nice to be able
> to point to a kernel patch in the changelog too.)

 I didn't yet, because have to time to retest it. Well, OK, will do it. But, 
can be already irrelevant because probably this was my fault - in previous 
version of the patch i forgot to add "64-bit" flag to the resource. So need to 
retest. Just don't have non-LPAE kernel build around at the moment.

> Anyway, applied to target-arm.next.

 Thank you very much.

Kind regards,
Pavel Fedin
Expert Engineer
Samsung Electronics Research center Russia