Bug 671376

Summary: Back port missing fixes for live snapshot support from upstream
Product: Red Hat Enterprise Linux 6 Reporter: Jes Sorensen <Jes.Sorensen>
Component: qemu-kvmAssignee: Jes Sorensen <Jes.Sorensen>
Status: CLOSED WONTFIX QA Contact: Virtualization Bugs <virt-bugs>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 6.1CC: amit.shah, juzhang, lihuang, llim, mkenneth, szhou, virt-maint
Target Milestone: rcKeywords: Triaged
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-03-17 15:02:40 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Jes Sorensen 2011-01-21 13:11:01 UTC 
Description of problem:
A couple of minor, but important bugfixes have been posted for strtosz()
which is required for live snapshot support.

Back port these as soon as they settle upstream.

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:
Comment 1 Lawrence Lim 2011-01-24 00:20:44 UTC 
In order for us to ensure no regression was introduced, would it be possible if you could list the backport planned and testing instructions.

Thanks.
Comment 2 Jes Sorensen 2011-01-26 12:45:49 UTC 
Hi Lawrence,

I had to do a respin of the original patch set, so the changes I had
planned for this bug are going in with that. It may turn out that this
one becomes a non-issue after all.

The two changes that were relevant was a fix for special characters
when specifying file sizes, which shouldn't be an issue, and a fix
that prevents qemu from crashing if one tries to do a snapshot without
specifying a filename for the snapshot image.

Cheers,
Jes
Comment 4 Jes Sorensen 2011-03-17 15:02:40 UTC 
The outstanding issues have been resolved through other patches, so I do
not believe this one is needed.