[devel] Проблема: kernel -> file_operations

[devel] Проблема: kernel -> file_operations

[Денис Смирнов]

[-- Attachment #1: Type: text/plain, Size: 455 bytes --]

В zaptel-base.c используется структура file_operations. В ней readv/writev
инициализируются (устанавливаются в NULL).

На наших ядрах 2.6.24 это не работает, ибо в в linux/fs.h описание
file_operations не содержит этих полей. В 2.6.18 -- содержит.

Соответственно с 2.6.24 код не компилируется.

Что с этим делать?

-- 
С уважением, Денис

http://freesource.info
----------------------------------------------------------------------------

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 197 bytes --]

Re: [devel] Проблема: kernel -> file_operations

[Михаил Якушин]

Денис Смирнов wrote:
> В zaptel-base.c используется структура file_operations. В ней readv/writev
> инициализируются (устанавливаются в NULL).
> 
> На наших ядрах 2.6.24 это не работает, ибо в в linux/fs.h описание
> file_operations не содержит этих полей. В 2.6.18 -- содержит.
> 
> Соответственно с 2.6.24 код не компилируется.
> 
> Что с этим делать
Раз в fs.h их нет, значит они больше не нужны, раз они устанавливаются в 
NULL значит они и самому драйверу не нужны. Думаю что стоит проверить 
действительно сам драйвер не использует readv/writev(покрайней мере он 
может где-то в другом месте readv/writev устанвливать в какое то 
значение)  если нет то убрать это обнуление.

Re: [devel] Проблема: kernel -> file_operations

[Sergey Vlasov]

[-- Attachment #1: Type: text/plain, Size: 658 bytes --]

On Tue, Mar 25, 2008 at 09:01:47AM +0300, Денис Смирнов wrote:
> В zaptel-base.c используется структура file_operations. В ней readv/writev
> инициализируются (устанавливаются в NULL).
> 
> На наших ядрах 2.6.24 это не работает, ибо в в linux/fs.h описание
> file_operations не содержит этих полей. В 2.6.18 -- содержит.

Правильно - эти методы удалили ещё в 2.6.19.  Апстрим с тех пор так и
не озадачился поддержкой свежих ядер?

> Соответственно с 2.6.24 код не компилируется.
> 
> Что с этим делать?

Если это только NULL - просто удалить инициализацию этих полей;
неуказанные поля в static-структуре будут установлены в 0
автоматически.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] Проблема: kernel -> file_operations

[Денис Смирнов]

[-- Attachment #1: Type: text/plain, Size: 786 bytes --]

On Tue, Mar 25, 2008 at 01:30:35PM +0300, Sergey Vlasov wrote:

>> На наших ядрах 2.6.24 это не работает, ибо в в linux/fs.h описание
>> file_operations не содержит этих полей. В 2.6.18 -- содержит.
SV> Правильно - эти методы удалили ещё в 2.6.19.  Апстрим с тех пор так и
SV> не озадачился поддержкой свежих ядер?

Виноват оказался OSLEC-патч (поддержка человеческого программного
эхоподавления). 

>> Соответственно с 2.6.24 код не компилируется.
>> Что с этим делать?
SV> Если это только NULL - просто удалить инициализацию этих полей;
SV> неуказанные поля в static-структуре будут установлены в 0
SV> автоматически.

Сделал, спасибо.

-- 
С уважением, Денис

http://freesource.info
----------------------------------------------------------------------------

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 197 bytes --]

Re: [devel] Проблема: kernel -> file_operations

[Денис Смирнов]

[-- Attachment #1: Type: text/plain, Size: 637 bytes --]

On Tue, Mar 25, 2008 at 09:49:31AM +0300, Михаил Якушин wrote:

МЯ> Раз в fs.h их нет, значит они больше не нужны, раз они устанавливаются в 
МЯ> NULL значит они и самому драйверу не нужны. Думаю что стоит проверить 
МЯ> действительно сам драйвер не использует readv/writev(покрайней мере он 
МЯ> может где-то в другом месте readv/writev устанвливать в какое то значение)  
МЯ> если нет то убрать это обнуление.

Ok, спасибо.

P.S. У меня нет прав на выкладывание модулей zaptel для xen-ядер :(

-- 
С уважением, Денис

http://freesource.info
----------------------------------------------------------------------------

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 197 bytes --]