ext4 fixes for v5.12
The following changes since commit a38fd8748464831584a19438cbb3082b5a2dab15:
Linux 5.12-rc2 (2021-03-05 17:33:41 -0800)
are available in the Git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/tytso/ext4.git tags/ext4_for_linus_stable
for you to fetch changes up to 64395d950bc476106b39341e42ebfd4d2eb71d2c:
ext4: initialize ret to suppress smatch warning (2021-03-21 00:45:37 -0400)
----------------------------------------------------------------
Miscellaneous ext4 bug fixes for v5.12.
----------------------------------------------------------------
Eric Biggers (1):
ext4: fix error handling in ext4_end_enable_verity()
Eric Whitney (1):
ext4: shrink race window in ext4_should_retry_alloc()
Harshad Shirwadkar (1):
ext4: fix rename whiteout with fast commit
Jan Kara (2):
ext4: add reclaim checks to xattr code
ext4: fix timer use-after-free on failed mount
Pan Bian (1):
ext4: stop inode update before return
Sabyrzhan Tasbolatov (1):
fs/ext4: fix integer overflow in s_log_groups_per_flex
Shijie Luo (1):
ext4: fix potential error in ext4_do_update_inode
Theodore Ts'o (1):
ext4: initialize ret to suppress smatch warning
Zhaolong Zhang (1):
ext4: fix bh ref count on error paths
zhangyi (F) (3):
ext4: find old entry again if failed to rename whiteout
ext4: do not iput inode under running transaction in ext4_rename()
ext4: do not try to set xattr into ea_inode if value is empty
fs/ext4/balloc.c | 38 +++++++++++++++++-------- fs/ext4/ext4.h | 3 ++ fs/ext4/extents.c | 2 +- fs/ext4/fast_commit.c | 9 ++++-- fs/ext4/inode.c | 18 ++++++------ fs/ext4/mballoc.c | 11 ++++++-- fs/ext4/namei.c | 50 +++++++++++++++++++++++++-------- fs/ext4/super.c | 7 ++++- fs/ext4/sysfs.c | 7 +++++ fs/ext4/verity.c | 89 ++++++++++++++++++++++++++++++++++++----------------------- fs/ext4/xattr.c | 6 +++- 11 files changed, 168 insertions(+), 72 deletions(-)
The pull request you sent on Sun, 21 Mar 2021 14:31:17 -0400:
The pull request you sent on Sun, 21 Mar 2021 14:31:17 -0400:
Quoted 1 line
git://git.kernel.org/pub/scm/linux/kernel/git/tytso/ext4.gittags/ext4_for_linus_stable
has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/d7f5f1bd3c240c4d527c0871a38dc3d61255ea9e
Thank you!
Signature · 2 lines
Deet-doot-dot, I am a bot. https://korg.docs.kernel.org/prtracker.html
Quoted 5 lines · Theodore Ts'o
zhangyi (F) (3):
ext4: find old entry again if failed to rename whiteout
ext4: do not iput inode under running transaction inext4_rename()
ext4: do not try to set xattr intoea_inodeif value is empty
Side note: this is obviously entirely up to the author, but I think it would be nice if we would encourage people to use their native names if/when they want to.
Maybe this "zhangyi (F)" is how they _want_ to write their name in the kernel, and that's obviously fine if so.
But at the same time, coming from Finland, I remember how people who had the "odd" characters (åäö) in their name ended up replacing them with the US-ASCII version (generally "aa" "ae" and "oe"), and it always just looked bad to a native speaker. Particularly annoying in public contexts.
At the same time, for the same reason, I can also understand people not wanting to even expose those characters at all, because then non-native speakers invariably messed it up even worse...
Anyway, I think and hope that we have the infrastructure to do it right not just for Latin1, but the more complex non-Western character sets too.
And as a result should possibly encourage people to use their native names if they want to. At least make people aware that it _should_ work.
Again, maybe I'm barking up the wrong tree, and in this case "zhangyi
(F) <[email protected]>" is just what zhangyi prefers simply because it's easier/more convenient.
But I just wanted to mention it, because we _do_ have examples of it working. Not many, but some:
git log --pretty="%an" --since=2.years | sort -u | tail
including examples of having the Westernized name in parenthesis for the "use that one if you can't do the real one" case..
Linus
Quoted 36 lines · Linus Torvalds
Quoted 5 lines · Theodore Ts'o
zhangyi (F) (3):
ext4: find old entry again if failed to rename whiteout
ext4: do not iput inode under running transaction inext4_rename()
ext4: do not try to set xattr intoea_inodeif value is emptySide note: this is obviously entirely up to the author, but I think it would be nice if we would encourage people to use their native names if/when they want to.
Maybe this "zhangyi (F)" is how they _want_ to write their name in the kernel, and that's obviously fine if so.
But at the same time, coming from Finland, I remember how people who had the "odd" characters (åäö) in their name ended up replacing them with the US-ASCII version (generally "aa" "ae" and "oe"), and it always just looked bad to a native speaker. Particularly annoying in public contexts.
At the same time, for the same reason, I can also understand people not wanting to even expose those characters at all, because then non-native speakers invariably messed it up even worse...
Anyway, I think and hope that we have the infrastructure to do it right not just for Latin1, but the more complex non-Western character sets too.
And as a result should possibly encourage people to use their native names if they want to. At least make people aware that it _should_ work.
Again, maybe I'm barking up the wrong tree, and in this case "zhangyi
(F) <[email protected]>" is just what zhangyi prefers simply because it's easier/more convenient.
Quoted 8 lines
But I just wanted to mention it, because we _do_ have examples of it working. Not many, but some:
git log --pretty="%an" --since=2.years | sort -u | tailincluding examples of having the Westernized name in parenthesis for the "use that one if you can't do the real one" case..
Hi, Linus.
I will use my real name "Yi Zhang" next time.
Thanks,
Yi.
Quoted 3 lines · zhangyi (F)
I will use my real name "Yi Zhang" next time.
Hi Yi,
I think what Linus was suggsting was that if people wanted, they could do something like this in their git commits:
From: 曹子德(Theodore Y Ts'o) <[email protected]>
I don't do this because my legal name is actually Theodore Yue Tak Ts'o (where Ts'o Yue Tak is the standard romanization of my Chinese name in Cantonese --- my parents were from Hong Kong), and even though Cantonese is technically the first langauge I learned as a child, at this point I'm probably more fluent in Spanish (my third language) than Cantonese. :-)
In any case, git and e-mail should be able to handle non-Roman characters so if you want to insert your name in Chinese in your Git authorship, please feel free to do so. Or not --- it's totally up to you.
Cheers,
- Ted
Quoted 2 lines · Theodore Ts'o
From: 曹子德(Theodore Y Ts'o) <[email protected]>
"Yue" doesn't seem to match your second character which is usually romanised as "Tze" in Cantonese, could it be
曹予德
Cheers,
Signature · 3 lines
Email: Herbert Xu <[email protected]> Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
Quoted 8 lines · Herbert Xu
Quoted 2 lines · Theodore Ts'o
From: 曹子德(Theodore Y Ts'o) <[email protected]>
"Yue" doesn't seem to match your second character which is usually romanised as "Tze" in Cantonese, could it be
曹予德
Quite right. I hadn't noticed that I had the wrong character when I cut and pasted. Thanks for pointing that out!
- Ted
Quoted 40 lines · zhangyi (F)
Quoted 39 lines · Linus Torvalds
Quoted 5 lines · Theodore Ts'o
zhangyi (F) (3):
ext4: find old entry again if failed to rename whiteout
ext4: do not iput inode under running transaction inext4_rename()
ext4: do not try to set xattr intoea_inodeif value is emptySide note: this is obviously entirely up to the author, but I think it would be nice if we would encourage people to use their native names if/when they want to.
Maybe this "zhangyi (F)" is how they _want_ to write their name in the kernel, and that's obviously fine if so.
But at the same time, coming from Finland, I remember how people who had the "odd" characters (åäö) in their name ended up replacing them with the US-ASCII version (generally "aa" "ae" and "oe"), and it always just looked bad to a native speaker. Particularly annoying in public contexts.
At the same time, for the same reason, I can also understand people not wanting to even expose those characters at all, because then non-native speakers invariably messed it up even worse...
Anyway, I think and hope that we have the infrastructure to do it right not just for Latin1, but the more complex non-Western character sets too.
And as a result should possibly encourage people to use their native names if they want to. At least make people aware that it _should_ work.
Again, maybe I'm barking up the wrong tree, and in this case "zhangyi
(F) <[email protected]>" is just what zhangyi prefers simply because it's easier/more convenient.But I just wanted to mention it, because we _do_ have examples of it working. Not many, but some:
Quoted 8 lines
Quoted 5 lines
git log --pretty="%an" --since=2.years | sort -u | tailincluding examples of having the Westernized name in parenthesis for the "use that one if you can't do the real one" case..
Hi, Linus.
I will use my real name "Yi Zhang" next time.
er.. Try to comment my thoughts about this a bit... just random noise.
I think the legel name would be "Zhang Yi" (family name goes first [1]) according to
The Chinese phonetic alphabet spelling rules for Chinese names [2].
Indeed, that is also what the legel name is written in alphabet on our passport or credit/debit cards.
Also, many official English-written materials use it in that way, for example, a somewhat famous bastetball player Yao Ming [3][4][5].
That is what I wrote my own name as this but I also noticed the western ordering of names is quite common for Chinese people in Linux kernel. Anyway, it's just my preliminary personal thought (might be just my own perference) according to (I think, maybe) formal occasions.
[1] https://en.wikipedia.org/wiki/Wikipedia:Naming_conventions_(Chinese)
[2] http://www.moe.gov.cn/ewebeditor/uploadfile/2015/01/13/20150113091249368.pdf
[3] https://en.wikipedia.org/wiki/Yao_Ming
[4] https://www.nbcsports.com/edge/basketball/nba/player/28778/yao-ming
[5] https://news.cgtn.com/news/2020-09-26/Spotlight-Ex-NBA-star-Yao-Ming-05-02-2018-U36mm3dYas/index.html
Thanks,
Gao Xiang
Quoted 3 lines
Thanks,
Yi.
Quoted 7 lines · Gao Xiang
I think the legel name would be "Zhang Yi" (family name goes first [1]) according to
The Chinese phonetic alphabet spelling rules for Chinese names [2].Indeed, that is also what the legel name is written in alphabet on our passport or credit/debit cards.
Quoted 7 lines
Also, many official English-written materials use it in that way, for example, a somewhat famous bastetball player Yao Ming [3][4][5].
That is what I wrote my own name as this but I also noticed the western ordering of names is quite common for Chinese people in Linux kernel. Anyway, it's just my preliminary personal thought (might be just my own perference) according to (I think, maybe) formal occasions.
Yeah, there doesn't seem to be a lot of consistency with the ordering of Chinese names when they are written in Roman characters. Some people put the family name first, and other people will put the personal (first) name first. In some cases it may be because the developer in question is living in America, and so they've decided to use the American naming convention. (Two example of this are former ext4 developers Mingming Cao and Jiaying Zhang, who live in Portland and Los Angelos, and their family names are Cao and Zhang, respectively.)
My personal opinion is people should use whatever name they are comfortable with, using whatever characters they prefer. The one thing that would be helpful for me is for people to give a hint about how they would prefer to be called --- for example, would you prefer that start an e-mail with the salutation, "Hi Gao", "Hi Xiang", or "Hi Gao Xiang"?
And if I don't know, and I guess wrong, please feel free to correct me, either privately, or publically on the e-mail list, if you think it would be helpful for more people to understand how you'd prefer to be called.
Cheers,
- Ted
Quoted 8 lines · Theodore Ts'o
Quoted 7 lines · Gao Xiang
I think the legel name would be "Zhang Yi" (family name goes first [1]) according to
The Chinese phonetic alphabet spelling rules for Chinese names [2].Indeed, that is also what the legel name is written in alphabet on our passport or credit/debit cards.
Quoted 8 lines
Quoted 7 lines
Also, many official English-written materials use it in that way, for example, a somewhat famous bastetball player Yao Ming [3][4][5].
That is what I wrote my own name as this but I also noticed the western ordering of names is quite common for Chinese people in Linux kernel. Anyway, it's just my preliminary personal thought (might be just my own perference) according to (I think, maybe) formal occasions.
Hi Ted,
Quoted 9 lines
Yeah, there doesn't seem to be a lot of consistency with the ordering of Chinese names when they are written in Roman characters. Some people put the family name first, and other people will put the personal (first) name first. In some cases it may be because the developer in question is living in America, and so they've decided to use the American naming convention. (Two example of this are former ext4 developers Mingming Cao and Jiaying Zhang, who live in Portland and Los Angelos, and their family names are Cao and Zhang, respectively.)
Yes, totally agree. I think that's all our own perference honestly (yet just showed some local official materials though.)
Quoted 3 lines
My personal opinion is people should use whatever name they are comfortable with, using whatever characters they prefer. The one
Totally agree.
Quoted 4 lines
thing that would be helpful for me is for people to give a hint about how they would prefer to be called --- for example, would you prefer that start an e-mail with the salutation, "Hi Gao", "Hi Xiang", or "Hi Gao Xiang"?
Honestly, I think either way would be fine on this even in Chinese speaking environment...
Quoted 5 lines
And if I don't know, and I guess wrong, please feel free to correct me, either privately, or publically on the e-mail list, if you think it would be helpful for more people to understand how you'd prefer to be called.
Nope, it's just a minor stuff though. I didn't tend to give any direct/ indirect opinion or hint on this. Sorry about that if some misleading :)
Thanks,
Gao Xiang
Quoted 5 lines
Cheers,
- Ted
Quoted 33 lines · Theodore Ts'o
Quoted 7 lines · Gao Xiang
I think the legel name would be "Zhang Yi" (family name goes first [1]) according to
The Chinese phonetic alphabet spelling rules for Chinese names [2].Indeed, that is also what the legel name is written in alphabet on our passport or credit/debit cards.
Quoted 7 lines
Also, many official English-written materials use it in that way, for example, a somewhat famous bastetball player Yao Ming [3][4][5].
That is what I wrote my own name as this but I also noticed the western ordering of names is quite common for Chinese people in Linux kernel. Anyway, it's just my preliminary personal thought (might be just my own perference) according to (I think, maybe) formal occasions.
Yeah, there doesn't seem to be a lot of consistency with the ordering of Chinese names when they are written in Roman characters. Some people put the family name first, and other people will put the personal (first) name first. In some cases it may be because the developer in question is living in America, and so they've decided to use the American naming convention. (Two example of this are former ext4 developers Mingming Cao and Jiaying Zhang, who live in Portland and Los Angelos, and their family names are Cao and Zhang, respectively.)
My personal opinion is people should use whatever name they are comfortable with, using whatever characters they prefer. The one thing that would be helpful for me is for people to give a hint about how they would prefer to be called --- for example, would you prefer that start an e-mail with the salutation, "Hi Gao", "Hi Xiang", or "Hi Gao Xiang"?
Quoted 5 lines
And if I don't know, and I guess wrong, please feel free to correct me, either privately, or publically on the e-mail list, if you think it would be helpful for more people to understand how you'd prefer to be called.
Hi, Ted and Xiang
I didn't think about it in depth before, and thanks for your suggestion, I think use "Zhang Yi" is more suitable for me now.
Thanks,
Yi
Hi.
Quoted 26 lines · Theodore Ts'o
Quoted 7 lines · Gao Xiang
I think the legel name would be "Zhang Yi" (family name goes first [1]) according to
The Chinese phonetic alphabet spelling rules for Chinese names [2].Indeed, that is also what the legel name is written in alphabet on our passport or credit/debit cards.
Quoted 7 lines
Also, many official English-written materials use it in that way, for example, a somewhat famous bastetball player Yao Ming [3][4][5].
That is what I wrote my own name as this but I also noticed the western ordering of names is quite common for Chinese people in Linux kernel. Anyway, it's just my preliminary personal thought (might be just my own perference) according to (I think, maybe) formal occasions.
Yeah, there doesn't seem to be a lot of consistency with the ordering of Chinese names when they are written in Roman characters. Some people put the family name first, and other people will put the personal (first) name first. In some cases it may be because the developer in question is living in America, and so they've decided to use the American naming convention. (Two example of this are former ext4 developers Mingming Cao and Jiaying Zhang, who live in Portland and Los Angelos, and their family names are Cao and Zhang, respectively.)
Quoted 6 lines
My personal opinion is people should use whatever name they are comfortable with, using whatever characters they prefer. The one thing that would be helpful for me is for people to give a hint about how they would prefer to be called --- for example, would you prefer that start an e-mail with the salutation, "Hi Gao", "Hi Xiang", or "Hi Gao Xiang"?
Is there a common way to indicate that? Like, erm, SPDX, but for names?
Saying, instead of just writing "Oleksandr Natalenko" in the email footer, I'll write "Oleksandr Natalenko / RMNA:NS" meaning "read my name as name-surname".
Hm?
Quoted 9 lines
And if I don't know, and I guess wrong, please feel free to correct me, either privately, or publically on the e-mail list, if you think it would be helpful for more people to understand how you'd prefer to be called.
Cheers,
- Ted
Signature · 1 line
Oleksandr Natalenko (post-factum)