[PATCH xserver 4/6] xfree86: remove xf86CaughtSignal etc.

Olivier Fourdan ofourdan at redhat.com
Wed Dec 13 15:20:00 UTC 2017


On Mon, Nov 20, 2017 at 9:43 PM, Adam Jackson <ajax at redhat.com> wrote:

> This no longer does anything useful.
>
> Signed-off-by: Adam Jackson <ajax at redhat.com>
> ---
>  hw/xfree86/common/xf86.h        |  2 --
>  hw/xfree86/common/xf86Events.c  | 11 -----------
>  hw/xfree86/common/xf86Globals.c |  1 -
>  hw/xfree86/common/xf86Helper.c  |  6 ------
>  hw/xfree86/common/xf86Init.c    | 17 +++--------------
>  hw/xfree86/common/xf86Priv.h    |  2 --
>  hw/xfree86/common/xf86Privstr.h |  1 -
>  hw/xfree86/doc/ddxDesign.xml    | 10 ----------
>  8 files changed, 3 insertions(+), 47 deletions(-)
>
> diff --git a/hw/xfree86/common/xf86.h b/hw/xfree86/common/xf86.h
> index 43b693143e..1c25468942 100644
> --- a/hw/xfree86/common/xf86.h
> +++ b/hw/xfree86/common/xf86.h
> @@ -296,8 +296,6 @@ xf86ServerIsResetting(void);
>  extern _X_EXPORT Bool
>  xf86ServerIsOnlyDetecting(void);
>  extern _X_EXPORT Bool
> -xf86CaughtSignal(void);
> -extern _X_EXPORT Bool
>  xf86GetVidModeAllowNonLocal(void);
>  extern _X_EXPORT Bool
>  xf86GetVidModeEnabled(void);
> diff --git a/hw/xfree86/common/xf86Events.c b/hw/xfree86/common/
> xf86Events.c
> index a27c7ff1b0..8a800bd8fd 100644
> --- a/hw/xfree86/common/xf86Events.c
> +++ b/hw/xfree86/common/xf86Events.c
> @@ -267,17 +267,6 @@ xf86RemoveEnabledDevice(InputInfoPtr pInfo)
>      InputThreadUnregisterDev(pInfo->fd);
>  }
>
> -/*
> - * xf86SigWrapper --
> - *    Catch unexpected signals and exit or continue cleanly.
> - */
> -int
> -xf86SigWrapper(int signo)
> -{
> -    xf86Info.caughtSignal = TRUE;
> -    return 1;                   /* abort */
> -}
> -
>  /*
>   * xf86PrintBacktrace --
>   *    Print a stack backtrace for debugging purposes.
> diff --git a/hw/xfree86/common/xf86Globals.c b/hw/xfree86/common/
> xf86Globals.c
> index ddf7a8696b..85efe3fc12 100644
> --- a/hw/xfree86/common/xf86Globals.c
> +++ b/hw/xfree86/common/xf86Globals.c
> @@ -107,7 +107,6 @@ xf86InfoRec xf86Info = {
>      .dontZap = FALSE,
>      .dontZoom = FALSE,
>      .notrapSignals = FALSE,
> -    .caughtSignal = FALSE,
>      .currentScreen = NULL,
>  #ifdef CSRG_BASED
>      .consType = -1,
> diff --git a/hw/xfree86/common/xf86Helper.c b/hw/xfree86/common/
> xf86Helper.c
> index 447ed3f8fe..393a7aa881 100644
> --- a/hw/xfree86/common/xf86Helper.c
> +++ b/hw/xfree86/common/xf86Helper.c
> @@ -1428,12 +1428,6 @@ xf86ServerIsOnlyDetecting(void)
>      return xf86DoConfigure;
>  }
>
> -Bool
> -xf86CaughtSignal(void)
> -{
> -    return xf86Info.caughtSignal;
> -}
> -
>  Bool
>  xf86GetVidModeAllowNonLocal(void)
>  {
> diff --git a/hw/xfree86/common/xf86Init.c b/hw/xfree86/common/xf86Init.c
> index 994b63b430..57b38d07e7 100644
> --- a/hw/xfree86/common/xf86Init.c
> +++ b/hw/xfree86/common/xf86Init.c
> @@ -298,16 +298,9 @@ xf86PrivsElevated(void)
>  }
>
>  static void
> -InstallSignalHandlers(void)
> +TrapSignals(void)
>  {
> -    /*
> -     * Install signal handler for unexpected signals
> -     */
> -    xf86Info.caughtSignal = FALSE;
> -    if (!xf86Info.notrapSignals) {
> -        OsRegisterSigWrapper(xf86SigWrapper);
> -    }
> -    else {
> +    if (xf86Info.notrapSignals) {
>          OsSignal(SIGSEGV, SIG_DFL);
>          OsSignal(SIGABRT, SIG_DFL);
>          OsSignal(SIGILL, SIG_DFL);
> @@ -423,7 +416,7 @@ InitOutput(ScreenInfo * pScreenInfo, int argc, char
> **argv)
>              }
>          }
>
> -        InstallSignalHandlers();
> +        TrapSignals();
>
>          /* Initialise the loader */
>          LoaderInit();
> @@ -960,10 +953,6 @@ ddxGiveUp(enum ExitCode error)
>      dbus_core_fini();
>
>      xf86CloseLog(error);
> -
> -    /* If an unexpected signal was caught, dump a core for debugging */
> -    if (xf86Info.caughtSignal)
> -        OsAbort();
>  }
>
>  /*
> diff --git a/hw/xfree86/common/xf86Priv.h b/hw/xfree86/common/xf86Priv.h
> index 22bf5ff240..4fe2b5f336 100644
> --- a/hw/xfree86/common/xf86Priv.h
> +++ b/hw/xfree86/common/xf86Priv.h
> @@ -135,8 +135,6 @@ DoShowOptions(void)
>
>  extern _X_EXPORT void
>  xf86Wakeup(void *blockData, int err);
> -extern _X_HIDDEN int
> -xf86SigWrapper(int signo);
>  extern _X_EXPORT void
>  xf86HandlePMEvents(int fd, void *data);
>  extern _X_EXPORT int (*xf86PMGetEventFromOs) (int fd, pmEvent * events,
> diff --git a/hw/xfree86/common/xf86Privstr.h b/hw/xfree86/common/
> xf86Privstr.h
> index e4b479f4f9..c5048a3997 100644
> --- a/hw/xfree86/common/xf86Privstr.h
> +++ b/hw/xfree86/common/xf86Privstr.h
> @@ -64,7 +64,6 @@ typedef struct {
>      Bool dontZap;
>      Bool dontZoom;
>      Bool notrapSignals;         /* don't exit cleanly - die at fault */
> -    Bool caughtSignal;
>
>      /* graphics part */
>      ScreenPtr currentScreen;
> diff --git a/hw/xfree86/doc/ddxDesign.xml b/hw/xfree86/doc/ddxDesign.xml
> index 7579850267..13994f7a91 100644
> --- a/hw/xfree86/doc/ddxDesign.xml
> +++ b/hw/xfree86/doc/ddxDesign.xml
> @@ -2053,16 +2053,6 @@ functions is as follows:
>
>         </para></blockquote>
>
> -      <blockquote><para>
> -         <programlisting>
> -    Bool xf86CaughtSignal();
> -         </programlisting>
> -         <blockquote><para>
> -      Returns <constant>TRUE</constant> if the server has caught a signal,
> -      and <constant>FALSE</constant> otherwise.
> -           </para></blockquote>
> -
> -       </para></blockquote>
>      </sect2>
>
>      <sect2>
> --
> 2.14.3
>
> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: https://lists.x.org/mailman/listinfo/xorg-devel


LGTM.

Reviewed-by: Olivier Fourdan <ofourdan at redhat.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.x.org/archives/xorg-devel/attachments/20171213/64a6c410/attachment-0001.html>


More information about the xorg-devel mailing list