[Qemu-devel] [PATCH] test-bitmap: test set 1 bit case for bitmap_set

Wei Yang posted 1 patch 17 weeks ago
Test FreeBSD passed
Test docker-mingw@fedora passed
Test asan passed
Test docker-clang@ubuntu passed
Test checkpatch passed
Test s390x failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20190814002723.5140-1-richardw.yang@linux.intel.com
tests/test-bitmap.c | 12 ++++++++++++
1 file changed, 12 insertions(+)

[Qemu-devel] [PATCH] test-bitmap: test set 1 bit case for bitmap_set

Posted by Wei Yang 17 weeks ago
All current bitmap_set test cases set range across word, while the
handle of a range within one word is different from that.

Add case to set 1 bit as a represent for set range within one word.

Signed-off-by: Wei Yang <richardw.yang@linux.intel.com>

---
Thanks for Paolo's finding.

---
 tests/test-bitmap.c | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/tests/test-bitmap.c b/tests/test-bitmap.c
index 18aa584591..087e02a26c 100644
--- a/tests/test-bitmap.c
+++ b/tests/test-bitmap.c
@@ -67,6 +67,18 @@ static void bitmap_set_case(bmap_set_func set_func)
 
     bmap = bitmap_new(BMAP_SIZE);
 
+    /* Set one bit at offset in second word */
+    for (offset = 0; offset <= BITS_PER_LONG; offset++) {
+        bitmap_clear(bmap, 0, BMAP_SIZE);
+        set_func(bmap, BITS_PER_LONG + offset, 1);
+        g_assert_cmpint(find_first_bit(bmap, 2 * BITS_PER_LONG),
+                        ==, BITS_PER_LONG + offset);
+        g_assert_cmpint(find_next_zero_bit(bmap,
+                                           3 * BITS_PER_LONG,
+                                           BITS_PER_LONG + offset),
+                        ==, BITS_PER_LONG + offset + 1);
+    }
+
     /* Both Aligned, set bits [BITS_PER_LONG, 3*BITS_PER_LONG] */
     set_func(bmap, BITS_PER_LONG, 2 * BITS_PER_LONG);
     g_assert_cmpuint(bmap[1], ==, -1ul);
-- 
2.17.1


Re: [Qemu-devel] [PATCH] test-bitmap: test set 1 bit case for bitmap_set

Posted by Paolo Bonzini 17 weeks ago
On 14/08/19 02:27, Wei Yang wrote:
> All current bitmap_set test cases set range across word, while the
> handle of a range within one word is different from that.
> 
> Add case to set 1 bit as a represent for set range within one word.
> 
> Signed-off-by: Wei Yang <richardw.yang@linux.intel.com>
> 
> ---
> Thanks for Paolo's finding.
> 
> ---
>  tests/test-bitmap.c | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
> 
> diff --git a/tests/test-bitmap.c b/tests/test-bitmap.c
> index 18aa584591..087e02a26c 100644
> --- a/tests/test-bitmap.c
> +++ b/tests/test-bitmap.c
> @@ -67,6 +67,18 @@ static void bitmap_set_case(bmap_set_func set_func)
>  
>      bmap = bitmap_new(BMAP_SIZE);
>  
> +    /* Set one bit at offset in second word */
> +    for (offset = 0; offset <= BITS_PER_LONG; offset++) {
> +        bitmap_clear(bmap, 0, BMAP_SIZE);
> +        set_func(bmap, BITS_PER_LONG + offset, 1);
> +        g_assert_cmpint(find_first_bit(bmap, 2 * BITS_PER_LONG),
> +                        ==, BITS_PER_LONG + offset);
> +        g_assert_cmpint(find_next_zero_bit(bmap,
> +                                           3 * BITS_PER_LONG,
> +                                           BITS_PER_LONG + offset),
> +                        ==, BITS_PER_LONG + offset + 1);
> +    }
> +
>      /* Both Aligned, set bits [BITS_PER_LONG, 3*BITS_PER_LONG] */
>      set_func(bmap, BITS_PER_LONG, 2 * BITS_PER_LONG);
>      g_assert_cmpuint(bmap[1], ==, -1ul);
> 

Queued, thanks for writing the testcase without no one asking! :)

Paolo

Re: [Qemu-devel] [PATCH] test-bitmap: test set 1 bit case for bitmap_set

Posted by Wei Yang 17 weeks ago
On Wed, Aug 14, 2019 at 02:38:24PM +0200, Paolo Bonzini wrote:
>On 14/08/19 02:27, Wei Yang wrote:
>> All current bitmap_set test cases set range across word, while the
>> handle of a range within one word is different from that.
>> 
>> Add case to set 1 bit as a represent for set range within one word.
>> 
>> Signed-off-by: Wei Yang <richardw.yang@linux.intel.com>
>> 
>> ---
>> Thanks for Paolo's finding.
>> 
>> ---
>>  tests/test-bitmap.c | 12 ++++++++++++
>>  1 file changed, 12 insertions(+)
>> 
>> diff --git a/tests/test-bitmap.c b/tests/test-bitmap.c
>> index 18aa584591..087e02a26c 100644
>> --- a/tests/test-bitmap.c
>> +++ b/tests/test-bitmap.c
>> @@ -67,6 +67,18 @@ static void bitmap_set_case(bmap_set_func set_func)
>>  
>>      bmap = bitmap_new(BMAP_SIZE);
>>  
>> +    /* Set one bit at offset in second word */
>> +    for (offset = 0; offset <= BITS_PER_LONG; offset++) {
>> +        bitmap_clear(bmap, 0, BMAP_SIZE);
>> +        set_func(bmap, BITS_PER_LONG + offset, 1);
>> +        g_assert_cmpint(find_first_bit(bmap, 2 * BITS_PER_LONG),
>> +                        ==, BITS_PER_LONG + offset);
>> +        g_assert_cmpint(find_next_zero_bit(bmap,
>> +                                           3 * BITS_PER_LONG,
>> +                                           BITS_PER_LONG + offset),
>> +                        ==, BITS_PER_LONG + offset + 1);
>> +    }
>> +
>>      /* Both Aligned, set bits [BITS_PER_LONG, 3*BITS_PER_LONG] */
>>      set_func(bmap, BITS_PER_LONG, 2 * BITS_PER_LONG);
>>      g_assert_cmpuint(bmap[1], ==, -1ul);
>> 
>
>Queued, thanks for writing the testcase without no one asking! :)
>

My pleasure to help :)

>Paolo

-- 
Wei Yang
Help you, Help me