[devel] linux/drivers/usb/scanner.c

[devel] linux/drivers/usb/scanner.c

[Sergey Vlasov]

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

Hello!

Какая версия драйвера USB-сканеров
(linux/drivers/usb/scanner.[ch]) сейчас в свежем ядре? В версии
0.4.7 с http://www1.jump.net/~dnelson/linux/usb/ есть серьезные
баги, попробую их удавить.

Посмотрите, пожалуйста - если файлы в ядре отличаются от 0.4.7 с
сайта, бросьте их мне. А то лимит трафика в этом месяце я уже и
так перебрал. Так что придется испытывать на 2.4.15.

-- 
Сергей Власов

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

Re: [devel] linux/drivers/usb/scanner.c

[Sergey Vlasov]

[-- Attachment #1.1: Type: text/plain, Size: 3928 bytes --]

Hello!

Итак, в  версии 0.4.7 драйвера USB-сканеров найдена куча багов,
которые должен бы исправить прилагаемый патч.

1. close_scanner: race при проверке p_scn_table[scn_minor] -
проверка выполнялась без захвата scn_mutex, что могло привести к
oops, если указатель в этот момент обнулялся в
disconnect_scanner.

2. Там же: если сканер был отсоединен до закрытия дескриптора,
не вызывался MOD_DEC_USE_COUNT, в результате модуль невозможно
было выгрузить.

3. ioctl_scanner: опять race, аналогичный close_scanner (тут про
блокировку вообще забыли, да и непонятно, зачем вообще было
лезть в p_scn_table при наличии нужного указателя в
file->private_data).

4. probe_scanner: при занятости всех 16 номеров устройств
происходило обращение за пределы массива p_scn_table.

5. disconnect_scanner: при уничтожении структуры scn_usb_data не
проверялось наличие ссылок на нее в file->private_data; если
после этого программа пыталась выполнить read или write,
происходило обращение к освобожденной памяти.

По поводу #5 пришлось менять схему работы со структурой
scn_usb_data: в нее был добавлен счетчик ссылок, введены функции
usb_scanner_inc_use и usb_scanner_dec_use. Теперь структура
удаляется либо в disconnect_scanner, либо при закрытии
последнего открытого дескриптора. Однако номер устройства
освобождается всегда в disconnect_scanner - при немедленном
подключении устройства назад не происходит сдвига номеров, как в
usblp.

Помимо этого, исправлено несколько неприятных особенностей, не
являющихся багами в полном смысле слова:

- вызовы down в {open,read,write}_scanner заменены на
down_interruptible для возможности прерывания процесса сигналом;
из close_scanner они вообще убраны за ненадобностью;

- для open_scanner сделан отдельный семафор open_sem - устраняет
блокировку open при выполнении read/write в другом процессе (это
могло мешать при наличии нескольких сканеров - SANE открывает
все устройства подряд);

- статус сообщения "open_scanner(%d): Unable to access minor
data" понижен с err до dbg - эта ситуация возникает при каждом
открытии отсутствующего устройства; поскольку SANE по умолчанию
открывает все возможные устройства /dev/usb/scanner*, это
сообщение засоряет логи.

Есть еще два бага, которые пока оставлены - надо исследовать
вопросы совместимости. В read_scanner и write_scanner при
прерывании сигналом возвращается -ERESTARTSYS, даже если часть
данных была передана - в общем случае это неправильно. Хотя,
скорее всего, ни один пользовательский драйвер сигналами не
пользуется - иначе эта ошибка всплыла бы; к тому же все равно
сигналы обрабатываются с большой задержкой - на время выполнения
usb_bulk_msg.

При проверке со сканером Mustek BearPaw 2400TA (тестовыми
программами с http://www.mivlgu.murom.ru/~vsu/gt68xx/) при
отключении сканера во время работы перестали происходить oops (с
неисправленным драйвером были). Осталась проблема, которая, судя
по всему, не связана с драйвером scanner.c - она проявляется и с
usbdevfs и libusb: изредка (удалось повторить раза с 20-го)
получается:

Mar 24 19:41:23 canopus kernel: usb.c: USB disconnect on device 21
Mar 24 19:41:33 canopus kernel: usb_control/bulk_msg: timeout
Mar 24 19:41:33 canopus kernel: usbdevfs: USBDEVFS_BULK failed dev 21 ep 0x81 len 4096 ret -110
Mar 24 19:41:55 canopus kernel: hub.c: USB new device connect on bus1/1, assigned device number 22
Mar 24 19:41:58 canopus kernel: usb_control/bulk_msg: timeout
Mar 24 19:41:58 canopus kernel: usb.c: USB device not accepting new address=22 (error=-110)
Mar 24 19:41:58 canopus kernel: hub.c: USB new device connect on bus1/1, assigned device number 23
Mar 24 19:42:01 canopus kernel: usb_control/bulk_msg: timeout
Mar 24 19:42:01 canopus kernel: usb.c: USB device not accepting new address=23 (error=-110)

После этого USB не работает до выполнения service usb restart
(предварительно rmmod scanner). Впрочем, возможно, это ошибки
старого ядра 2.4.15-alt0.61-up.

-- 
Сергей Власов

[-- Attachment #1.2: scanner-0.4.7-fix.patch --]
[-- Type: application/octet-stream, Size: 5936 bytes --]

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

Re: [devel] linux/drivers/usb/scanner.c

[Konstantin Volckov]

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

On Sun, 24 Mar 2002 20:25:05 +0300
Sergey Vlasov <vsu@mivlgu.murom.ru> wrote:

> Hello!

Hi!

> Итак, в  версии 0.4.7 драйвера USB-сканеров найдена куча багов,
> которые должен бы исправить прилагаемый патч.

[skip]

Здорово! Насколько я разобрался - все изменения к месту. Вы отослали патч
автору драйвера? Если нет - отошлите - его должны принять. Я же включу его
в следующую сборку 2.4.18 - alt5.

> Есть еще два бага, которые пока оставлены - надо исследовать
> вопросы совместимости. В read_scanner и write_scanner при
> прерывании сигналом возвращается -ERESTARTSYS, даже если часть
> данных была передана - в общем случае это неправильно. Хотя,
> скорее всего, ни один пользовательский драйвер сигналами не
> пользуется - иначе эта ошибка всплыла бы; к тому же все равно
> сигналы обрабатываются с большой задержкой - на время выполнения
> usb_bulk_msg.

Не думаю, что это критично - насколько я знаю, sane игнорирует не
полностью просканированные файлы.

> При проверке со сканером Mustek BearPaw 2400TA (тестовыми
> программами с http://www.mivlgu.murom.ru/~vsu/gt68xx/) при
> отключении сканера во время работы перестали происходить oops (с
> неисправленным драйвером были). Осталась проблема, которая, судя
> по всему, не связана с драйвером scanner.c - она проявляется и с
> usbdevfs и libusb: изредка (удалось повторить раза с 20-го)
> получается:
> 
[skip]

> 
> После этого USB не работает до выполнения service usb restart
> (предварительно rmmod scanner). Впрочем, возможно, это ошибки
> старого ядра 2.4.15-alt0.61-up.

В новых ядрах usb было несколько раз зафиксено (в частности, у меня лучше
стал работать flash-reader) - попробуйте 2.4.18.

> -- 
> Сергей Власов


-- 
Успехов,
Konstantin

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

Re: [devel] linux/drivers/usb/scanner.c

[Sergey Vlasov]

On Mon, 25 Mar 2002 20:59:55 +0300
Konstantin Volckov <goldhead@altlinux.ru> wrote:

> On Sun, 24 Mar 2002 20:25:05 +0300
> Sergey Vlasov <vsu@mivlgu.murom.ru> wrote:
> > Итак, в  версии 0.4.7 драйвера USB-сканеров найдена куча багов,
> > которые должен бы исправить прилагаемый патч.
> 
> [skip]
> 
> Здорово! Насколько я разобрался - все изменения к месту. Вы отослали патч
> автору драйвера? Если нет - отошлите - его должны принять. Я же включу его
> в следующую сборку 2.4.18 - alt5.

Докладываю: получил ответ от мантейнера - он согласен принять
этот патч.

> > Есть еще два бага, которые пока оставлены - надо исследовать
> > вопросы совместимости. В read_scanner и write_scanner при
> > прерывании сигналом возвращается -ERESTARTSYS, даже если часть
> > данных была передана - в общем случае это неправильно. Хотя,
> > скорее всего, ни один пользовательский драйвер сигналами не
> > пользуется - иначе эта ошибка всплыла бы; к тому же все равно
> > сигналы обрабатываются с большой задержкой - на время выполнения
> > usb_bulk_msg.
> 
> Не думаю, что это критично - насколько я знаю, sane игнорирует не
> полностью просканированные файлы.

На этом месте может вылезти глюк, если frontend попытается
пользоваться pthreads - там внутри используются сигналы.

Кстати, в старом POSIX возврат EINTR после частичного чтения
допускался. В новой версии IEEE Std 1003.1-2001 это уже
запретили.

Так что придется править. В принципе там вообще надо переписать
этот кусок, чтобы использовались непосредственно URB вместо
usb_bulk_msg - иначе, похоже, для BearPaw 2400TA получаются
слишком большие задержки между блоками. Но это уже позже.

[skip]

Re: [devel] linux/drivers/usb/scanner.c

[Aleksey Novodvorsky]

Sergey Vlasov wrote:

> On Mon, 25 Mar 2002 20:59:55 +0300
> Konstantin Volckov <goldhead@altlinux.ru> wrote:
>
> > On Sun, 24 Mar 2002 20:25:05 +0300
> > Sergey Vlasov <vsu@mivlgu.murom.ru> wrote:
> > > Итак, в  версии 0.4.7 драйвера USB-сканеров найдена куча багов,
> > > которые должен бы исправить прилагаемый патч.
> >
> > [skip]
> >
> > Здорово! Насколько я разобрался - все изменения к месту. Вы отослали патч
> > автору драйвера? Если нет - отошлите - его должны принять. Я же включу его
> > в следующую сборку 2.4.18 - alt5.
>
> Докладываю: получил ответ от мантейнера - он согласен принять
> этот патч.
>

WOW!
Еще один kernel hacker в наших рядах!

Rgrds, AEN