Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 24 May 2015 21:25:15 +0300
From:      Konstantin Belousov <kostikbel@gmail.com>
To:        Dmitry Chagin <dchagin@FreeBSD.org>
Cc:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r283479 - in head/sys: amd64/amd64 amd64/ia32 arm/arm i386/i386 mips/mips powerpc/powerpc sparc64/sparc64
Message-ID:  <20150524182515.GF2499@kib.kiev.ua>
In-Reply-To: <201505241756.t4OHu3ot035613@svn.freebsd.org>
References:  <201505241756.t4OHu3ot035613@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Sun, May 24, 2015 at 05:56:03PM +0000, Dmitry Chagin wrote:
> Author: dchagin
> Date: Sun May 24 17:56:02 2015
> New Revision: 283479
> URL: https://svnweb.freebsd.org/changeset/base/283479
> 
> Log:
>   The kernel sends signals to the processes via ABI specific sv_sendsig method.
>   Native ABI do not need signal conversion, only emulators may want this. Usually
>   emulators implements its own sv_sendsig method. For now only ibcs2 emulator does
>   not have own sv_sendsig implementation and depends on native sendsig() method.
>   So, remove any extra attempts to convert signal numbers from native sendsig()
>   methods except from i386 where ibsc2 is living.
> 
> Modified:
>   head/sys/amd64/amd64/machdep.c
>   head/sys/amd64/ia32/ia32_signal.c
>   head/sys/arm/arm/machdep.c
>   head/sys/i386/i386/machdep.c
>   head/sys/mips/mips/freebsd32_machdep.c
>   head/sys/mips/mips/pm_machdep.c
>   head/sys/powerpc/powerpc/exec_machdep.c
>   head/sys/sparc64/sparc64/machdep.c
> 
> Modified: head/sys/amd64/amd64/machdep.c
> ==============================================================================
> --- head/sys/amd64/amd64/machdep.c	Sun May 24 17:53:48 2015	(r283478)
> +++ head/sys/amd64/amd64/machdep.c	Sun May 24 17:56:02 2015	(r283479)
> @@ -398,10 +398,6 @@ sendsig(sig_t catcher, ksiginfo_t *ksi, 
>  	/* Align to 16 bytes. */
>  	sfp = (struct sigframe *)((unsigned long)sp & ~0xFul);
>  
> -	/* Translate the signal if appropriate. */
> -	if (p->p_sysent->sv_sigtbl && sig <= p->p_sysent->sv_sigsize)
> -		sig = p->p_sysent->sv_sigtbl[_SIG_IDX(sig)];
> -
Does struct sysent still need sv_sigtbl and sv_sigsize members then ?



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20150524182515.GF2499>