[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <201203251721.q2PHLjaJ016562@bugzilla.kernel.org>
Date: Sun, 25 Mar 2012 17:21:45 GMT
From: bugzilla-daemon@...zilla.kernel.org
To: linux-ext4@...r.kernel.org
Subject: [Bug 42986] ext2_fs.h requires undefined type umode_t
https://bugzilla.kernel.org/show_bug.cgi?id=42986
--- Comment #5 from Theodore Tso <tytso@....edu> 2012-03-25 17:21:44 ---
It's not going to be installed. Al Viro has patches in his git tree which is
going to make it ***disappear*** from the kernel source tree altogether. The
definitions which are needed for fs/ext2 will be moved to fs/ext2/ext2.h. The
userspace definitions are being deleted. The few places in the rest of the
kernel which were using ext2_fs.h and implying knowledge of the ext2 superblock
layout are being changed to use a function exported by the ext2 code, thus
properly providing better abstraction between the various kernel components.
There should be no valid user of this header, and there should not have been
one for at least eight years. If they do exist, we'll find out when they
scream, and then we'll tell them that they are broken, and they should be using
the ext2_fs.h being shipped with e2fsprogs, and ideally libext2fs being shipped
with e2fsprogs.
The whole point is that it's insane to be maintaining two userspace headers,
especially since the one in the kernel tree has effectively not been maintained
for years (at least for use in userspace).
Again, if you know of other actual users of the ext2_fs.h being shipped in the
kernel, please let me know, and I'll personally reach out to them (and tell
them that they are close to a decade behind the times....)
--
Configure bugmail: https://bugzilla.kernel.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching the assignee of the bug.
--
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