[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v8] hw/arm/virt: Add high MMIO PCI region, 512G
From: |
Pavel Fedin |
Subject: |
Re: [Qemu-devel] [PATCH v8] hw/arm/virt: Add high MMIO PCI region, 512G in size |
Date: |
Fri, 04 Sep 2015 10:13:56 +0300 |
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