[CRIU] [PATCH 1/2] tty: add vt support, v2

Ruslan Kuprieiev kupruser at gmail.com
Fri Dec 19 08:46:25 PST 2014



19.12.2014 18:32, Cyrill Gorcunov пишет:
> On Fri, Dec 19, 2014 at 07:28:00PM +0300, Pavel Emelyanov wrote:
>> On 12/19/2014 07:20 PM, Cyrill Gorcunov wrote:
>>> On Fri, Dec 19, 2014 at 07:05:54PM +0300, Pavel Emelyanov wrote:
>>>>
>>>>> diff --git a/tty.c b/tty.c
>>>>> index b076f06..7b184c7 100644
>>>>> --- a/tty.c
>>>>> +++ b/tty.c
>>>>> @@ -118,6 +118,7 @@ static LIST_HEAD(all_ttys);
>>>>>   #define MAX_TTYS	1024
>>>>>   #define MAX_PTY_INDEX	1000
>>>>>   #define CONSOLE_INDEX	1002
>>>>> +#define VT_INDEX	1004
>>>>
>>>> 1004? Why not 1003 or 1005?
>>>
>>> I'm not sure about shell_job, but for indices -- better to
>>> use odd numbers here in case if we need pairs one day,
>>> so leave an even holes here.
>>
>> I don't mind, but let's comment this fact in the code.
>
> OK. Ruslan add a comment please right after #define MAX_TTYS
>
> /*
>   * Custom indices should be odd numbers just in case if we
>   * need evens for pair numbering someday.
>   */
>

Done, see [PATCH 3/2] tty:add comment explaining indices.
Thanks=).


More information about the CRIU mailing list