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:


-------------------------------------------------------------------------
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