qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Philippe Mathieu-Daudé" <philmd@linaro.org>
To: Peter Maydell <peter.maydell@linaro.org>,
	Guenter Roeck <linux@roeck-us.net>
Cc: "QEMU Developers" <qemu-devel@nongnu.org>,
	qemu-arm <qemu-arm@nongnu.org>,
	"Marcin Juszkiewicz" <marcin.juszkiewicz@linaro.org>,
	"Aaro Koskinen" <aaro.koskinen@iki.fi>,
	"Janusz Krzysztofik" <jmkrzyszt@gmail.com>,
	"Tony Lindgren" <tony@atomide.com>,
	linux-omap@vger.kernel.org, "Arnd Bergmann" <arnd@arndb.de>,
	"Daniel Mack" <daniel@zonque.org>,
	"Robert Jarzmik" <robert.jarzmik@free.fr>,
	"Haojian Zhuang" <haojian.zhuang@gmail.com>,
	"Thomas Huth" <thuth@redhat.com>,
	"Cédric Le Goater" <clg@kaod.org>
Subject: Re: possible deprecation and removal of some old QEMU Arm machine types (pxa2xx, omap, sa1110)
Date: Thu, 17 Oct 2024 14:53:45 -0300	[thread overview]
Message-ID: <5b6e9f3f-36e7-4b0f-8698-15a419095558@linaro.org> (raw)
In-Reply-To: <CAFEAcA9AfH7cTO8TYSh9E+FnYN8SbTLDkXW8W5Jwi9Kfcddr6g@mail.gmail.com>

On 17/10/24 13:07, Peter Maydell wrote:
> On Thu, 17 Oct 2024 at 16:29, Guenter Roeck <linux@roeck-us.net> wrote:


>>> By the way, it looks to me like QEMU has a regression
>>> somewhere where we can't boot that sx1 test for the SD
>>> card version -- it hangs during kernel boot waiting for
>>> the MMC card. (An elderly QEMU binary I have boots OK.)
>>> I'm looking into what's happened there.
>>>
>>
>> Yes, you are correct. I did a quick check; the problem started with v9.1.
>> v9.0 boots fine.
> 
> It's an issue with commit 1ab08790bb75e4 -- when we did a refactor
> of the SD card emulation we didn't notice that the omap mmc
> controller was also using the sd_cmd_type_t enum and relied
> on the values of that enum matching the meanings of the
> different values of the controller's MMC_CMD register Type field.
> I'm just testing a patch.

Doh indeed I missed that use in omap_mmc_command()...

Thanks for the quick analysis!

Phil.



  reply	other threads:[~2024-10-17 17:54 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-02-12 12:32 possible deprecation and removal of some old QEMU Arm machine types (pxa2xx, omap, sa1110) Peter Maydell
2024-02-12 14:03 ` Guenter Roeck
2024-02-12 14:36 ` Guenter Roeck
2024-02-13 13:42   ` Marcin Juszkiewicz
2024-02-13 14:36     ` Guenter Roeck
2024-02-13 15:14   ` Peter Maydell
2024-02-13 15:36     ` Guenter Roeck
2024-02-13 20:11       ` Arnd Bergmann
2024-02-13 21:21         ` Linus Walleij
2024-02-14 10:14           ` Arnd Bergmann
2024-02-14 12:26           ` Dmitry Baryshkov
2024-02-14 13:39             ` Marcin Juszkiewicz
2024-02-14 22:15               ` Andrea Adami
2024-02-14 22:42             ` Arnd Bergmann
2024-02-15  8:31               ` Andreas Kemnade
2024-02-15  8:45                 ` Arnd Bergmann
2024-02-15  8:52                   ` Dmitry Baryshkov
2024-02-15 10:06                     ` Marcin Juszkiewicz
2024-02-15 11:31                 ` H. Nikolaus Schaller
2024-02-14  1:27         ` Aaro Koskinen
2024-02-14  5:53           ` Tony Lindgren
2024-02-14 14:57           ` Arnd Bergmann
2024-02-14  9:08         ` Daniel Mack
2024-03-08 15:41       ` Peter Maydell
2024-03-08 15:50         ` Guenter Roeck
2024-10-15 17:13         ` Peter Maydell
2024-10-15 17:30           ` Guenter Roeck
2024-10-15 17:35             ` Peter Maydell
2024-10-15 18:12               ` Guenter Roeck
2024-10-17 12:42                 ` Peter Maydell
2024-10-17 14:12                   ` Guenter Roeck
2024-10-17 14:29                     ` Peter Maydell
2024-10-17 15:29                       ` Guenter Roeck
2024-10-17 16:07                         ` Peter Maydell
2024-10-17 17:53                           ` Philippe Mathieu-Daudé [this message]
2024-02-14  7:37 ` Philippe Mathieu-Daudé
2024-02-14  7:49   ` Philippe Mathieu-Daudé

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=5b6e9f3f-36e7-4b0f-8698-15a419095558@linaro.org \
    --to=philmd@linaro.org \
    --cc=aaro.koskinen@iki.fi \
    --cc=arnd@arndb.de \
    --cc=clg@kaod.org \
    --cc=daniel@zonque.org \
    --cc=haojian.zhuang@gmail.com \
    --cc=jmkrzyszt@gmail.com \
    --cc=linux-omap@vger.kernel.org \
    --cc=linux@roeck-us.net \
    --cc=marcin.juszkiewicz@linaro.org \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-arm@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    --cc=robert.jarzmik@free.fr \
    --cc=thuth@redhat.com \
    --cc=tony@atomide.com \
    /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).