linux-ext4.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jan Kara <jack@suse.cz>
To: linux-fsdevel@vger.kernel.org
Cc: Dave Kleikamp <shaggy@kernel.org>,
	jfs-discussion@lists.sourceforge.net, tytso@mit.edu,
	Jeff Mahoney <jeffm@suse.de>, Mark Fasheh <mfasheh@suse.com>,
	Dave Chinner <david@fromorbit.com>,
	reiserfs-devel@vger.kernel.org, xfs@oss.sgi.com,
	cluster-devel@redhat.com, Joel Becker <jlbec@evilplan.org>,
	Jan Kara <jack@suse.cz>,
	linux-ext4@vger.kernel.org, hch@lst.de,
	Steven Whitehouse <swhiteho@redhat.com>,
	ocfs2-devel@oss.oracle.com, viro@zeniv.linux.org.uk
Subject: [PATCH 0/12 v4] Moving i_dquot out of struct inode
Date: Tue,  4 Nov 2014 12:19:41 +0100	[thread overview]
Message-ID: <1415099993-18034-1-git-send-email-jack@suse.cz> (raw)

  Hello,

  here is a new version of my patch series which moves i_dquot array from
struct inode into filesystem private part of the inode. Thus filesystems which
don't need it save 2 pointers in their inodes (would be 3 after we add project
quota support into generic quota).

The patch series also contains a change to quotactl so that each filesystem
can set quota types it supports. This is in the end unrelated change
(originally it was necessary for i_dquot moving but in the end I changed
things so that it's not anymore). I can move that into a separate series
but I was somewhat reluctant to do that since that would mean another 6
one-line patches to the same files we are changing here...

If people like the patches, I will queue this series into my tree for
the next merge window. For that I'd prefer to get acks from affected fs
maintainers (the changes are pretty trivial and I don't feel it's a must but
still I'd prefer fs maintainers to ack they are aware of the changes).

								Honza

Changes since v1:
* Inode field names are now named enum
* Quota type masks now have names like QTYPE_MASK_{USR|GRP|PRJ} instead of
  opencoding shifts.

Changes since v2:
* Use ->get_dquots callback instead of inode fields framework
* rebased on Linus' tree as of 3.18-rc1 + something.

Changes since v3:
* Moved bitmask of allowed quota types out of s_dquot into sb itself
* Removed pointless #ifdef CONFIG_QUOTA from OCFS2

------------------------------------------------------------------------------

             reply	other threads:[~2014-11-04 11:19 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-04 11:19 Jan Kara [this message]
2014-11-04 11:19 ` [PATCH 01/12] quota: Remove const from function declarations Jan Kara
2014-11-04 11:19 ` [PATCH 02/12] quota: Allow each filesystem to specify which quota types it supports Jan Kara
2014-11-04 11:19 ` [PATCH 03/12] gfs2: Set allowed quota types Jan Kara
2014-11-04 11:19 ` [PATCH 04/12] xfs: " Jan Kara
2014-11-04 11:19 ` [PATCH 05/12] quota: Use function to provide i_dquot pointers Jan Kara
2014-11-04 11:19 ` [PATCH 06/12] ext2: Convert to private i_dquot field Jan Kara
2014-11-04 11:19 ` [PATCH 07/12] ext3: " Jan Kara
2014-11-04 11:19 ` [PATCH 08/12] ext4: " Jan Kara
2014-11-04 14:24   ` Theodore Ts'o
2014-11-04 11:19 ` [PATCH 09/12] ocfs2: " Jan Kara
2014-11-04 11:19 ` [PATCH 10/12] reiserfs: " Jan Kara
2014-11-04 11:19 ` [PATCH 11/12] jfs: " Jan Kara
2014-11-04 14:17   ` Dave Kleikamp
2014-11-04 11:19 ` [PATCH 12/12] vfs: Remove i_dquot field from inode Jan Kara
2014-11-08 13:21 ` [PATCH 0/12 v4] Moving i_dquot out of struct inode Christoph Hellwig
2014-11-10  9:08   ` Jan Kara

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=1415099993-18034-1-git-send-email-jack@suse.cz \
    --to=jack@suse.cz \
    --cc=cluster-devel@redhat.com \
    --cc=david@fromorbit.com \
    --cc=hch@lst.de \
    --cc=jeffm@suse.de \
    --cc=jfs-discussion@lists.sourceforge.net \
    --cc=jlbec@evilplan.org \
    --cc=linux-ext4@vger.kernel.org \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=mfasheh@suse.com \
    --cc=ocfs2-devel@oss.oracle.com \
    --cc=reiserfs-devel@vger.kernel.org \
    --cc=shaggy@kernel.org \
    --cc=swhiteho@redhat.com \
    --cc=tytso@mit.edu \
    --cc=viro@zeniv.linux.org.uk \
    --cc=xfs@oss.sgi.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).