[CRIU] Re: [PATCH 0/8] Inotify support v7

[Pavel Emelyanov]

On 04/17/2012 08:33 PM, Cyrill Gorcunov wrote:
> On Tue, Apr 17, 2012 at 08:27:29PM +0400, Pavel Emelyanov wrote:
>>>
>>> inotify_new_watch
>>>   dget(dentry);
>>
>> This dget was in the code before your patch, you didn't add it.
>>
> 
> Паш, да как это не добавлял, у меня же в патче
> 
> +#if defined(CONFIG_CHECKPOINT_RESTORE) && defined(CONFIG_EXPORTFS)
> +	dget(dentry);

Уй, бля :( Надо пойти поспать...

> +	tmp_i_mark->fsn_mark.target = dentry;
> +#endif
> 
> и не было его там раньше, inotify уже с инодами работает,
> мне самому пришлось добавить
> 
>  struct fsnotify_mark {
> +#if defined(CONFIG_CHECKPOINT_RESTORE) && defined(CONFIG_EXPORTFS)
> +	struct dentry *target;		/* target which mark is watching on */
> +#endif
> 
> и прицепить на него
> 
> inotify_new_watch
>   dget(dentry);
> 
> ты что?
> 
> 	Cyrill