* [PATCH] ext4, doc: Fix incorrect h_reserved size
@ 2022-03-31 6:37 wangjianjian (C)
2022-04-01 12:07 ` wangjianjian (C)
2022-04-16 8:58 ` Jonathan Corbet
0 siblings, 2 replies; 5+ messages in thread
From: wangjianjian (C) @ 2022-03-31 6:37 UTC (permalink / raw)
To: tytso, adilger.kernel, corbet; +Cc: linux-ext4, linux-doc
According to document and code, ext4_xattr_header's size is 32 bytes, so
h_reserved size should be 3.
Signed-off-by: Wang Jianjian <wangjianjian3@huawei.com>
---
Documentation/filesystems/ext4/attributes.rst | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/filesystems/ext4/attributes.rst
b/Documentation/filesystems/ext4/attributes.rst
index 54386a010a8d..871d2da7a0a9 100644
--- a/Documentation/filesystems/ext4/attributes.rst
+++ b/Documentation/filesystems/ext4/attributes.rst
@@ -76,7 +76,7 @@ The beginning of an extended attribute block is in
- Checksum of the extended attribute block.
* - 0x14
- \_\_u32
- - h\_reserved[2]
+ - h\_reserved[3]
- Zero.
The checksum is calculated against the FS UUID, the 64-bit block number
--
2.32.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH] ext4, doc: Fix incorrect h_reserved size
2022-03-31 6:37 [PATCH] ext4, doc: Fix incorrect h_reserved size wangjianjian (C)
@ 2022-04-01 12:07 ` wangjianjian (C)
2022-04-03 14:53 ` Theodore Ts'o
2022-04-16 8:58 ` Jonathan Corbet
1 sibling, 1 reply; 5+ messages in thread
From: wangjianjian (C) @ 2022-04-01 12:07 UTC (permalink / raw)
To: linux-doc, linux-ext4
According to document and code, ext4_xattr_header's size is 32 bytes, so
h_reserved size should be 3.
Signed-off-by: Wang Jianjian <wangjianjian3@huawei.com>
---
Documentation/filesystems/ext4/attributes.rst | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/filesystems/ext4/attributes.rst
b/Documentation/filesystems/ext4/attributes.rst
index 54386a010a8d..871d2da7a0a9 100644
--- a/Documentation/filesystems/ext4/attributes.rst
+++ b/Documentation/filesystems/ext4/attributes.rst
@@ -76,7 +76,7 @@ The beginning of an extended attribute block is in
- Checksum of the extended attribute block.
* - 0x14
- \_\_u32
- - h\_reserved[2]
+ - h\_reserved[3]
- Zero.
The checksum is calculated against the FS UUID, the 64-bit block number
--
2.32.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] ext4, doc: Fix incorrect h_reserved size
2022-04-01 12:07 ` wangjianjian (C)
@ 2022-04-03 14:53 ` Theodore Ts'o
0 siblings, 0 replies; 5+ messages in thread
From: Theodore Ts'o @ 2022-04-03 14:53 UTC (permalink / raw)
To: wangjianjian (C), linux-doc, linux-ext4; +Cc: Theodore Ts'o
On Fri, 1 Apr 2022 20:07:35 +0800, wangjianjian (C) wrote:
> According to document and code, ext4_xattr_header's size is 32 bytes, so
> h_reserved size should be 3.
>
>
Applied, thanks!
[1/1] ext4, doc: Fix incorrect h_reserved size
commit: e830f1a05b203e8b9b81606caf6b5d68458ba732
Note that this patch was whitespace damaged, so I had to manually
apply the patch. It appears that all TAB characters had been replaced
with the bytes 0xC2 0x020, which is the UTF-8 encoding for U+0082, aka
'BREAK PERMITTED HERE'. That seems.... wierd; I don't know how that
would ever be considered a valid character set transformation.
In any case, you might want to look at your Mail User Agent and see if
there are some settings you can adjust.
Best regards,
--
Theodore Ts'o <tytso@mit.edu>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] ext4, doc: Fix incorrect h_reserved size
2022-03-31 6:37 [PATCH] ext4, doc: Fix incorrect h_reserved size wangjianjian (C)
2022-04-01 12:07 ` wangjianjian (C)
@ 2022-04-16 8:58 ` Jonathan Corbet
2022-04-19 8:31 ` wangjianjian (C)
1 sibling, 1 reply; 5+ messages in thread
From: Jonathan Corbet @ 2022-04-16 8:58 UTC (permalink / raw)
To: wangjianjian (C), tytso, adilger.kernel; +Cc: linux-ext4, linux-doc
"wangjianjian (C)" <wangjianjian3@huawei.com> writes:
> According to document and code, ext4_xattr_header's size is 32 bytes, so
> h_reserved size should be 3.
>
> Signed-off-by: Wang Jianjian <wangjianjian3@huawei.com>
> ---
> Documentation/filesystems/ext4/attributes.rst | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/filesystems/ext4/attributes.rst
> b/Documentation/filesystems/ext4/attributes.rst
> index 54386a010a8d..871d2da7a0a9 100644
> --- a/Documentation/filesystems/ext4/attributes.rst
> +++ b/Documentation/filesystems/ext4/attributes.rst
> @@ -76,7 +76,7 @@ The beginning of an extended attribute block is in
> - Checksum of the extended attribute block.
> * - 0x14
> - \_\_u32
> - - h\_reserved[2]
> + - h\_reserved[3]
> - Zero.
>
So this patch looks whitespace-damaged, please be sure that you can send
applyable patches to the list.
Beyond that, though, while you're in the neighborhood, please fix the
unnecessary underscore escaping (i.e. s/\_/_/).
Thanks,
jon
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] ext4, doc: Fix incorrect h_reserved size
2022-04-16 8:58 ` Jonathan Corbet
@ 2022-04-19 8:31 ` wangjianjian (C)
0 siblings, 0 replies; 5+ messages in thread
From: wangjianjian (C) @ 2022-04-19 8:31 UTC (permalink / raw)
To: Jonathan Corbet, tytso, adilger.kernel; +Cc: linux-ext4, linux-doc
>>So this patch looks whitespace-damaged, please be sure that you can send
>> applyable patches to the list.
Thanks for the notice.
>> Beyond that, though, while you're in the neighborhood, please fix the
>> unnecessary underscore escaping (i.e. s/\_/_/).
Sure. Let me fix it.
Thanks,
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2022-04-19 8:31 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-03-31 6:37 [PATCH] ext4, doc: Fix incorrect h_reserved size wangjianjian (C)
2022-04-01 12:07 ` wangjianjian (C)
2022-04-03 14:53 ` Theodore Ts'o
2022-04-16 8:58 ` Jonathan Corbet
2022-04-19 8:31 ` wangjianjian (C)
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.