lkml.org 
[lkml]   [2006]   [Sep]   [4]   [last100]   RSS Feed
Views: [more markup]  [less markup]  [headers]  [forward] 
 
Messages in this thread
/
DateMon, 4 Sep 2006 09:37:09 +0200 (MEST)
FromJan Engelhardt <>
SubjectRe: [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@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2006-09-04 09:43    [from the cache]
©2003-2008