[CRIU] [PATCH 08/17] fsnotify: Add try_open_handle helper

Pavel Emelyanov xemul at parallels.com
Mon Dec 24 08:47:26 EST 2012


On 12/24/2012 05:43 PM, Cyrill Gorcunov wrote:
> On Mon, Dec 24, 2012 at 05:19:53PM +0400, Pavel Emelyanov wrote:
>> On 12/11/2012 01:34 AM, Cyrill Gorcunov wrote:
>>>
>>> The helper chooses which path to get, if remapped
>>> name present (ghost file) we use it, otherwise we
>>> try to open file handle.
>>>
>>> The reason for separate helper is that both inotify
>>> and fanotify will use it, so we should try to eliminate
>>> code duplication.
>>>
>>> Signed-off-by: Cyrill Gorcunov <gorcunov at openvz.org>
>>
>> This does more than just split one fn into two. Don't merge these
>> two changes in one patch.
> 
> Hmm, not sure if I follow here. Could you please elaborate?
> We've added try_open_handle helper here and used it in
> restore_one_fsnotify, what's wrong?
> 
> 	Cyrill
> .
> 


You moved piece of restore_one_fsnotify in try_open_handle and
in the same patch -- piece of try_open_handle inside one of the
if-s branches. Let's split these two changed into two patches.


More information about the CRIU mailing list