lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <20130225182516.GI22221@lenny.home.zabbo.net> Date: Mon, 25 Feb 2013 10:25:16 -0800 From: Zach Brown <zab@...bo.net> To: Al Viro <viro@...IV.linux.org.uk> Cc: Li Zefan <lizefan@...wei.com>, linux-fsdevel@...r.kernel.org, LKML <linux-kernel@...r.kernel.org>, Ext4 Developers List <linux-ext4@...r.kernel.org>, Jan Kara <jack@...e.cz>, Theodore Ts'o <tytso@....edu>, Andrew Morton <akpm@...ux-foundation.org>, andi@...stfloor.org, Wuqixuan <wuqixuan@...wei.com>, gregkh@...uxfoundation.org, Linus Torvalds <torvalds@...ux-foundation.org> Subject: Re: [RFC] f_pos in readdir() (was Re: [RFC][PATCH] vfs: always protect diretory file->fpos with inode mutex) > As for ->readdir(), I'd like to resurrect an old proposal to change the ABI > of that sucker. Quoting the thread from 4 years ago: I'd love to see the readdir() interface cleaned up, yes please. > Comments? Hmm. Do we want to think about letting callers copy the name to userspace in fragments? I was looking at current readdir paths and saw that btrfs has an annoying wrinkle that its directory entries might be in large blocks that span disjoint pages. So it copies them to a kmalloced()ed buffer to pass to filldir(). Maybe it wouldn't be worth having to track incomplete entries in the core, but it'd be nice to get rid of this copy in btrfs. (btrfs could also be more clever and only do this dance when the name really does span pages.. it's very rare. :/). - z -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists