GitLab

Не понимаю зачем сравнение, а после этого xor. Если функция ничего не считала, то она на стеке так и останется 0 (можно ничего не проверять)

syscall read может вернуть -1 в случае ошибки

Согласен, тогда лучше проверять...

Функция не вызывает другие функции, поэтому не стоит использовать caller-saved registers.

Если я правильно понимаю то к сисколлам применяются такие же конвенции как и к вызову функций и я не могу гарантировать что у меня остальные регистры сохранятся, это не так?

Это не так, почитайте какие регистры может изменить syscall

Не нашел что бы при сисколле гарантировали что с регистрами в которых передаются параметры ничего не случиться, в чате посоветовали к сисколлам относиться как к частным случаям вызова функций, так что считаю что здесь использование caller-saved регистров имеет смысл

В документации явно говорится о изменении зачения регистров rcx, r11 и rax. В остальном они следуют соглашениям о вызовах.

The Linux AMD64 kernel uses internally the same calling conventions as user level applications (see section 3.2.3 for details). User-level applications that like to call system calls should use the functions from the C library. The interface between the C library and the Linux kernel is the same as for the user-level applications with the following differences:

1. User-level applications use as integer registers for passing the sequence %rdi, %rsi, %rdx, %rcx, %r8 and %r9. Thekernelinterfaceuses %rdi, %rsi, %rdx, %r10, %r8 and %r9.
2. A system-call is done via the syscall instruction. The kernel destroys registers %rcx and %r11.
3. The number of the syscall has to be passed in register %rax.
4. System-calls are limited to six arguments, no argument is passed directly on the stack.
5. Returning from the syscall, register %rax contains the result of the system-call. A value in the range between-4095 and-1 indicates an error, it is-errno.
6. Only values of class INTEGER or class MEMORYarepassed to the kernel.

Лучше сразу копировать и считать длину.

Не совсем согласен, при больших буферах и частых неправильных буферах будем много времени зря тратить на перекладывание битиков, но учитывая что у нас учебный проект, переделал

Данные все-равно считываются, а запись на современном процессоре происходить относительно быстро, так как она не блокирует остальные действия. Может быть вы и правы, но хотелось бы иметь какие-то цифры.

ревью зачтено

added 1 commit

• a13d3c53 - fix errors

Compare with previous version

@andrew

Приянто

closed