Укажите отработанное время

Отработанное время:
Продуктивное время:
Bug 2773 - Не собирается etercifs для ASP 12   Make a simular bug
Summary: Не собирается etercifs для ASP 12
Status: CLOSED FIXED
Alias: None
Product: CIFS@Etersoft
Classification: Продукты (Products)
Component: компиляция модуля (show other bugs)
Version: не указана
Hardware: PC ASP Linux
: P4 critical
Target Milestone: ---
Assignee: Денис Баранов
QA Contact:
URL: http://rt.etersoft.ru/Ticket/Display....
Whiteboard:
Keywords:
Depends on:
Blocks: 777
  Show dependency tree / graph
 
In work:
Reported: 2008-10-30 15:28 MSK by Leonid Shadevsky
Modified: 2008-10-30 18:17 MSK (History)
4 users (show)

See Also:
Заявки RT:
Связано с:
Дата напоминания:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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 собралось и запустилось нормально.