[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Pine.LNX.4.61.0609040935070.9108@yvahk01.tjqt.qr>
Date: Mon, 4 Sep 2006 09:37:09 +0200 (MEST)
From: Jan Engelhardt <jengelh@...ux01.gwdg.de>
To: Josef Sipek <jsipek@...sunysb.edu>
cc: linux-kernel@...r.kernel.org, linux-fsdevel@...r.kernel.org,
hch@...radead.org, akpm@...l.org, viro@....linux.org.uk
Subject: Re: [PATCH 15/22][RFC] Unionfs: Privileged operations workqueue
>+void __unionfs_create(void *data)
>+{
>+ struct sioq_args *args = data;
>+
>+ args->err = vfs_create(args->u.create.parent, args->u.create.dentry,
>+ args->u.create.mode, args->u.create.nd);
>+ complete(&args->comp);
>+}
Suggestion
{
struct sioq_args *args = data;
struct create_args *c = &args->u.create;
args->err = vfs_create(c->parent, c->dentry, c->mode, c->nd);
complete(&args->comp);
}
Similar for others.
>+ union {
>+ struct deletewh_args deletewh;
>+ struct isopaque_args isopaque;
>+ struct create_args create;
>+ struct mkdir_args mkdir;
>+ struct mknod_args mknod;
>+ struct symlink_args symlink;
>+ struct unlink_args unlink;
>+ } u;
Anonymous unions (and structs) are allowed, use them if you think they sound
cool.
Jan Engelhardt
--
--
VGER BF report: H 0.127232
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists