From: "Theodore Tso" <tytso@mit.edu>
To: eaujames@ddn.com
Cc: linux-ext4@vger.kernel.org, adilger@thelustrecollective.com,
dongyangli@ddn.com
Subject: Re: [PATCH 2/4] libext2fs: add quota to libext2fs
Date: Wed, 12 Aug 2026 22:43:02 -0400 [thread overview]
Message-ID: <anyHu4U8pf5iMW96@mit.edu> (raw)
In-Reply-To: <ahl_GhW8EhYTEun7@eaujamesFR0130>
On Fri, May 29, 2026 at 01:57:14PM -0500, eaujames@ddn.com wrote:
> add quota related interface to libext2fs and install the
> relevant headers.
The reason why the quota functions are in the libsupport library as
opposed to the libext2fs directory is that the quota functions weren't
ready to be moved into libext2fs. The libext2fs library is published
as a shared library, so the function ABI's must be stable. This
requirement does not exist for libsupport.a (which is only statically
linked, so function stability is not a requirement). The functions in
libsupport also in some cases try to write to the standard out, which
is not allowed for libext2fs.
As a result of this, the quota and ea_inode feature is not directly
supported by libext2fs, and hence, not by programs like fuse2fs.
So to address the problem you are trying to solve in this patch series
--- which is to allow e2fsck to correctly handle processing processing
an orphaned file which has an extended attribute value stored in an
inode and quota tracking enabled --- we have two choices
* Clean up the quota handling functions so we can move safely move
them from lib/support to lib/ext2fs. This is the long-term plan /
goal, since it would also allow us to add support for ea_inode and
quota into programs like fuse2fs.
* Don't add the functionality to lib/ext2fs, but just arrange to have
release_orphan_inode() in e2fsck/super.c call the functions in
lib/support as necessary. (This is how we handle quota support in
e2fsck today. We just didn't deal with the combination of ea_inode
| quota correctly.)
- Ted
prev parent reply other threads:[~2026-08-13 2:44 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-05-29 11:57 [PATCH 2/4] libext2fs: add quota to libext2fs eaujames
2026-08-13 2:43 ` Theodore Tso [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=anyHu4U8pf5iMW96@mit.edu \
--to=tytso@mit.edu \
--cc=adilger@thelustrecollective.com \
--cc=dongyangli@ddn.com \
--cc=eaujames@ddn.com \
--cc=linux-ext4@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.