Re: [PATCH] cifs-utils: new plugin architecture for ID mapping code

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 2012-12-07 at 10:30 -0500, Jeff Layton wrote:
> 
> Ok, first "official" posting for the plugin architecture. Changes
> since
> the original RFC set:
> 
> 1/ init_plugin() and exit_plugin() functions have been added to the
>    plugin API. The init passes back an opaque handle so we can
>    potentially use this in threaded programs later if needed.
> 
> 2/ the plugin function names are more distinct, now prefixed with
>    "cifs_idmap_".
> 
> 3/ the code has been changed to never dlclose() the plugin and rely
>    on it to be closed at exit. That makes things a lot simpler...
> 
> Original description follows...

Looks generally good to me, I like where you painted the line between
functionality and complexity :)

Simo.

-- 
Simo Sorce
Samba Team GPL Compliance Officer <simo@xxxxxxxxx>
Principal Software Engineer at Red Hat, Inc. <simo@xxxxxxxxxx>

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


[Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux