Bug 513263

Summary: mp_state is defined as int but used as uint32_t
Product: Red Hat Enterprise Linux 5 Reporter: Juan Quintela <quintela>
Component: kvmAssignee: Juan Quintela <quintela>
Status: CLOSED DUPLICATE QA Contact: Lawrence Lim <llim>
Severity: medium Docs Contact:
Priority: low    
Version: 5.4CC: ehabkost, tools-bugs, virt-maint, ykaul
Target Milestone: rc   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2009-09-16 18:28:31 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:
Attachments:
Description Flags
change mp_state type to uint32_t none

Description Juan Quintela 2009-07-22 18:11:19 UTC 
Description of problem:

We use a variable of type int to store a uint32_t variable.
This variable is send through the wire and between kernel/userland.

Analysis:
- kernel declares int mp_state;
- kernel transfers to/from userspace over ioctl _u32 mp_state.
- qemu stores internall int mp_state.
- qemu saves/load during migration uint32_t mp_state.

qemu don't use that variable at all, just save it for migration.
Changing it to uint32_t will make everything working


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


How reproducible:


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


Expected results:


Additional info:
Comment 1 Juan Quintela 2009-07-22 18:12:00 UTC 
Created attachment 354760 [details]
change mp_state type to uint32_t
Comment 3 Juan Quintela 2009-09-16 18:28:31 UTC 

*** This bug has been marked as a duplicate of bug 515749 ***