On 10/26/2012 06:44 PM, Carlos Maiolino wrote:

Hi,

On Thu, Oct 25, 2012 at 05:08:19PM +0530, Abhijit Pawar wrote:

quoted

Hi,
set_anon_super is called by many filesystems. Some call directly and
some call through the wrapper. Many of them in the wrapper's call to
this function are passing the second argument to this function which
is not used anywhere.

This patch replaces the second variable with NULL.

If the variable isn't used anymore, why don't just get rid of it, instead of
call the function passing a NULL pointer on it?

At the moment its a callback function with two params so that 
filesystems are free to override it while mounting. This is to support 
filesystem specific information at mount time.

nfs uses it to get its server specific information. btrfs uses it to 
populate its filesystem information. So the signature can not be changed 
without affecting these filesystems.

sysfs, ceph, 9p will also be affected if we are to change the signature.

-- 
-
Abhijit