[CRIU] Re: [PATCH] parasite-syscall: some cleanup

Cyrill Gorcunov gorcunov at openvz.org
Tue Feb 21 10:57:03 EST 2012


On Tue, Feb 21, 2012 at 07:49:22PM +0400, Kinsbursky Stanislav wrote:
...
> >>Preparing arguments before execution call looks straight-forward to me.
> >>You, confused, because cmd will be copied to ctl internals inside
> >>parasite_execute(), don't you?
> >>Yes, it will. But if we will left only data argument, then these calls will not
> >>look as clear current solution.
> >>But, anyway, I don't really insist here.
> >>Let's wait for third opinion, what do you think?
> >>
> >Ну да, выглядит как-то мутновато. Придумай что-нибудь более веселое :)
> 
> Ну вы, блин, даёте, ребята.
> Сами нафигачили чёрте-чего, не прибрали, я теперь вам, видишь ли, "мутновато".
> Если убрать команду-аргумент - не думаю, что будет сильно выглядеть веселее.
> Но как хочите.
> 

Стас, на все что связано с паразитом: главная метрика -- должна быть простая
отладка, т.е. я бы даже оставил лишнее копирование аргументов и т.п. в ущерб
производительности (там, на самом деле размеры аргументов небольшие и копируются
на раз-два), если это сделает код удобочитаемым.

	Cyrill


More information about the CRIU mailing list