-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
yaml --- r: 202375 b: refs/heads/master c: cf3e3d3 h: refs/heads/master i: 202373: 47f51fd 202371: 8eaed5e 202367: bb55bde v: v3
- Loading branch information
Avi Kivity
committed
Aug 1, 2010
1 parent
69a2829
commit 32c5797
Showing
2 changed files
with
39 additions
and
1 deletion.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,2 +1,2 @@ | ||
--- | ||
refs/heads/master: 3e0075094734de122e4cb09f930fa853a3c59f09 | ||
refs/heads/master: cf3e3d3e19868ca01da163200bbfc687523df0fc |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,38 @@ | ||
Review checklist for kvm patches | ||
================================ | ||
|
||
1. The patch must follow Documentation/CodingStyle and | ||
Documentation/SubmittingPatches. | ||
|
||
2. Patches should be against kvm.git master branch. | ||
|
||
3. If the patch introduces or modifies a new userspace API: | ||
- the API must be documented in Documentation/kvm/api.txt | ||
- the API must be discoverable using KVM_CHECK_EXTENSION | ||
|
||
4. New state must include support for save/restore. | ||
|
||
5. New features must default to off (userspace should explicitly request them). | ||
Performance improvements can and should default to on. | ||
|
||
6. New cpu features should be exposed via KVM_GET_SUPPORTED_CPUID2 | ||
|
||
7. Emulator changes should be accompanied by unit tests for qemu-kvm.git | ||
kvm/test directory. | ||
|
||
8. Changes should be vendor neutral when possible. Changes to common code | ||
are better than duplicating changes to vendor code. | ||
|
||
9. Similarly, prefer changes to arch independent code than to arch dependent | ||
code. | ||
|
||
10. User/kernel interfaces and guest/host interfaces must be 64-bit clean | ||
(all variables and sizes naturally aligned on 64-bit; use specific types | ||
only - u64 rather than ulong). | ||
|
||
11. New guest visible features must either be documented in a hardware manual | ||
or be accompanied by documentation. | ||
|
||
12. Features must be robust against reset and kexec - for example, shared | ||
host/guest memory must be unshared to prevent the host from writing to | ||
guest memory that the guest has not reserved for this purpose. |