Re: [git patches] Ocfs2 and Configfs updates for 2.6.27
From: Linus Torvalds
Date: Thu Jul 17 2008 - 14:02:50 EST
On Thu, 17 Jul 2008, Mark Fasheh wrote:
> The only two things that could be described as features would be:
> - Joel updated configfs's ->make_item() and ->make_group() callbacks to
> return more descriptive error types. This update is responsible for the
> changes to drivers/net/netconsole.c and fs/dlm/config.c as shown in the diffstat.
.. I really dislike how this was done.
11c3b79218390a139f2d474ee1e983a672d5839a ("configfs: Allow ->make_item()
and ->make_group() to return detailed errors.")
is just ugly. Returning one value by passing a pointer to a pointer, and
the error value by returning an 'int' errno is _not_ how we do things in
No, we use error pointers.
IOW, you should have left the calling convention for "(*make_item)()" and
"(*make_group)()" alone, but allowed people to just do
etc for errors, and then in the caller you just check for IS_ERR().
That would have made the result much cleaner, and avoided that whole
"return two different things" issue.
I pulled it, but I want to register my protest.
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/