[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] fuzz: Avoid deprecated misuse of -drive if=sd
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH] fuzz: Avoid deprecated misuse of -drive if=sd |
Date: |
Mon, 22 Mar 2021 15:20:32 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.8.0 |
On 3/19/21 3:40 PM, Alexander Bulekov wrote:
> On 210319 1420, Markus Armbruster wrote:
>> qemu-fuzz-i386-target-generic-fuzz-sdhci-v3 uses -drive=sd where it
>> should use -drive if=none instead. This prints a deprecation warning:
>>
>> $ ./build-oss-fuzz/DEST_DIR/qemu-fuzz-i386-target-generic-fuzz-sdhci-v3
>> -runs=1 -seed=1
>> [ASan warnings snipped...]
>> --> i386: -drive if=sd,index=0,file=null-co://,format=raw,id=mydrive:
>> warning: bogus if=sd is deprecated, use if=none
>> INFO: Seed: 1
>> [More normal output snipped...]
>>
>> Support for this usage will be gone soon. Adjust the test.
>>
>> Signed-off-by: Markus Armbruster <armbru@redhat.com>
>> ---
>> tests/qtest/fuzz/generic_fuzz_configs.h | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>
> Reviewed-by: Alexander Bulekov <alxndr@bu.edu>
This patch doesn't apply:
$ git am -s 20210319_armbru_fuzz_avoid_deprecated_misuse_of_drive_if_sd.mbx
Applying: fuzz: Avoid deprecated misuse of -drive if=sd
error: patch failed: tests/qtest/fuzz/generic_fuzz_configs.h:182
error: tests/qtest/fuzz/generic_fuzz_configs.h: patch does not apply
Patch failed at 0001 fuzz: Avoid deprecated misuse of -drive if=sd
hint: Use 'git am --show-current-patch=diff' to see the failed patch
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".
Which makes sense because it is already in the tree as commit
9585376ab5e.
To ease maintainer/reviewers email based workflow, does it make
sense to configure patchew to reply "this patch/series has been
merged as sha1..range" or or "this patch/series has been superseded
by this one: [link of v(N+1)]"?
Thanks,
Phil.