2021-04-23 21:00:36

by Dai Ngo

Subject: [PATCH v4 0/2] NFSD: delay unmount source's export after inter-server copy completed.

Hi Olga,

Currently the source's export is mounted and unmounted on every
inter-server copy operation. This causes unnecessary overhead
for each copy.

This patch series is an enhancement to allow the export to remain
mounted for a configurable period (default to 15 minutes). If the
export is not being used for the configured time it will be unmounted
by a delayed task. If it's used again then its expiration time is
extended for another period.

Since mount and unmount are no longer done on every copy request,
the restriction of copy size (14*rsize), in __nfs4_copy_file_range,
is removed.

-Dai

v2: fix compiler warning of missing prototype.
v3: remove the used of semaphore.
eliminated all RPC calls for subsequence mount by allowing
all exports from one server to share one vfsmount.
make inter-server threshold a module configuration parameter.
v4: convert nsui_refcnt to use refcount_t.
add note about 20secs wait in nfsd4_interssc_connect.
removed (14*rsize) restriction from __nfs4_copy_file_range.

2021-05-03 16:39:56

by Dai Ngo

Subject: Re: [PATCH v4 0/2] NFSD: delay unmount source's export after inter-server copy completed.

Hi Olga,

Just a reminder that v4 patch is available for your review.

Thanks,

-Dai

On 4/23/21 1:59 PM, Dai Ngo wrote:
> Hi Olga,
>
> Currently the source's export is mounted and unmounted on every
> inter-server copy operation. This causes unnecessary overhead
> for each copy.
>
> This patch series is an enhancement to allow the export to remain
> mounted for a configurable period (default to 15 minutes). If the
> export is not being used for the configured time it will be unmounted
> by a delayed task. If it's used again then its expiration time is
> extended for another period.
>
> Since mount and unmount are no longer done on every copy request,
> the restriction of copy size (14*rsize), in __nfs4_copy_file_range,
> is removed.
>
> -Dai
>
> v2: fix compiler warning of missing prototype.
> v3: remove the used of semaphore.
> eliminated all RPC calls for subsequence mount by allowing
> all exports from one server to share one vfsmount.
> make inter-server threshold a module configuration parameter.
> v4: convert nsui_refcnt to use refcount_t.
> add note about 20secs wait in nfsd4_interssc_connect.
> removed (14*rsize) restriction from __nfs4_copy_file_range.
>
>
>

2021-05-12 17:42:29

by Dai Ngo

Subject: Re: [PATCH v4 0/2] NFSD: delay unmount source's export after inter-server copy completed.

On 5/3/21 9:37 AM, [email protected] wrote:

> Hi Olga,
>
> Just a reminder that v4 patch is available for your review.

Ping again in case you did not get the last reminder.

-Dai

>
> Thanks,
>
> -Dai
>
> On 4/23/21 1:59 PM, Dai Ngo wrote:
>> Hi Olga,
>>
>> Currently the source's export is mounted and unmounted on every
>> inter-server copy operation. This causes unnecessary overhead
>> for each copy.
>>
>> This patch series is an enhancement to allow the export to remain
>> mounted for a configurable period (default to 15 minutes). If the
>> export is not being used for the configured time it will be unmounted
>> by a delayed task. If it's used again then its expiration time is
>> extended for another period.
>>
>> Since mount and unmount are no longer done on every copy request,
>> the restriction of copy size (14*rsize), in __nfs4_copy_file_range,
>> is removed.
>>
>> -Dai
>>
>> v2: fix compiler warning of missing prototype.
>> v3: remove the used of semaphore.
>>      eliminated all RPC calls for subsequence mount by allowing
>>         all exports from one server to share one vfsmount.
>>      make inter-server threshold a module configuration parameter.
>> v4: convert nsui_refcnt to use refcount_t.
>>      add note about 20secs wait in nfsd4_interssc_connect.
>>      removed (14*rsize) restriction from __nfs4_copy_file_range.
>>
>>
>>