Bug 484364

Summary: block-rw-range-check.patch breaks qcow2
Product: [Fedora] Fedora Reporter: Mark McLoughlin <markmc>
Component: qemuAssignee: Glauber Costa <gcosta>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: low    
Version: rawhideCC: berrange, bugproxy, clalance, dwmw2, ehabkost, gcosta, markmc, quintela, virt-maint
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 484948 (view as bug list) Environment:
Last Closed: 2009-03-16 12:52:49 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 480594, 484948    
Attachments:
Description Flags
Screenshot showing hard drive not found message
none
Screenshot of a warning message we got
none
mx3605b cpuinfo none

Description Mark McLoughlin 2009-02-06 14:24:42 UTC
If I create a new qcow2 file and try and install Fedora 8 on it, the installer never shows sda in the partitioning screen. anaconda.log shows:

  removing drive sda from disk lists; adding to skip list

Turns out this isn't a problem with kvm-83 from git, so I added some debugging to block-rw-range-check.patch:

+    if (ret)
+      printf("Failing bdrv_rd_badreq_sectors(%ld, %d) : total_sectors = %ld\n",
+             sector_num, nb_sectors, bs->total_sectors);

and I see:

  Failing bdrv_rd_badreq_sectors(192, 1) : total_sectors = 192

Comment 1 Eduardo Habkost 2009-02-10 20:43:35 UTC
Reproduced here. Investigating.

Comment 2 Eduardo Habkost 2009-02-11 13:17:46 UTC
Found the issue: the patch on Rawhide is using bdrv_rd_badreq_sectors() instead of bdrv_rw_badreq_sectors() on bdrv_aio_write().

Comment 3 Eduardo Habkost 2009-02-11 13:36:55 UTC
Fixed on Rawhide CVS (kvm-83-4.fc11)

Comment 4 Daniel Berrangé 2009-03-11 12:40:14 UTC
Reassigning: The kvm package no longer exists in rawhide/F11, since it is now part of 'qemu'.

Comment 5 Daniel Berrangé 2009-03-16 12:52:49 UTC
Fix is present in new merged RPM qemu-0.10-0.9.kvm20090310git.fc11.x86_64

Comment 6 Mark McLoughlin 2009-03-25 10:38:18 UTC
*** Bug 491582 has been marked as a duplicate of this bug. ***

Comment 7 IBM Bug Proxy 2009-03-25 10:43:36 UTC
Created attachment 336611 [details]
Screenshot showing hard drive not found message

Comment 8 IBM Bug Proxy 2009-03-25 10:43:44 UTC
Created attachment 336612 [details]
Screenshot of a warning message we got

Comment 9 IBM Bug Proxy 2009-03-25 10:43:51 UTC
Created attachment 336613 [details]
mx3605b cpuinfo

Comment 10 IBM Bug Proxy 2009-04-09 05:41:48 UTC
------- Comment From pavan.naregundi.com 2009-04-09 01:30 EDT-------
Tested in F11beta and qemu-kvm could recognize the hard drive.

Thanks
Pavan