qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Peter Maydell <peter.maydell@linaro.org>
To: Pavel Fedin <p.fedin@samsung.com>
Cc: 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: Fri, 4 Sep 2015 09:50:25 +0100	[thread overview]
Message-ID: <CAFEAcA843LGwanP8nej0SAXAmvDFcjs0TdtTm7RrATCxJnuACA@mail.gmail.com> (raw)
In-Reply-To: <013201d0e6e1$43c49090$cb4db1b0$@samsung.com>

On 4 September 2015 at 08:13, Pavel Fedin <p.fedin@samsung.com> wrote:
>> 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.

Yeah, we only let people with accounts create new ones, to
avoid spam. If you tell me what username you'd like I'll
create an account for you.

>> 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.

We need to actually confirm that there's a problem before
we put this in master, because if there aren't guests
that complain then we don't need to have the userfacing
switch which avoids using high memory.

Can you test this reasonably soon, please? Otherwise
I'll have to drop it from target-arm again.

thanks
-- PMM

  reply	other threads:[~2015-09-04  8:50 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 [this message]
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
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=CAFEAcA843LGwanP8nej0SAXAmvDFcjs0TdtTm7RrATCxJnuACA@mail.gmail.com \
    --to=peter.maydell@linaro.org \
    --cc=agraf@suse.de \
    --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).