[CRIU] Re: [PATCH] Rework parasite code

[Cyrill Gorcunov]

On Fri, Feb 17, 2012 at 04:48:51PM +0400, Pavel Emelyanov wrote:
> On 02/17/2012 04:38 PM, Cyrill Gorcunov wrote:
> > On Fri, Feb 17, 2012 at 04:34:51PM +0400, Pavel Emelyanov wrote:
> > ...
> >>
> >> Split it please.
> >>
> > 
> > Pavel, it's almost impossible to split since the changes are
> > really bound. So I've merged it. Lets move on (there were obvious
> > errors in the former code too, like trying to cure seized task
> > from IP where parasite code lives itself, which is bloody wrong,
> > since we have to cure it only via IP where first syscall was injected).
> 
> Тогда убери из патча non-relevant изменения. Вот список того, что к патчу
> ну никакого отношения не имеет.
> 
> * Пододвинутые комментарии в struct parasite_ctl
> * Переформатированные parasite_dump_pages_seized, parasite_dump_misc_seized, etc.
> * Пододвинутые get_vma_by_ip, can_run_syscall
> * Подчищенный код в parasite_file_cmd и иже с ним
> 
> И это только при первом проходе.
> 

Паш, код уже залит, оттестирован и работает. То что было до этого,
было неудобочитаемо. Более того, большая часть кода там сидела
с первых прототипов, поэтому функции работали фактически независимо
одна от другой (иначе отлаживать было совсем не возможно). После
чего, ты выслал серию, которая серьезно подчистила код, но тем не
менее, кой-какие плюхи остались (и в частности, inflight signals).
поэтому я переделал все. Теперь можно просто подчистить поверх все
что нам не нравится.

	Cyrill