[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 22/32] qtest: Cover qdev properties for disk geo
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH 22/32] qtest: Cover qdev properties for disk geometry |
Date: |
Thu, 05 Jul 2012 14:08:31 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.3 (gnu/linux) |
Kevin Wolf <address@hidden> writes:
> Am 29.06.2012 17:34, schrieb Markus Armbruster:
>> Signed-off-by: Markus Armbruster <address@hidden>
>> ---
>> tests/hd-geo-test.c | 6 ++++--
>> 1 files changed, 4 insertions(+), 2 deletions(-)
>>
>> diff --git a/tests/hd-geo-test.c b/tests/hd-geo-test.c
>> index 0ab573c..02eb5c2 100644
>> --- a/tests/hd-geo-test.c
>> +++ b/tests/hd-geo-test.c
>> @@ -321,13 +321,15 @@ static void test_ide_drive_user(const char *dev, bool
>> trans)
>> const chst expected_chst = { secs / (4 * 32) , 4, 32, trans };
>>
>> argc = setup_common(argv, ARRAY_SIZE(argv));
>> - opts = g_strdup_printf(",cyls=%d,heads=%d,secs=%d%s",
>> + opts = g_strdup_printf("%s,cyls=%d,heads=%d,secs=%d%s",
>> + dev && !trans ? dev : "",
>> expected_chst.cyls, expected_chst.heads,
>> expected_chst.secs,
>> trans ? ",trans=lba" : "");
>> cur_ide[0] = &expected_chst;
>> argc = setup_ide(argc, argv, ARRAY_SIZE(argv),
>> - 0, dev, backend_small, mbr_chs, opts);
>> + 0, dev && !trans ? opts : NULL, backend_small, mbr_chs,
>> + dev && !trans ? "" : opts);
>> qtest_start(g_strjoinv(" ", argv));
>> test_cmos();
>> qtest_quit(global_qtest);
>
> I've spent more time parsing this test code than I needed for the review
> for most patches that touch actual code... Maybe an explicit if (dev &&
> !trans) would help somewhat. don't know.
Sorry about that. It gets better again in PATCH 26/32.