[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