Re:sigcontext on Linux-ppc in user space, another hack.

Hi,

Please ignore the last patch!
Anyway, in the code example the last argument is not a sigcontext
structure but an ucontext_t structure, which is defined in <ucontext.h>.

Yes.

Here is the revised sample program and it works:

[root@localhost kbhend]# ./junk
SIGUSR1 = 10
uc = 7ffff794
uc->uc_mcontext.signal = 10
[root@localhost kbhend]# cat junk.c
#include <stdio.h>
#include <signal.h>
#include <ucontext.h>

/* Function Prototypes */
void install_sigusr1_handler(void);
void sigusr_handler(int , siginfo_t *, struct ucontext * uc);

int main(void)
{
  install_sigusr1_handler();
  printf("SIGUSR1 = %d\n", SIGUSR1);
  raise(SIGUSR1);
  exit(0);
}

void install_sigusr1_handler(void)
{
  struct sigaction newAct;

  if (sigemptyset(&newAct.sa_mask) != 0) {
    fprintf(stderr, "Warning, sigemptyset failed.\n");
  }

  newAct.sa_flags = 0;
  newAct.sa_flags |= SA_SIGINFO | SA_RESTART;

  newAct.sa_sigaction = (void
			 (*)(int,siginfo_t*,void*))sigusr_handler;

  if (sigaction(SIGUSR1, &newAct, NULL) != 0) {
    fprintf(stderr, "Couldn't install SIGUSR1 handler.\n");
    fprintf(stderr, "Exiting.\n");
    exit(1);
  }
}

void sigusr_handler(int signo, siginfo_t *siginfp, struct ucontext * uc)
{
  printf("uc = %08x\n", uc);
  printf("uc->uc_mcontext.signal = %d\n", uc->uc_mcontext.signal);
}

[[root@localhost kbhend]# cat junk.c
#include <stdio.h>
#include <signal.h>
#include <ucontext.h>

/* Function Prototypes */
void install_sigusr1_handler(void);
void sigusr_handler(int , siginfo_t *, struct ucontext * uc);

int main(void)
{
  install_sigusr1_handler();
  printf("SIGUSR1 = %d\n", SIGUSR1);
  raise(SIGUSR1);
  exit(0);
}

void install_sigusr1_handler(void)
{
  struct sigaction newAct;

  if (sigemptyset(&newAct.sa_mask) != 0) {
    fprintf(stderr, "Warning, sigemptyset failed.\n");
  }

  newAct.sa_flags = 0;
  newAct.sa_flags |= SA_SIGINFO | SA_RESTART;

  newAct.sa_sigaction = (void
			 (*)(int,siginfo_t*,void*))sigusr_handler;

  if (sigaction(SIGUSR1, &newAct, NULL) != 0) {
    fprintf(stderr, "Couldn't install SIGUSR1 handler.\n");
    fprintf(stderr, "Exiting.\n");
    exit(1);
  }
}

void sigusr_handler(int signo, siginfo_t *siginfp, struct ucontext * uc)
{
  printf("uc = %08x\n", uc);
  printf("uc->uc_mcontext.signal = %d\n", uc->uc_mcontext.signal);
}



[root@localhost kbhend]# ./junk
SIGUSR1 = 10
uc = 7ffff794
uc->uc_mcontext.signal = 10

So no patch needed.

Thanks for finding that.

Kevin

** Sent via the linuxppc-dev mail list. See http://lists.linuxppc.org/