Re: [PATCH 06/13] SQUASHME pnfs_submit: remove uninitialize_mountpoint

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

 



On 04/29/2010 07:27 PM, Boaz Harrosh wrote:
> On 04/29/2010 01:24 AM, andros@xxxxxxxxxx wrote:
>> From: Andy Adamson <andros@xxxxxxxxxx>
>>
>> The layoutdriver_io_operations uninitialize_mountpoint function is used
>> to free a layout driver specific device id cache.
>> The device id cache is now shared and moved to struct nfs_client and can be
>> removed in the generic unmount_pnfs_layoutdriver routine.
>>
>> Signed-off-by: Andy Adamson <andros@xxxxxxxxxx>
> 
> What ? no.
> 
> uninitialize_mountpoint is where the mountid private data is deallocated.
> The one that was allocated at initialize_mountpoint.
> 
> Files layout might not currently have any, but we do and it will has well.
> Any way what kind of API has initialize call and not an uninitialize call
> Even if it's empty
> 
> NACK
> 

You are right that currently it's only device cache things. But I have code in
Q that have a page pool cache for RAID5/6 IO, and other initialization I need.

Please keep the initialize and a symmetric uninitialize and keep some private
data for each sb. It does not cost any thing. You could get rid of the double
indirection though.

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

[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux