[devel] Re: lua5-posix-2003.11.07-alt2: rebuild failed [1]

[devel] Re: lua5-posix-2003.11.07-alt2: rebuild failed [1]

[Alexey Tourbin]

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

On Tue, Jun 28, 2005 at 03:48:25PM +0400, QA Team Robot wrote:
> + gcc -shared -pipe -Wall -O2 -march=i586 -mcpu=i686 -fPIC -DPIC lposix.c -o posix.so -llua -llualib
> + lua5 -e 'assert(loadlib("./posix.so", "luaopen_posix"))(); assert(posix)' test.lua
> /usr/src/tmp/rpm-tmp.93540: line 96: 10053 Segmentation fault      lua5 -e 'assert(loadlib("./posix.so", "luaopen_posix"))(); assert(posix)' test.lua

lua сломана.

$ lua -e 'print(2*0.1+5)'
zsh: segmentation fault  lua -e 'print(2*0.1+5)'
$

Из-за безобидного патча, который сделал mouse (#6539).
lua-5.0.2-alt-lu_hash-intptr.patch

--- lua-5.0.2/src/llimits.h.orig	2003-02-20 22:33:23 +0300
+++ lua-5.0.2/src/llimits.h	2004-08-30 23:12:06 +0400
@@ -11,6 +11,7 @@
 #include <limits.h>
 #include <stddef.h>
 
+#include <stdint.h>
 
 #include "lua.h"
 
@@ -41,7 +42,7 @@
 */
 
 /* an unsigned integer to hold hash values */
-typedef unsigned int lu_hash;
+typedef intptr_t lu_hash;
 /* its signed equivalent */
 typedef int ls_hash;
 
Если пересобрать без этого патча, то всё работает.
Что-то прямо не знаю, что теперь делать.

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

[devel] Re: lua5-posix-2003.11.07-alt2: rebuild failed [1]

[Anton Farygin]

В письме Wed, 29 Jun 2005 07:00:21 +0400, Alexey Tourbin
написал:

> On Tue, Jun 28, 2005 at 03:48:25PM +0400, QA Team Robot wrote:
>> + gcc -shared -pipe -Wall -O2 -march=i586 -mcpu=i686 -fPIC -DPIC lposix.c -o posix.so -llua -llualib
>> + lua5 -e 'assert(loadlib("./posix.so", "luaopen_posix"))(); assert(posix)' test.lua
>> /usr/src/tmp/rpm-tmp.93540: line 96: 10053 Segmentation fault      lua5 -e 'assert(loadlib("./posix.so", "luaopen_posix"))(); assert(posix)' test.lua
> 
> lua сломана.
> 
> $ lua -e 'print(2*0.1+5)'
> zsh: segmentation fault  lua -e 'print(2*0.1+5)'
> $
> 
> Из-за безобидного патча, который сделал mouse (#6539).
> lua-5.0.2-alt-lu_hash-intptr.patch
> 
> --- lua-5.0.2/src/llimits.h.orig	2003-02-20 22:33:23 +0300
> +++ lua-5.0.2/src/llimits.h	2004-08-30 23:12:06 +0400
> @@ -11,6 +11,7 @@
>  #include <limits.h>
>  #include <stddef.h>
>  
> +#include <stdint.h>
>  
>  #include "lua.h"
>  
> @@ -41,7 +42,7 @@
>  */
>  
>  /* an unsigned integer to hold hash values */
> -typedef unsigned int lu_hash;
> +typedef intptr_t lu_hash;
>  /* its signed equivalent */
>  typedef int ls_hash;
>  
> Если пересобрать без этого патча, то всё работает.
> Что-то прямо не знаю, что теперь делать.

Отрывать патч, естественно.

Это похоже последствия портирования на x86_64 ;-)

Rgds,
Rider

[devel] Re: lua5-posix-2003.11.07-alt2: rebuild failed [1]

[Alexey Tourbin]

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

On Wed, Jun 29, 2005 at 09:40:22AM +0400, Anton Farygin wrote:
> >  /* an unsigned integer to hold hash values */
> > -typedef unsigned int lu_hash;
> > +typedef intptr_t lu_hash;
> >  /* its signed equivalent */
> >  typedef int ls_hash;
> >  
> > Если пересобрать без этого патча, то всё работает.
> > Что-то прямо не знаю, что теперь делать.
> 
> Отрывать патч, естественно.
> Это похоже последствия портирования на x86_64 ;-)

Угу.  Я попробовал посмотерть в gdb, что там происходит,
но пока ничего не понял....

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

Re: [devel] Re: lua5-posix-2003.11.07-alt2: rebuild failed [1]

[Anton D. Kachalov]

On Wed, Jun 29, 2005 at 09:40:22AM +0400, Anton Farygin wrote:
> Отрывать патч, естественно.
Ничего естественного. Фиксить надо, а не отрывать.
Кстати, фиксил я lua5-5.x.x, а не lua5-posix-...

> 
> Это похоже последствия портирования на x86_64 ;-)

Это кривая lua. Вот из-за чего взялся этот патч (для lua5-5.0.2):
ltable.c: In function `luaH_mainposition':
ltable.c:108: warning: cast from pointer to integer of different size
ltable.c:110: warning: cast from pointer to integer of different size

Это из-за функции IntPoint (llimits.h):
/*
** conversion of pointer to integer
** this is for hashing only; there is no problem if the integer
** cannot hold the whole pointer value
*/
#define IntPoint(p)  ((lu_hash)(p))

А ещё <stdint.h>:
#if __WORDSIZE == 64
typedef long int                intptr_t;
typedef unsigned long int       uintptr_t;
#else
typedef int                     intptr_t;
typedef unsigned int            uintptr_t;
#endif

Есть предложение заменить intptr_t на uintptr_t.

Rgds,
Anton

[devel] Re: lua5-posix-2003.11.07-alt2: rebuild failed [1]

[Alexey Tourbin]

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

On Wed, Jun 29, 2005 at 10:09:16AM +0400, Anton D. Kachalov wrote:
> On Wed, Jun 29, 2005 at 09:40:22AM +0400, Anton Farygin wrote:
> > Отрывать патч, естественно.
> Ничего естественного. Фиксить надо, а не отрывать.
> Кстати, фиксил я lua5-5.x.x, а не lua5-posix-...

И тот и другой теперь не работают.  lua5-posix демонстрирует segfault
во время теста при сборке.  Я первый раз этому большого значения не
придал, а когда появилось "rebuild failed [1]", стало ясно, что что-то
не так.  Я ещё иногда использую lua в качестве кальукулятора, и сегодня
при перемножении двух чисел получил segfault.  Тут-то я и всполошился. :)

> > Это похоже последствия портирования на x86_64 ;-)
> 
> Это кривая lua. Вот из-за чего взялся этот патч (для lua5-5.0.2):
> ltable.c: In function `luaH_mainposition':
> ltable.c:108: warning: cast from pointer to integer of different size
> ltable.c:110: warning: cast from pointer to integer of different size
> 
> Это из-за функции IntPoint (llimits.h):
> /*
> ** conversion of pointer to integer
> ** this is for hashing only; there is no problem if the integer
> ** cannot hold the whole pointer value
> */
> #define IntPoint(p)  ((lu_hash)(p))
> 
> А ещё <stdint.h>:
> #if __WORDSIZE == 64
> typedef long int                intptr_t;
> typedef unsigned long int       uintptr_t;
> #else
> typedef int                     intptr_t;
> typedef unsigned int            uintptr_t;
> #endif
> 
> Есть предложение заменить intptr_t на uintptr_t.

Сейчас попробую.

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

[devel] Re: lua5-posix-2003.11.07-alt2: rebuild failed [1]

[Alexey Tourbin]

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

On Wed, Jun 29, 2005 at 10:30:00AM +0400, Alexey Tourbin wrote:
> > А ещё <stdint.h>:
> > #if __WORDSIZE == 64
> > typedef long int                intptr_t;
> > typedef unsigned long int       uintptr_t;
> > #else
> > typedef int                     intptr_t;
> > typedef unsigned int            uintptr_t;
> > #endif
> > 
> > Есть предложение заменить intptr_t на uintptr_t.
> 
> Сейчас попробую.

Работает.  Исправленная версия ушла в /i/S.

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