[CRIU] [PATCH v2] filemap: collect mapped file fs type

[Stanislav Kinsburskiy]

29.07.2016 04:37, Andrew Vagin пишет:
> On Thu, Jul 28, 2016 at 12:06:41PM +0300, Stanislav Kinsburskiy wrote:
>> This is essential to define whether this file is located on NFS (and thus has
>> to be "link-remapped") or not (see nfs_silly_renamed).
>>
>> v2:
>> Compilation issue fixed
>>
>> Signed-off-by: Stanislav Kinsburskiy <skinsbursky at virtuozzo.com>
>> ---
>>   criu/cr-dump.c |    8 ++++++++
>>   1 file changed, 8 insertions(+)
>>
>> diff --git a/criu/cr-dump.c b/criu/cr-dump.c
>> index 06ff2d7..2288908 100644
>> --- a/criu/cr-dump.c
>> +++ b/criu/cr-dump.c
>> @@ -365,12 +365,20 @@ static int dump_filemap(struct vma_area *vma_area, int fd)
>>   	struct fd_parms p = FD_PARMS_INIT;
>>   	VmaEntry *vma = vma_area->e;
>>   	int ret = 0;
>> +	struct statfs fst;
>>   	u32 id;
>>   
>>   	BUG_ON(!vma_area->vmst);
>>   	p.stat = *vma_area->vmst;
>>   	p.mnt_id = vma_area->mnt_id;
>>   
>> +	if (fstatfs(fd, &fst)) {
>> +		pr_perror("Unable to statfs fd %d", fd);
>> +		return -1;
>> +	}
>> +
>> +	p.fs_type = fst.f_type;
> Can we get fs_type in handle_vma()? We get vmst and mnt_id there.

What do you mean?
Call fstatfs in handle_vma() or use of mnt_id?
And why we should do it in handle_vma?

>> +
>>   	/*
>>   	 * AUFS support to compensate for the kernel bug
>>   	 * exposing branch pathnames in map_files.
>>
>> _______________________________________________
>> CRIU mailing list
>> CRIU at openvz.org
>> https://lists.openvz.org/mailman/listinfo/criu