scons: Make the USE_KVM variable available in C++.

Information
Submitter:	Gabe Black
Repository:	gem5
Branch:	default
Bugs:
Depends On:
Reviewers
Groups:	Default
People:

Description

Changeset 10548:07c3cbac4cdd
---------------------------
scons: Make the USE_KVM variable available in C++.

We need it to determine whether we should expect KVM related parameters
exist in the cirrus graphics device.

Testing Done

The change itself makes sense, but I'd really prefer if we could avoid conditional compilation and use a kvm-agnostic interface to handle device memory. See my reply in the email thread for RB #2513.

Consider this a "ship it" if it really turns out that using conditional compilation is the way to go.

Gabe Black Dec. 9, 2014, 9:51 p.m. (Dec. 9, 2014, 9:51 p.m.)

This is next in the series. The dependent change it was originally needed for is gone, but now it's needed for http://reviews.gem5.org/r/2526/.

Ship It!

Andreas Hansson Dec. 17, 2014, 5:48 a.m. (Dec. 17, 2014, 5:48 a.m.)

Actually, looking at the Cirrus device, could we not simply check if KvmVM is nullptr?

You have a pending review.

Review Board 2.0.15

This change has been marked as submitted.

Screenshots

Files

Status: Closed (submitted)