From: Alexander Larsson <alexl@redhat.com>
To: Amir Goldstein <amir73il@gmail.com>, Miklos Szeredi <miklos@szeredi.hu>
Cc: linux-unionfs@vger.kernel.org
Subject: Re: [PATCH 2/7] ovl: use OVL_E() and OVL_E_FLAGS() accessors
Date: Mon, 17 Apr 2023 16:20:15 +0200 [thread overview]
Message-ID: <4893401820bb3339194dc6cd89a43ebc5c505ce5.camel@redhat.com> (raw)
In-Reply-To: <20230408164302.1392694-3-amir73il@gmail.com>
On Sat, 2023-04-08 at 19:42 +0300, Amir Goldstein wrote:
> Instead of open coded instances, because we are about to split
> the two apart.
>
> Signed-off-by: Amir Goldstein <amir73il@gmail.com>
Reviewed-by: Alexander Larsson <alexl@redhat.com>
> ---
> fs/overlayfs/export.c | 2 +-
> fs/overlayfs/namei.c | 8 ++++----
> fs/overlayfs/ovl_entry.h | 5 +++++
> fs/overlayfs/super.c | 2 +-
> fs/overlayfs/util.c | 20 ++++++++++----------
> 5 files changed, 21 insertions(+), 16 deletions(-)
>
> diff --git a/fs/overlayfs/export.c b/fs/overlayfs/export.c
> index 5c36fb3a7bab..2cfdfcca5659 100644
> --- a/fs/overlayfs/export.c
> +++ b/fs/overlayfs/export.c
> @@ -341,7 +341,7 @@ static struct dentry *ovl_obtain_alias(struct
> super_block *sb,
> /* Get the upper or lower dentry in stack whose on layer @idx */
> static struct dentry *ovl_dentry_real_at(struct dentry *dentry, int
> idx)
> {
> - struct ovl_entry *oe = dentry->d_fsdata;
> + struct ovl_entry *oe = OVL_E(dentry);
> int i;
>
> if (!idx)
> diff --git a/fs/overlayfs/namei.c b/fs/overlayfs/namei.c
> index 100a492d2b2a..e66352f19755 100644
> --- a/fs/overlayfs/namei.c
> +++ b/fs/overlayfs/namei.c
> @@ -790,7 +790,7 @@ struct dentry *ovl_lookup_index(struct ovl_fs
> *ofs, struct dentry *upper,
> */
> int ovl_path_next(int idx, struct dentry *dentry, struct path *path)
> {
> - struct ovl_entry *oe = dentry->d_fsdata;
> + struct ovl_entry *oe = OVL_E(dentry);
>
> BUG_ON(idx < 0);
> if (idx == 0) {
> @@ -833,8 +833,8 @@ struct dentry *ovl_lookup(struct inode *dir,
> struct dentry *dentry,
> struct ovl_entry *oe;
> const struct cred *old_cred;
> struct ovl_fs *ofs = dentry->d_sb->s_fs_info;
> - struct ovl_entry *poe = dentry->d_parent->d_fsdata;
> - struct ovl_entry *roe = dentry->d_sb->s_root->d_fsdata;
> + struct ovl_entry *poe = OVL_E(dentry->d_parent);
> + struct ovl_entry *roe = OVL_E(dentry->d_sb->s_root);
> struct ovl_path *stack = NULL, *origin_path = NULL;
> struct dentry *upperdir, *upperdentry = NULL;
> struct dentry *origin = NULL;
> @@ -1157,7 +1157,7 @@ struct dentry *ovl_lookup(struct inode *dir,
> struct dentry *dentry,
>
> bool ovl_lower_positive(struct dentry *dentry)
> {
> - struct ovl_entry *poe = dentry->d_parent->d_fsdata;
> + struct ovl_entry *poe = OVL_E(dentry->d_parent);
> const struct qstr *name = &dentry->d_name;
> const struct cred *old_cred;
> unsigned int i;
> diff --git a/fs/overlayfs/ovl_entry.h b/fs/overlayfs/ovl_entry.h
> index fd11fe6d6d45..4c7312126b3b 100644
> --- a/fs/overlayfs/ovl_entry.h
> +++ b/fs/overlayfs/ovl_entry.h
> @@ -124,6 +124,11 @@ static inline struct ovl_entry *OVL_E(struct
> dentry *dentry)
> return (struct ovl_entry *) dentry->d_fsdata;
> }
>
> +static inline unsigned long *OVL_E_FLAGS(struct dentry *dentry)
> +{
> + return &OVL_E(dentry)->flags;
> +}
> +
> struct ovl_inode {
> union {
> struct ovl_dir_cache *cache; /* directory */
> diff --git a/fs/overlayfs/super.c b/fs/overlayfs/super.c
> index 49b6956468f9..108824b359e6 100644
> --- a/fs/overlayfs/super.c
> +++ b/fs/overlayfs/super.c
> @@ -138,7 +138,7 @@ static int ovl_revalidate_real(struct dentry *d,
> unsigned int flags, bool weak)
> static int ovl_dentry_revalidate_common(struct dentry *dentry,
> unsigned int flags, bool
> weak)
> {
> - struct ovl_entry *oe = dentry->d_fsdata;
> + struct ovl_entry *oe = OVL_E(dentry);
> struct inode *inode = d_inode_rcu(dentry);
> struct dentry *upper;
> unsigned int i;
> diff --git a/fs/overlayfs/util.c b/fs/overlayfs/util.c
> index 6a0652bd51f2..01e6b4ec3074 100644
> --- a/fs/overlayfs/util.c
> +++ b/fs/overlayfs/util.c
> @@ -143,7 +143,7 @@ bool ovl_dentry_weird(struct dentry *dentry)
>
> enum ovl_path_type ovl_path_type(struct dentry *dentry)
> {
> - struct ovl_entry *oe = dentry->d_fsdata;
> + struct ovl_entry *oe = OVL_E(dentry);
> enum ovl_path_type type = 0;
>
> if (ovl_dentry_upper(dentry)) {
> @@ -176,7 +176,7 @@ void ovl_path_upper(struct dentry *dentry, struct
> path *path)
>
> void ovl_path_lower(struct dentry *dentry, struct path *path)
> {
> - struct ovl_entry *oe = dentry->d_fsdata;
> + struct ovl_entry *oe = OVL_E(dentry);
>
> if (oe->numlower) {
> path->mnt = oe->lowerstack[0].layer->mnt;
> @@ -188,7 +188,7 @@ void ovl_path_lower(struct dentry *dentry, struct
> path *path)
>
> void ovl_path_lowerdata(struct dentry *dentry, struct path *path)
> {
> - struct ovl_entry *oe = dentry->d_fsdata;
> + struct ovl_entry *oe = OVL_E(dentry);
>
> if (oe->numlower) {
> path->mnt = oe->lowerstack[oe->numlower - 1].layer-
> >mnt;
> @@ -231,14 +231,14 @@ struct dentry *ovl_dentry_upper(struct dentry
> *dentry)
>
> struct dentry *ovl_dentry_lower(struct dentry *dentry)
> {
> - struct ovl_entry *oe = dentry->d_fsdata;
> + struct ovl_entry *oe = OVL_E(dentry);
>
> return oe->numlower ? oe->lowerstack[0].dentry : NULL;
> }
>
> const struct ovl_layer *ovl_layer_lower(struct dentry *dentry)
> {
> - struct ovl_entry *oe = dentry->d_fsdata;
> + struct ovl_entry *oe = OVL_E(dentry);
>
> return oe->numlower ? oe->lowerstack[0].layer : NULL;
> }
> @@ -251,7 +251,7 @@ const struct ovl_layer *ovl_layer_lower(struct
> dentry *dentry)
> */
> struct dentry *ovl_dentry_lowerdata(struct dentry *dentry)
> {
> - struct ovl_entry *oe = dentry->d_fsdata;
> + struct ovl_entry *oe = OVL_E(dentry);
>
> return oe->numlower ? oe->lowerstack[oe->numlower - 1].dentry
> : NULL;
> }
> @@ -329,17 +329,17 @@ void ovl_set_dir_cache(struct inode *inode,
> struct ovl_dir_cache *cache)
>
> void ovl_dentry_set_flag(unsigned long flag, struct dentry *dentry)
> {
> - set_bit(flag, &OVL_E(dentry)->flags);
> + set_bit(flag, OVL_E_FLAGS(dentry));
> }
>
> void ovl_dentry_clear_flag(unsigned long flag, struct dentry
> *dentry)
> {
> - clear_bit(flag, &OVL_E(dentry)->flags);
> + clear_bit(flag, OVL_E_FLAGS(dentry));
> }
>
> bool ovl_dentry_test_flag(unsigned long flag, struct dentry *dentry)
> {
> - return test_bit(flag, &OVL_E(dentry)->flags);
> + return test_bit(flag, OVL_E_FLAGS(dentry));
> }
>
> bool ovl_dentry_is_opaque(struct dentry *dentry)
> @@ -1015,7 +1015,7 @@ int ovl_check_metacopy_xattr(struct ovl_fs
> *ofs, const struct path *path)
>
> bool ovl_is_metacopy_dentry(struct dentry *dentry)
> {
> - struct ovl_entry *oe = dentry->d_fsdata;
> + struct ovl_entry *oe = OVL_E(dentry);
>
> if (!d_is_reg(dentry))
> return false;
--
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
=-=-=
Alexander Larsson Red Hat,
Inc
alexl@redhat.com alexander.larsson@gmail.com
He's a one-legged devious stage actor for the 21st century. She's a
mistrustful blonde research scientist prone to fits of savage,
blood-crazed rage. They fight crime!
next prev parent reply other threads:[~2023-04-17 14:21 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-04-08 16:42 [PATCH 0/7] Prepare for lazy lowerdata lookup Amir Goldstein
2023-04-08 16:42 ` [PATCH 1/7] ovl: update of dentry revalidate flags after copy up Amir Goldstein
2023-04-14 7:21 ` Gao Xiang
2023-04-27 5:48 ` Amir Goldstein
2023-04-27 6:11 ` Gao Xiang
2023-04-08 16:42 ` [PATCH 2/7] ovl: use OVL_E() and OVL_E_FLAGS() accessors Amir Goldstein
2023-04-17 14:20 ` Alexander Larsson [this message]
2023-04-08 16:42 ` [PATCH 3/7] ovl: use ovl_numlower() and ovl_lowerstack() accessors Amir Goldstein
2023-04-17 14:38 ` Alexander Larsson
2023-04-08 16:42 ` [PATCH 4/7] ovl: factor out ovl_free_entry() and ovl_stack_*() helpers Amir Goldstein
2023-04-17 15:00 ` Alexander Larsson
2023-04-08 16:43 ` [PATCH 5/7] ovl: move ovl_entry into ovl_inode Amir Goldstein
2023-04-18 7:55 ` Alexander Larsson
2023-04-27 6:07 ` Amir Goldstein
2023-04-08 16:43 ` [PATCH 6/7] ovl: deduplicate lowerpath and lowerstack[0] Amir Goldstein
2023-04-18 8:10 ` Alexander Larsson
2023-04-27 6:29 ` Amir Goldstein
2023-04-26 12:07 ` Miklos Szeredi
2023-04-27 6:27 ` Amir Goldstein
2023-04-08 16:43 ` [PATCH 7/7] ovl: replace lowerdata inode reference with lowerdata redirect Amir Goldstein
2023-04-18 8:19 ` Alexander Larsson
2023-04-26 12:38 ` Miklos Szeredi
2023-04-27 6:32 ` Amir Goldstein
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=4893401820bb3339194dc6cd89a43ebc5c505ce5.camel@redhat.com \
--to=alexl@redhat.com \
--cc=amir73il@gmail.com \
--cc=linux-unionfs@vger.kernel.org \
--cc=miklos@szeredi.hu \
/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