Hi Jan,
On 30/11/2021 10:31, Jan Beulich wrote:
On 30.11.2021 10:37, Ayan Kumar Halder wrote:
On 30/11/2021 07:18, Jan Beulich wrote:
On 29.11.2021 20:44, Ayan Kumar Halder wrote:
GENMASK(30, 21) should be 0x07fe00000
Please can this have a meaningful title? E.g. "bitops: fix incorrect
value in comment"?
--- a/xen/include/xen/bitops.h
+++ b/xen/include/xen/bitops.h
@@ -5,7 +5,7 @@
/*
* Create a contiguous bitmask starting at bit position @l and ending at
* position @h. For example
- * GENMASK(30, 21) gives us the 32bit vector 0x01fe00000.
+ * GENMASK(30, 21) gives us the 32bit vector 0x07fe00000.
Once at it I think you also want to
- replace the word "vector",
Should I replace "vector" with "value" ?
Perhaps, yes.
Thanks for the review. I have sent a v2 patch.
- Ayan
Jan
