Bug 2773

Summary:	Не собирается etercifs для ASP 12
Product:	CIFS@Etersoft	Reporter:	Leonid Shadevsky <leonid>
Component:	компиляция модуля	Assignee:	Денис Баранов <baraka>
Status:	CLOSED FIXED	QA Contact:
Severity:	critical
Priority:	P4	CC:	kondratyuk, lav, sin, yurifil
Version:	не указана
Target Milestone:	---
Hardware:	PC
OS:	ASP Linux
URL:	http://rt.etersoft.ru/Ticket/Display.html?id=8791
Whiteboard:
Заявки RT:		Связано с:
Дата напоминания:
Bug Depends on:
Bug Blocks:	777

Description Leonid Shadevsky 2008-10-30 15:28:58 MSK

Ядро 2.6.23

Из заявки (вывод service etercifs build):

[root@inet ~]# service etercifs build

Build for 2.6.23.17-88.0.120asp Linux kernel (headers in 
/lib/modules/2.6.23.17-88.0.120asp/build)
/usr/bin/gcc
make: Entering directory `/usr/src/kernels/2.6.23.17-88.0.120asp-i686'
make: Leaving directory `/usr/src/kernels/2.6.23.17-88.0.120asp-i686'
make: Entering directory `/usr/src/kernels/2.6.23.17-88.0.120asp-i686'
CC [M] /tmp/Etercifs.UDNB6663/kernel-source-etercifs-2.6.23-1.50/cifsfs.o
CC [M] /tmp/Etercifs.UDNB6663/kernel-source-etercifs-2.6.23-1.50/cifssmb.o
/tmp/Etercifs.UDNB6663/kernel-source-etercifs-2.6.23-1.50/cifssmb.c: В 
функции 'SMBLegacyOpen'
/tmp/Etercifs.UDNB6663/kernel-source-etercifs-2.6.23-1.50/cifssmb.c:1171: 
предупреждение: unused variable 'count'
/tmp/Etercifs.UDNB6663/kernel-source-etercifs-2.6.23-1.50/cifssmb.c:1170: 
предупреждение: unused variable 'name_len'
/tmp/Etercifs.UDNB6663/kernel-source-etercifs-2.6.23-1.50/cifssmb.c:1169: 
предупреждение: unused variable 'bytes_returned'
/tmp/Etercifs.UDNB6663/kernel-source-etercifs-2.6.23-1.50/cifssmb.c:1168: 
предупреждение: unused variable 'pSMBr'
/tmp/Etercifs.UDNB6663/kernel-source-etercifs-2.6.23-1.50/cifssmb.c:1167: 
предупреждение: unused variable 'pSMB'
/tmp/Etercifs.UDNB6663/kernel-source-etercifs-2.6.23-1.50/cifssmb.c: На 
верхнем уровне:
/tmp/Etercifs.UDNB6663/kernel-source-etercifs-2.6.23-1.50/cifssmb.c:1130: 
предупреждение: 'convert_disposition' defined but not used
CC [M] 
/tmp/Etercifs.UDNB6663/kernel-source-etercifs-2.6.23-1.50/cifs_debug.o
CC [M] /tmp/Etercifs.UDNB6663/kernel-source-etercifs-2.6.23-1.50/connect.o
CC [M] /tmp/Etercifs.UDNB6663/kernel-source-etercifs-2.6.23-1.50/dir.o
CC [M] /tmp/Etercifs.UDNB6663/kernel-source-etercifs-2.6.23-1.50/file.o
CC [M] /tmp/Etercifs.UDNB6663/kernel-source-etercifs-2.6.23-1.50/inode.o
/tmp/Etercifs.UDNB6663/kernel-source-etercifs-2.6.23-1.50/inode.c: В 
функции 'cifs_setattr'
/tmp/Etercifs.UDNB6663/kernel-source-etercifs-2.6.23-1.50/inode.c:1510: 
предупреждение: passing argument 7 of 'SMBLegacyOpen' makes pointer from 
integer without a cast
/tmp/Etercifs.UDNB6663/kernel-source-etercifs-2.6.23-1.50/inode.c:1510: 
предупреждение: passing argument 8 of 'SMBLegacyOpen' from incompatible 
pointer type
/tmp/Etercifs.UDNB6663/kernel-source-etercifs-2.6.23-1.50/inode.c:1510: 
предупреждение: passing argument 9 of 'SMBLegacyOpen' from incompatible 
pointer type
/tmp/Etercifs.UDNB6663/kernel-source-etercifs-2.6.23-1.50/inode.c:1510: 
предупреждение: passing argument 11 of 'SMBLegacyOpen' makes integer 
from pointer without a cast
/tmp/Etercifs.UDNB6663/kernel-source-etercifs-2.6.23-1.50/inode.c:1510: 
ошибка: слишком много аргументов в вызове функции 'SMBLegacyOpen'
/tmp/Etercifs.UDNB6663/kernel-source-etercifs-2.6.23-1.50/inode.c:1641: 
предупреждение: passing argument 7 of 'CIFSSMBOpen' makes integer from 
pointer without a cast
/tmp/Etercifs.UDNB6663/kernel-source-etercifs-2.6.23-1.50/inode.c:1641: 
предупреждение: passing argument 8 of 'CIFSSMBOpen' from incompatible 
pointer type
/tmp/Etercifs.UDNB6663/kernel-source-etercifs-2.6.23-1.50/inode.c:1641: 
предупреждение: passing argument 10 of 'CIFSSMBOpen' from incompatible 
pointer type
/tmp/Etercifs.UDNB6663/kernel-source-etercifs-2.6.23-1.50/inode.c:1641: 
предупреждение: passing argument 11 of 'CIFSSMBOpen' makes pointer from 
integer without a cast
/tmp/Etercifs.UDNB6663/kernel-source-etercifs-2.6.23-1.50/inode.c:1641: 
ошибка: too few arguments to function 'CIFSSMBOpen'
make[1]: *** 
[/tmp/Etercifs.UDNB6663/kernel-source-etercifs-2.6.23-1.50/inode.o] Ошибка 1
make: *** 
[_module_/tmp/Etercifs.UDNB6663/kernel-source-etercifs-2.6.23-1.50] Ошибка 2
make: Leaving directory `/usr/src/kernels/2.6.23.17-88.0.120asp-i686'
can't locate built module etercifs.ko, continue

Comment 1 Leonid Shadevsky 2008-10-30 15:30:28 MSK

У нас воспризвелось - что на 3.5, что 3.6 - ошибка аналогичная описанной в заявке.

Comment 2 Konstantin Baev 2008-10-30 15:51:42 MSK

Если срочно, то вот:

diff --git a/sources/2.6.23/inode.c b/sources/2.6.23/inode.c
index 21f8cc1..a395d6d 100644
--- a/sources/2.6.23/inode.c
+++ b/sources/2.6.23/inode.c
@@ -1634,7 +1634,7 @@ int cifs_setattr(struct dentry *direntry, struct iattr *attrs)
                        /* BB we could scan to see if we already have it open
                           and pass in pid of opener to function */
                        rc = CIFSSMBOpen(xid, pTcon, full_path, FILE_OPEN,
-                                        SYNCHRONIZE | FILE_WRITE_ATTRIBUTES,
+                                        SYNCHRONIZE | FILE_WRITE_ATTRIBUTES, FILE_SHARE_ALL,
                                         CREATE_NOT_DIR, &netfid, &oplock,
                                         NULL, cifs_sb->local_nls,
                                         cifs_sb->mnt_cifs_flags &

И сейчас запакую и отправлю новую версию с исправлением.

Comment 3 Konstantin Baev 2008-10-30 15:53:06 MSK

(In reply to comment #2)

> +                                        SYNCHRONIZE | FILE_WRITE_ATTRIBUTES,
> FILE_SHARE_ALL,

Это - одна строка всмысле.

Comment 4 Konstantin Baev 2008-10-30 15:56:31 MSK

Виноват. Вот исправленный вариант патча:

diff --git a/sources/2.6.23/inode.c b/sources/2.6.23/inode.c
index 21f8cc1..7dc7184 100644
--- a/sources/2.6.23/inode.c
+++ b/sources/2.6.23/inode.c
@@ -1503,7 +1503,7 @@ int cifs_setattr(struct dentry *direntry, struct iattr *attrs)

                                rc = SMBLegacyOpen(xid, pTcon, full_path,
                                        FILE_OPEN,
-                                       SYNCHRONIZE | FILE_WRITE_ATTRIBUTES, FILE_SHARE_ALL,
+                                       SYNCHRONIZE | FILE_WRITE_ATTRIBUTES,
                                        CREATE_NOT_DIR, &netfid, &oplock,
                                        NULL, cifs_sb->local_nls,
                                        cifs_sb->mnt_cifs_flags &
@@ -1634,7 +1634,7 @@ int cifs_setattr(struct dentry *direntry, struct iattr *attrs)
                        /* BB we could scan to see if we already have it open
                           and pass in pid of opener to function */
                        rc = CIFSSMBOpen(xid, pTcon, full_path, FILE_OPEN,
-                                        SYNCHRONIZE | FILE_WRITE_ATTRIBUTES,
+                                        SYNCHRONIZE | FILE_WRITE_ATTRIBUTES, FILE_SHARE_ALL,
                                         CREATE_NOT_DIR, &netfid, &oplock,
                                         NULL, cifs_sb->local_nls,
                                         cifs_sb->mnt_cifs_flags &

Comment 5 Konstantin Baev 2008-10-30 16:21:03 MSK

Fixed in

etercifs-3.6.1-alt1
linux-cifs-3.4.1-alt1

Проверьте, пожалуйста.

Comment 6 Юрий Филь 2008-10-30 17:49:09 MSK

Пересобрал под все системы.

Comment 7 Vitaly Lipatov 2008-10-30 17:54:09 MSK

Леонид, проверь пожалуйста на новой сборке.

Comment 8 Денис Баранов 2008-10-30 18:17:24 MSK

(In reply to comment #7)
> Леонид, проверь пожалуйста на новой сборке.
> 

Проверил на новой сборке etercifs-3.6.1 собралось и запустилось нормально.