Messages in this thread | | | Subject | Re: [PATCH 1/1] selftests: kvm: Add overlapped memory regions test | From | Paolo Bonzini <> | Date | Thu, 16 Apr 2020 15:53:36 +0200 |
| |
On 16/04/20 01:57, Krish Sadhukhan wrote: > >> Add the test_overlap_memory_regions() test case in >> set_memory_region_test. This should check that overlapping >> memory regions on the guest physical address cannot be added. > > > I think the commit header and the body need some improvement. For example, > > Header: Test that overlapping guest memory regions can not be added > > Body: Enhance the existing tests in set_memory_region_test.c so > that it tests overlapping guest > > memory regions. The new test verifies that adding > overlapping guest memory regions fails.
I like Wainer's header and your body. :)
Paolo
| |