[devel] Fwd: ndiswrapper is broken since kernel 2.6.37

[devel] Fwd: ndiswrapper is broken since kernel 2.6.37

[REAL]

Привет!

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

-------- Исходное сообщение --------
Тема: ndiswrapper is broken since kernel 2.6.37
Дата: Wed, 11 May 2011 19:15:40 +0400
От: (GalaxyMaster) <galaxy@openwall.com>
Кому: ldv@altlinux.org
Копия: Eugeny A. Rostovtsev <real@altlinux.org>

Dmitry,

I just wanted to notify you that the following commit to git.alt breaks
the ndiswrapper:
===
author	Eugeny A. Rostovtsev <real@altlinux.org>	
Mon, 28 Feb 2011 04:09:11 +0000 (04:09 +0000)
committer	Eugeny A. Rostovtsev <real@altlinux.org>	
Mon, 28 Feb 2011 04:09:11 +0000 (04:09 +0000)
commit	cc4ebec7b18be55447e4e8c8170ab7d52ed98cd0
http://git.altlinux.org/gears/k/kernel-modules-ndiswrapper-std-def.git?p=kernel-modules-ndiswrapper-std-def.git;a=commit;h=cc4ebec7b18be55447e4e8c8170ab7d52ed98cd0
===

It includes the following patch:
===
--- ndiswrapper-1.56/driver/loader.c.old	2011-02-28 04:07:27.000000000 
+0000
+++ ndiswrapper-1.56/driver/loader.c	2011-02-28 04:07:33.000000000 +0000
@@ -831,7 +831,7 @@

  static struct file_operations wrapper_fops = {
     .owner          = THIS_MODULE,
-	.ioctl		= wrapper_ioctl,
+	.unlocked_ioctl		= wrapper_ioctl,
     .release	= wrapper_ioctl_release,
  };
===

but replacing .ioctl with .unlocked_ioctl without updating 
wrapper_ioctl() is
plainly wrong since unlocked_ioctl() has different arguments (e.g. 
there is no
inode structure anymore).  I believe, removing the first argument from the
wrapper_ioctl() definition would be enough to fix the issue.

Could you please be so kind and forward this message to the maintainer (I
failed to locate one).  I'm also CC'ing Eugeny.



-- 
(GM)

Re: [devel] Fwd: ndiswrapper is broken since kernel 2.6.37

[Damir Shayhutdinov]

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

Нужно убрать первый параметр у функции wrapper_ioctl. Простого
переименования .ioctl в .unlocked_ioctl недостаточно.

Re: [devel] Fwd: ndiswrapper is broken since kernel 2.6.37

[REAL]

12.05.2011 11:28, Damir Shayhutdinov пишет:
>> Честно говоря, не понял, а чём речь. Мой фикс был взят из открытых
>> источников, если чего-то там не хватает, жду патчей.
>
> Нужно убрать первый параметр у функции wrapper_ioctl. Простого
> переименования .ioctl в .unlocked_ioctl недостаточно.

Положил к себе в git.alt (репозиторий kernel-modules), бранч 
template/ndiswrapper/sisyphus

Не знаю, правда, понадобится ли кому такое (2.6.37 у нас пока/уже 
никто не собирает).

-- 

REAL aka Евгений Ростовцев, программист ЦНИТ КемГУ

Re: [devel] Fwd: ndiswrapper is broken since kernel 2.6.37

[Michael Shigorin]

On Thu, May 12, 2011 at 12:16:34PM +0700, REAL wrote:
> Не знаю, правда, понадобится ли кому такое (2.6.37 у нас
> пока/уже никто не собирает).

"since" -- это ">=", а не "==".

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

Re: [devel] Fwd: ndiswrapper is broken since kernel 2.6.37

[REAL]

12.05.2011 12:58, Michael Shigorin пишет:
>> Не знаю, правда, понадобится ли кому такое (2.6.37 у нас
>> пока/уже никто не собирает).
>
> "since" -- это">=", а не "==".

Патч, о котором идёт речь, прикладывается _только_ к ядру 2.6.37.

-- 

REAL aka Евгений Ростовцев, программист ЦНИТ КемГУ

Re: [devel] Fwd: ndiswrapper is broken since kernel 2.6.37

[REAL]

12.05.2011 13:19, Anton Protopopov пишет:
>     Патч, о котором идёт речь, прикладывается _только_ к ядру 2.6.37.
>
>
> Не, именно начиная с 37-го:
>    $ git blame kernel-modules-ndiswrapper.spec -L70,70
>    e21213f8 (Eugeny A. Rostovtsev 2011-02-28 04:14:06 +0000 70) %if
> "%kversion" >= "2.6.37"

Чёрт, похоже, я смотрел недообновлённый gear из своего же git.alt 
репозитория :)

-- 

REAL aka Евгений Ростовцев, программист ЦНИТ КемГУ