Re: [PATCH 08/13] RFC: pnfs: filelayout: introduce minimal file layout driver

Fred Isaman
2010-09-13T15:16:02Z

On Mon, Sep 13, 2010 at 8:08 AM, Christoph Hellwig <hch@infradead.org> =
wrote:
> On Thu, Sep 02, 2010 at 02:00:14PM -0400, Fred Isaman wrote:
>> +static int __init nfs4filelayout_init(void)
>> +{
>> + =A0 =A0 printk(KERN_INFO "%s: NFSv4 File Layout Driver Registering=
=2E..\n",
>> + =A0 =A0 =A0 =A0 =A0 =A0__func__);
>> +
>> + =A0 =A0 /*
>> + =A0 =A0 =A0* Need to register file_operations struct with global l=
ist to indicate
>> + =A0 =A0 =A0* that NFS4 file layout is a possible pNFS I/O module
>> + =A0 =A0 =A0*/
>> + =A0 =A0 return pnfs_register_layoutdriver(&filelayout_type);
>
> And I thought we were dealing with an UFO detection device here..
>
> Sarcasm aside, this is one of the most pointless and incorrect commen=
ts I've
> seen for a while.
>
>> + =A0 =A0 /* Unregister NFS4 file layout driver with pNFS client*/
>> + =A0 =A0 pnfs_unregister_layoutdriver(&filelayout_type);
>
> This one is not too useful either.
>

OK

=46red

> --
> To unsubscribe from this list: send the line "unsubscribe linux-nfs" =
in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at =A0http://vger.kernel.org/majordomo-info.html
>