Bug 870484

Summary:	cpu 'host-model' should support features in force or disable mode
Product:	Red Hat Enterprise Linux 6	Reporter:	Dave Allan <dallan>
Component:	libvirt	Assignee:	Peter Krempa <pkrempa>
Status:	CLOSED WONTFIX	QA Contact:	Virtualization Bugs <virt-bugs>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	6.4	CC:	acathrow, crobinso, cwei, dallan, dfediuck, dyuan, honzhang, mzhan, pbonzini, rjones, virt-maint
Target Milestone:	rc	Keywords:	Upstream
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:	799354
Clones:	956468 (view as bug list)		Environment:
Last Closed:	2014-04-04 21:00:13 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:	799354
Bug Blocks:	865505, 956468, 1059007

Description Dave Allan 2012-10-26 15:40:16 UTC

+++ This bug was initially created as a clone of Bug #799354 +++

Would be handy to have <cpu mode='host-model'> support <feature policy='force'/> or policy=disable. That way tools have an option to turn on x2apic for kvm guests which provides a performance boost with no downsides, but doesn't require host CPU support.

--- Additional comment from rjones on 2012-10-25 14:05:16 EDT ---

Also <feature policy='disable'/>.  For one use case, see:
https://bugzilla.redhat.com/show_bug.cgi?id=870071#c2

Comment 1 Jiri Denemark 2012-10-29 13:13:31 UTC

I'm the owner of upstream brother of this bug.

Comment 2 Andrew Cathrow 2012-12-31 22:11:37 UTC

Depending on the implementation of this we may want to close 891015 which covers host passthrough mode.

Comment 3 Peter Krempa 2013-07-16 09:14:32 UTC

Fixed upstream with:

commit 3c8be55c04eb2647fd5c9d51efae78f7afdcec85
Author: Peter Krempa <pkrempa>
Date:   Mon Jul 15 17:38:55 2013 +0200

    cpu: Allow fine tuning of "host-model" cpu
    
    https://bugzilla.redhat.com/show_bug.cgi?id=799354
    
    Until now, the "host-model" cpu mode couldn't be influenced. This patch
    allows to use the <feature> elements to either enable or disable
    specific CPU flags. This can be used to force flags that can be emulated
    even if the host CPU doesn't support them.

commit 13cdd389ed06e81277f208c269d825046cdbb4cb
Author: Peter Krempa <pkrempa>
Date:   Mon Jul 15 14:36:09 2013 +0200

    cpu: Add virCPUDefUpdateFeature()
    
    This new function updates or adds a feature to a existing cpu model
    definition. This function will be helpful to allow tuning of
    "host-model" features in later patches.

v1.1.0-193-g3c8be55

Comment 5 Jiri Denemark 2013-07-31 11:07:57 UTC

This bug has already been fixed upstream but it is considered
unsuitable for backporting to RHEL 6.5 because at least one of the
following conditions is met:

- the patches required to address this bug are complex or invasive
  causing the backport to be too risky
- this bug is not important enough to justify backporting non-trivial
  patches for it

Thus I'm pushing this bug to RHEL 6.6 (and setting Upstream keyword to
indicate we have patches upstream) for now. If you don't agree with
this resolution, please, give us reasons which you think are strong
enough for us to reevaluate the decision not to backport patches for
this bug.

Comment 7 RHEL Program Management 2014-04-04 21:00:13 UTC

Development Management has reviewed and declined this request.
You may appeal this decision by reopening this request.