From: Peter Maydell <peter.maydell@linaro.org>
To: Peter Crosthwaite <crosthwaitepeter@gmail.com>
Cc: Pavel Fedin <p.fedin@samsung.com>,
QEMU Developers <qemu-devel@nongnu.org>,
Alexander Graf <agraf@suse.de>
Subject: Re: [Qemu-devel] [PATCH v8] hw/arm/virt: Add high MMIO PCI region, 512G in size
Date: Sat, 3 Oct 2015 19:24:40 +0100 [thread overview]
Message-ID: <CAFEAcA-Kss5MZjwAtmXufi2m5JMmpE4EiKx_EMf0uYnHmfo1_Q@mail.gmail.com> (raw)
In-Reply-To: <CAPokK=qBgHAwv6_z0kzmAmoHURBcGBxiZiM63t8=bQ5aUjsHEw@mail.gmail.com>
On 3 October 2015 at 19:04, Peter Crosthwaite
<crosthwaitepeter@gmail.com> wrote:
> On Thu, Sep 24, 2015 at 5:13 PM, Peter Maydell <peter.maydell@linaro.org> wrote:
>> On 4 September 2015 at 00:13, Pavel Fedin <p.fedin@samsung.com> wrote:
>>> Peter Maydell wrote:
>>>> 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.
>>
>> Nudge -- have you reported this as a kernel bug against the
>> PCI generic driver yet?
>>
>
> So the use case that causes this bug is very vanilla (I just got bit).
> Can we reverse the default of the highmem switch so the unmodified
> command line continues to work?
That would then mean we practically never would use the
highmem window, and we'd have to tell everybody "you need
to use this extra weird command line option", which I don't
like.
Once again: has anybody reported this as a kernel bug?
thanks
-- PMM
next prev parent reply other threads:[~2015-10-03 18:25 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-08-24 7:31 [Qemu-devel] [PATCH v8] hw/arm/virt: Add high MMIO PCI region, 512G in size Pavel Fedin
2015-08-24 7:46 ` Alexander Graf
2015-08-31 12:17 ` Igor Mammedov
2015-09-01 13:28 ` Shannon Zhao
2015-09-03 18:06 ` Peter Maydell
2015-09-04 7:13 ` Pavel Fedin
2015-09-04 8:50 ` Peter Maydell
2015-09-04 8:52 ` Pavel Fedin
2015-09-04 9:32 ` Pavel Fedin
2015-09-25 0:13 ` Peter Maydell
2015-10-03 18:04 ` Peter Crosthwaite
2015-10-03 18:24 ` Peter Maydell [this message]
2015-10-07 10:50 ` Pavel Fedin
2015-10-07 21:16 ` Peter Maydell
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAFEAcA-Kss5MZjwAtmXufi2m5JMmpE4EiKx_EMf0uYnHmfo1_Q@mail.gmail.com \
--to=peter.maydell@linaro.org \
--cc=agraf@suse.de \
--cc=crosthwaitepeter@gmail.com \
--cc=p.fedin@samsung.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).