On Wed, Oct 22, 2008 at 11:59 PM, Subrata Modak
<[EMAIL PROTECTED]> wrote:
>
> On Tue, 2008-10-21 at 21:29 +0900, Masatake YAMATO wrote:
>> > See: <http://code.google.com/p/fbterm/issues/detail?id=1>. Looks like 
>> > another changed struct field name.
>> > -Garrett
>>
>> Thanks. Attached page is based on your advise.
>>
>> Signed-off-by: Masatake YAMATO <[EMAIL PROTECTED]>
>>
>> However, the page tells *older* version of 2.6.23 uses signo instead
>> of ssi_signo as a field name. Subrata wrote:
>>
>>     Hi Yamato,
>>
>>     Can you please look into some compilation error(s) reported on the
>>     following machine at today´s snapshot.
>>
>>     1) uname -a
>>     Linux 2.6.27-autokern1 #1 SMP PREEMPT Mon Oct 20 07:30:55 UTC 2008
>>     x86_64 GNU/Linux
>>
>> He used 2.6.27 for testing. Very strange.
>>
>> Subrata, could you try the patch on your 2.6.27 first.
>
> Yes, it solves the issue nicely. Thanks. Merged.
>
> Regards--
> Subrata
>
>> Then if you still have a trouble with the patch, could you send me the
>> output of following command on the platform:
>>
>>        cc -Wall  -I../../include -g -Wall -I../../../../include          \
>>           -DHAS_LINUX_SIGNALFD_H -DHAS_SIGNALFD   -Wall     signalfd01.c \
>>           -L../../../../lib -lltp  -E
>>
>> Regards,
>> Masatake YAMATO
>>
>> diff --git a/testcases/kernel/syscalls/signalfd/signalfd01.c 
>> b/testcases/kernel/syscalls/signalfd/signalfd01.c
>> index 15d69df..f4ac38f 100644
>> --- a/testcases/kernel/syscalls/signalfd/signalfd01.c
>> +++ b/testcases/kernel/syscalls/signalfd/signalfd01.c
>> @@ -51,6 +51,13 @@ int TST_TOTAL = 1;
>>  extern int Tst_count;
>>
>>  #ifdef HAS_SIGNALFD
>> +#include <linux/version.h>
>> +#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,23)
>> +#define SSI_SIGNO signo
>> +#else
>> +#define SSI_SIGNO ssi_signo
>> +#endif
>> +
>>
>>  #ifdef HAS_SYS_SIGNALFD_H
>>
>> @@ -170,15 +177,15 @@ do_test1(int ntst, int sig)
>>               goto out;
>>       }
>>
>> -     if (fdsi.ssi_signo == sig) {
>> +     if (fdsi.SSI_SIGNO == sig) {
>>               tst_resm(TPASS, "got expected signal");
>>               sfd_for_next = sfd;
>>               goto out;
>>       }
>>       else {
>>               tst_resm(TFAIL, "got unexpected signal: signal=%d : %s",
>> -                      fdsi.ssi_signo,
>> -                      strsignal(fdsi.ssi_signo));
>> +                      fdsi.SSI_SIGNO,
>> +                      strsignal(fdsi.SSI_SIGNO));
>>               sfd_for_next = -1;
>>               close(sfd);
>>               goto out;
>> @@ -264,14 +271,14 @@ do_test2(int ntst, int fd, int sig)
>>               goto out;
>>       }
>>
>> -     if (fdsi.ssi_signo == sig) {
>> +     if (fdsi.SSI_SIGNO == sig) {
>>               tst_resm(TPASS, "got expected signal");
>>               goto out;
>>       }
>>       else {
>>               tst_resm(TFAIL, "got unexpected signal: signal=%d : %s",
>> -                      fdsi.ssi_signo,
>> -                      strsignal(fdsi.ssi_signo));
>> +                      fdsi.SSI_SIGNO,
>> +                      strsignal(fdsi.SSI_SIGNO));
>>               goto out;
>>       }
>>  out:

kernel headers can be older than the kernel version. A very important,
but subtle point...

[EMAIL PROTECTED] ~ $ esearch linux-headers
[ Results for search key : linux-headers ]
[ Applications found : 1 ]

*  sys-kernel/linux-headers
      Latest version available: 2.6.23-r3
      Latest version installed: 2.6.23-r3
      Size of downloaded files: [no/bad digest]
      Homepage:    http://www.kernel.org/ http://www.gentoo.org/
      Description: Linux system headers
      License:     GPL-2


[EMAIL PROTECTED] ~ $ uname -a
Linux headless-horseman 2.6.25.11 #1 SMP Mon Aug 11 02:17:05 PDT 2008
i686 Intel(R) Xeon(R) CPU 5140 @ 2.33GHz GenuineIntel GNU/Linux

Cheers,
-Garrett

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to