[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RISU PATCH 1/5] risu_i386: move reginfo_t and related
From: |
Alex Bennée |
Subject: |
Re: [Qemu-devel] [RISU PATCH 1/5] risu_i386: move reginfo_t and related defines to risu_reginfo_i386.h |
Date: |
Thu, 25 Apr 2019 14:39:05 +0100 |
User-agent: |
mu4e 1.3.1; emacs 26.1 |
Jan Bobek <address@hidden> writes:
> In order to build risu successfully for i386, we need files
> risu_reginfo_i386.{h,c}; this patch adds the former by extracting the
> relevant code from risu_i386.c.
>
> This patch is pure code motion; no functional changes were made.
>
> Signed-off-by: Jan Bobek <address@hidden>
> ---
> risu_i386.c | 23 +----------------------
> risu_reginfo_i386.h | 37 +++++++++++++++++++++++++++++++++++++
> 2 files changed, 38 insertions(+), 22 deletions(-)
> create mode 100644 risu_reginfo_i386.h
>
> diff --git a/risu_i386.c b/risu_i386.c
> index 5e7e01d..6798a78 100644
> --- a/risu_i386.c
> +++ b/risu_i386.c
> @@ -14,28 +14,7 @@
> #include <string.h>
>
> #include "risu.h"
> -
> -/* This is the data structure we pass over the socket.
> - * It is a simplified and reduced subset of what can
> - * be obtained with a ucontext_t*
> - */
> -struct reginfo {
> - uint32_t faulting_insn;
> - gregset_t gregs;
> -};
> -
> -#ifndef REG_GS
> -/* Assume that either we get all these defines or none */
> -#define REG_GS 0
> -#define REG_FS 1
> -#define REG_ES 2
> -#define REG_DS 3
> -#define REG_ESP 7
> -#define REG_TRAPNO 12
> -#define REG_EIP 14
> -#define REG_EFL 16
> -#define REG_UESP 17
> -#endif
> +#include "risu_reginfo_i386.h"
>
> struct reginfo master_ri, apprentice_ri;
>
> diff --git a/risu_reginfo_i386.h b/risu_reginfo_i386.h
> new file mode 100644
> index 0000000..5bba439
> --- /dev/null
> +++ b/risu_reginfo_i386.h
> @@ -0,0 +1,37 @@
> +/*******************************************************************************
> + * Copyright (c) 2010 Linaro Limited
> + * All rights reserved. This program and the accompanying materials
> + * are made available under the terms of the Eclipse Public License v1.0
> + * which accompanies this distribution, and is available at
> + * http://www.eclipse.org/legal/epl-v10.html
> + *
> + * Contributors:
> + * Peter Maydell (Linaro) - initial implementation
> +
> ******************************************************************************/
> +
> +#ifndef RISU_REGINFO_I386_H
> +#define RISU_REGINFO_I386_H
> +
> +/* This is the data structure we pass over the socket.
> + * It is a simplified and reduced subset of what can
> + * be obtained with a ucontext_t*
> + */
> +struct reginfo {
> + uint32_t faulting_insn;
> + gregset_t gregs;
> +};
> +
The actual reg definitions can be dropped here (as rth does in his fixup
series). We can get them from ucontext.
Otherwise:
Reviewed-by: Alex Bennée <address@hidden>
> +#ifndef REG_GS
> +/* Assume that either we get all these defines or none */
> +# define REG_GS 0
> +# define REG_FS 1
> +# define REG_ES 2
> +# define REG_DS 3
> +# define REG_ESP 7
> +# define REG_TRAPNO 12
> +# define REG_EIP 14
> +# define REG_EFL 16
> +# define REG_UESP 17
> +#endif /* !defined(REG_GS) */
> +
> +#endif /* RISU_REGINFO_I386_H */
--
Alex Bennée
- [Qemu-devel] [RISU PATCH 0/5] Fix RISU build for i386, Jan Bobek, 2019/04/08
- [Qemu-devel] [RISU PATCH 1/5] risu_i386: move reginfo_t and related defines to risu_reginfo_i386.h, Jan Bobek, 2019/04/08
- Re: [Qemu-devel] [RISU PATCH 1/5] risu_i386: move reginfo_t and related defines to risu_reginfo_i386.h,
Alex Bennée <=
- [Qemu-devel] [RISU PATCH 3/5] risu_reginfo_i386: implement arch-specific reginfo interface, Jan Bobek, 2019/04/08
- [Qemu-devel] [RISU PATCH 5/5] risu_i386: remove old unused code, Jan Bobek, 2019/04/08
- [Qemu-devel] [RISU PATCH 2/5] risu_i386: move reginfo-related code to risu_reginfo_i386.c, Jan Bobek, 2019/04/08
- [Qemu-devel] [RISU PATCH 4/5] risu_i386: implement missing CPU-specific functions, Jan Bobek, 2019/04/08
- Re: [Qemu-devel] [RISU PATCH 0/5] Fix RISU build for i386, Richard Henderson, 2019/04/08
- Re: [Qemu-devel] [RISU PATCH 0/5] Fix RISU build for i386, Alex Bennée, 2019/04/25