On Tue, 23 May 2023 23:49:47 -0400, Theodore Ts'o wrote:
> This fixes a number of problems with ea_inode handling which were
> pointed out by syzbot. The first and third add some additional
> checking for invalid / maliciously fuzzed file systems. The second
> and fourth patch adds some lockdep annotations to avoid some false
> positive reports from lockdep.
>
> There is still one remaining syzbot report[1] relating to ea_inodes
> not handled by this patch series, and that is an apparently deadlock
> which happens when a kernel thread is freeing an ea_inode racing with
> another thread which is trying to find the mbcache entry (presumably
> with the intent of reusing it). The problem is apparently hard to
> reproduce; it's only been hit 4 times, and there is no C reproducer;
> just a syzkaller reproducer. So we'll leave that for another day/
>
> [...]
Applied, thanks!
[1/4] ext4: add EA_INODE checking to ext4_iget()
commit: b3e6bcb94590dea45396b9481e47b809b1be4afa
[2/4] ext4: set lockdep subclass for the ea_inode in ext4_xattr_inode_cache_find()
commit: d08927b3e89fde1b224d22d2bddcb8dc4fe616db
[3/4] ext4: disallow ea_inodes with extended attributes
commit: 1e0e51238f151e26ccd0a8bd5f5cf32e85c19ac3
[4/4] ext4: add lockdep annotations for i_data_sem for ea_inode's
commit: f901459a1f277ed921e255d4c3d54485769f7dbd
Best regards,
--
Theodore Ts'o <tytso@xxxxxxx>
