qemu-arm
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v5] tests/qtest: add qtests for npcm7xx sdhci


From: Philippe Mathieu-Daudé
Subject: Re: [PATCH v5] tests/qtest: add qtests for npcm7xx sdhci
Date: Tue, 31 May 2022 10:26:38 +0200
User-agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:91.0) Gecko/20100101 Thunderbird/91.9.1

On 26/5/22 18:21, Patrick Venture wrote:
On Thu, May 26, 2022 at 8:54 AM Peter Maydell <peter.maydell@linaro.org <mailto:peter.maydell@linaro.org>> wrote:

    On Fri, 25 Feb 2022 at 17:45, Hao Wu <wuhaotsh@google.com
    <mailto:wuhaotsh@google.com>> wrote:
     >
     > From: Shengtan Mao <stmao@google.com <mailto:stmao@google.com>>
     >
     > Reviewed-by: Hao Wu <wuhaotsh@google.com
    <mailto:wuhaotsh@google.com>>
     > Reviewed-by: Chris Rauer <crauer@google.com
    <mailto:crauer@google.com>>
     > Signed-off-by: Shengtan Mao <stmao@google.com
    <mailto:stmao@google.com>>
     > Signed-off-by: Patrick Venture <venture@google.com
    <mailto:venture@google.com>>

    Hi; John Snow tells me that this test fails in the tests/vm/netbsd
    VM (you can test this with 'make vm-build-netbsd') because the
    assert() on the ftruncate() call fails:

     > +    ret = ftruncate(fd, NPCM7XX_TEST_IMAGE_SIZE);
     > +    g_assert_cmpint(ret, ==, 0);

     > +#define NPCM7XX_TEST_IMAGE_SIZE (1 << 30)

    I haven't investigated the exact cause, but this is a
    gigabyte, right? That's a pretty massive file for a test case to
    create -- can we make the test use a more sensible size of
    sd card image ?


It looks like the nuvoton part had an issue with a smaller image size, but we can resurrect that thread and poke at it a bit and see what shakes out.

Could you use the null-co block driver instead?

    -blockdev driver=null-co,size=1G,read-zeroes=on,node-name=null0
    -device sd-card,drive=null0



reply via email to

[Prev in Thread] Current Thread [Next in Thread]