Re: не собирается на одной машине .....

[Michail Baikov <lists@xxxxxxxxxx>]

Igor Sysoev wrote:
> On Wed, 22 Feb 2006, Igor Sysoev wrote:
> 
>> On Wed, 22 Feb 2006, Igor Sysoev wrote:
>>
>>>>> Есть ощущение, что здесь как-то пояивлся -fPIC, но его нигде не видно.
>>>
>>>> Да нет - он тут может быть даже и был - машина то hardened. Но раньше
>>>
>>> А что такое hardened ?
>>>
>>>> все собиралось - 26 версия стоит и работает нормально.....
>>>
>>> В 0.3.27 появилось:
>>>
>>>    *) Добавление: определение размера строки кэша распространённых
>>>       процессоров при старте.
>>>
>>> До этого не было. В ngx_cpuinfo() вставляется ассемблерный код cpuid,
>>> который использует регистр ebx, а gcc использует этот же регистр для
>>> своих
>>> целей, если используется -fPIC код.
>>
>> Прилагаемый патч должен помочь.
> 
> Новая версия патча.
> 
> 
> Игорь Сысоев
> http://sysoev.ru
> 
> 
> ------------------------------------------------------------------------
> 
> --- src/core/ngx_cpuinfo.c    Wed Feb  8 14:39:29 2006
> +++ src/core/ngx_cpuinfo.c    Wed Feb 22 12:59:56 2006
> @@ -14,6 +14,38 @@
>  static ngx_inline void ngx_cpuid(uint32_t i, uint32_t *buf);
>  
>  
> +#if ( __i386__ )
> +
> +static ngx_inline void
> +ngx_cpuid(uint32_t i, uint32_t *buf)
> +{
> +
> +    /*
> +     * we could not use %ebx as parameter if gcc building with -fPIC
> +     * and we could not push %ebx on stack
> +     */
> +
> +    __asm__ (
> +
> +    "    mov    %%ebx, %%esi;  "
> +
> +    "    cpuid;                "
> +    "    mov    %%eax, %0;     "
> +    "    mov    %%ebx, %1;     "
> +    "    mov    %%edx, %2;     "
> +    "    mov    %%ecx, %3;     "
> +
> +    "    mov    %%esi, %%ebx;  "
> +
> +    : "=m" (buf[0]), "=m" (buf[1]), "=m" (buf[2]), "=m" (buf[3])
> +    : "a" (i)
> +    : "ecx", "edx", "esi" );
> +}
> +
> +
> +#else /* __amd64__ */
> +
> +
>  static ngx_inline void
>  ngx_cpuid(uint32_t i, uint32_t *buf)
>  {
> @@ -32,6 +64,8 @@
>  }
>  
>  
> +#endif
> +
>  /* auto detect the L2 cache line size of modern and widespread CPUs */
>  
>  void
> @@ -53,6 +87,8 @@
>      if (vbuf[0] == 0) {
>          return;
>      }
> +
> +ngx_log_error(NGX_LOG_ALERT, ngx_cycle->log, 0, "CPU: \"%s\"", vendor);
>  
>      ngx_cpuid(1, cpu);
>  
Версия патча которая накладывается на 0.3.29 ;-) Ваша же версия, только
без виндовых ^M и короче работает :)

Спасиб.

-- 
Mike Baikov <mike at baikov dot com>
GameDot Labs Developer

diff -ur ./nginx-0.3.29/src/core/ngx_cpuinfo.c 
./nginx-0.3.29.2/src/core/ngx_cpuinfo.c
--- ./nginx-0.3.29/src/core/ngx_cpuinfo.c       2006-02-08 14:39:29.000000000 
+0300
+++ ./nginx-0.3.29.2/src/core/ngx_cpuinfo.c     2006-02-22 13:25:07.000000000 
+0300
@@ -13,6 +13,34 @@
 
 static ngx_inline void ngx_cpuid(uint32_t i, uint32_t *buf);
 
+#if ( __i386__ )
+
+
+static ngx_inline void ngx_cpuid(uint32_t i, uint32_t *buf)
+{
+
+    /*^
+     * we could not use %ebx as parameter if gcc building with -fPIC^
+     * and we could not push %ebx on stack^
+     */
+
+    __asm__ (
+
+    "    mov    %%ebx, %%esi;  "
+    "    cpuid;                "
+    "    mov    %%eax, %0;     "
+    "    mov    %%ebx, %1;     "
+    "    mov    %%edx, %2;     "
+    "    mov    %%ecx, %3;     "
+    "    mov    %%esi, %%ebx;  "
+    : "=m" (buf[0]), "=m" (buf[1]), "=m" (buf[2]), "=m" (buf[3])
+    : "a" (i)
+    : "ecx", "edx", "esi" );
+}
+
+
+#else /* __amd64__ */
+
 
 static ngx_inline void
 ngx_cpuid(uint32_t i, uint32_t *buf)
@@ -32,6 +60,9 @@
 }
 
 
+#endif
+
+
 /* auto detect the L2 cache line size of modern and widespread CPUs */
 
 void
@@ -53,6 +84,8 @@
     if (vbuf[0] == 0) {
         return;
     }
+    
+    ngx_log_error(NGX_LOG_ALERT, ngx_cycle->log, 0, "CPU: \"%s\"", vendor);    
 
     ngx_cpuid(1, cpu);