Re: [Qemu-devel] [PATCH 2/4] IDE: test flush on empty CDROM

[John Snow]

On 08/08/2017 03:20 PM, Eric Blake wrote:
> On 08/08/2017 12:57 PM, John Snow wrote:
>> From: Kevin Wolf 
>>
>> Signed-off-by: Kevin Wolf 
>> Signed-off-by: John Snow 
>> ---
>>  tests/ide-test.c | 19 +++
>>  1 file changed, 19 insertions(+)
>>
> 
>> +static void test_flush_empty_drive(void)
>> +{
>> +QPCIDevice *dev;
>> +QPCIBar bmdma_bar, ide_bar;
>> +
>> +ide_test_start("-device ide-cd,bus=ide.0");
>> +dev = get_pci_device(_bar, _bar);
>> +
>> +/* FLUSH CACHE command on device 0*/
> 
> Space before */
> 
> Reviewed-by: Eric Blake 
> 
> I agree with your assessment of 1 and 2 being 2.10 material.
> 

Yep, thanks. I just wanted to include Kevin's attempt at fixing the root
problem to make it clear that:

(A) The root problem is known and being worked on, but
(B) Is evidently not ready for prime time.

I'll stage 1 & 2 with your minor typo edit here, thank you.

--js

Re: [Qemu-devel] [PATCH 2/4] IDE: test flush on empty CDROM

[Eric Blake]

On 08/08/2017 12:57 PM, John Snow wrote:
> From: Kevin Wolf 
> 
> Signed-off-by: Kevin Wolf 
> Signed-off-by: John Snow 
> ---
>  tests/ide-test.c | 19 +++
>  1 file changed, 19 insertions(+)
> 

> +static void test_flush_empty_drive(void)
> +{
> +QPCIDevice *dev;
> +QPCIBar bmdma_bar, ide_bar;
> +
> +ide_test_start("-device ide-cd,bus=ide.0");
> +dev = get_pci_device(_bar, _bar);
> +
> +/* FLUSH CACHE command on device 0*/

Space before */

Reviewed-by: Eric Blake 

I agree with your assessment of 1 and 2 being 2.10 material.

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.   +1-919-301-3266
Virtualization:  qemu.org | libvirt.org



signature.asc
Description: OpenPGP digital signature