[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH v2 4/4] tests: add some sdcard qtest
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [RFC PATCH v2 4/4] tests: add some sdcard qtest |
Date: |
Fri, 5 Jan 2018 15:31:36 +0000 |
User-agent: |
Mutt/1.9.1 (2017-09-22) |
On Wed, Jan 03, 2018 at 06:49:25PM -0300, Philippe Mathieu-Daudé wrote:
> +static const char *machines[PROTO_COUNT] = {
> + [PROTO_SD] = "nuri",
> + //[PROTO_MMC] = "vexpress-a9",
> + //[PROTO_SPI] = "lm3s6965evb"
> +};
> +
> +static const uint64_t sizes[] = {
> + //512 * M_BYTE,
> + //1 * G_BYTE,
> + 4 * G_BYTE,
> + //64 * G_BYTE,
> +};
Why are these commented out?
> +static void test1(SDBusAdapter *mmc, uint64_t size)
> +{
> + uint8_t *response;
> + uint16_t rca;
> + ssize_t sz;
> +
> + sz = sdbus_do_cmd(mmc, GO_IDLE_STATE, 0, NULL);
> + g_assert_cmpuint(sz, ==, 0);
> +
> + sz = sdbus_do_cmd(mmc, SEND_IF_COND, 0x1aa, NULL);
> + //g_assert_cmpuint(sz, ==, 0);
Why is this commented out?
> + // TODO 8x: sdcard_read_data len 512
> +
> + //sz = sdbus_do_acmd(mmc, SEND_STATUS, 0, rca, &response);
> + //g_free(response);
Why is there commenteded out code and TODO here? Please either
implement this stuff or remove it from the patch.
> +int main(int argc, char **argv)
> +{
> + const char *arch = qtest_get_arch();
> + int iproto, isize;
> + gchar *path;
> + TestCase *test;
> +
> + g_test_init(&argc, &argv, NULL);
> +
> + if (strcmp(arch, "arm") == 0 || strcmp(arch, "aarch64") == 0) {
> + for (iproto = 0; iproto < PROTO_COUNT; iproto++) {
> + if (!machines[iproto]) {
> + continue;
> + }
> + for (isize = 0; isize < ARRAY_SIZE(sizes); isize++) {
> + test = g_new(TestCase, 1);
> +
> + test->protocol = iproto;
> + test->size = sizes[isize];
> +
> + path = g_strdup_printf("sdcard/%s/%lu", proto_name[iproto],
> sizes[isize]);
> + qtest_add_data_func(path, test, sdcard_tests);
> + g_free(path);
> + // g_free(test)?
Please remove this. qtest_add_data_func() keeps the pointer to test so
it must not be freed.
signature.asc
Description: PGP signature
- Re: [Qemu-devel] [PATCH v2 1/4] sdbus: add a QMP command to access a SDBus, (continued)
[Qemu-devel] [RFC PATCH v2 2/4] libqos: add a sdbus API, Philippe Mathieu-Daudé, 2018/01/03
[Qemu-devel] [RFC PATCH v2 3/4] libqos: implement sdbus QMP driver, Philippe Mathieu-Daudé, 2018/01/03
[Qemu-devel] [RFC PATCH v2 4/4] tests: add some sdcard qtest, Philippe Mathieu-Daudé, 2018/01/03
- Re: [Qemu-devel] [RFC PATCH v2 4/4] tests: add some sdcard qtest,
Stefan Hajnoczi <=
Re: [Qemu-devel] [RFC PATCH v2 0/4] sdbus: testing sdcards, no-reply, 2018/01/03
Re: [Qemu-devel] [RFC PATCH v2 0/4] sdbus: testing sdcards, Stefan Hajnoczi, 2018/01/05