[PATCH 1/2] linx-user: Remove MAX_SIGQUEUE_SIZE

Warner Losh posted 2 patches 4 years ago
Maintainers: Laurent Vivier <laurent@vivier.eu>
[PATCH 1/2] linx-user: Remove MAX_SIGQUEUE_SIZE
Posted by Warner Losh 4 years ago
It's been unused for 7 years since 907f5fddaa67 when linux-user stopped
queueing any signals.

Signed-off-by: Warner Losh <imp@bsdimp.com>
---
 linux-user/qemu.h | 2 --
 1 file changed, 2 deletions(-)

diff --git a/linux-user/qemu.h b/linux-user/qemu.h
index 5c713fa8ab2..7910ce59cc8 100644
--- a/linux-user/qemu.h
+++ b/linux-user/qemu.h
@@ -89,8 +89,6 @@ struct vm86_saved_state {
 #include "nwfpe/fpa11.h"
 #endif
 
-#define MAX_SIGQUEUE_SIZE 1024
-
 struct emulated_sigtable {
     int pending; /* true if signal is pending */
     target_siginfo_t info;
-- 
2.33.1


Re: [PATCH 1/2] linx-user: Remove MAX_SIGQUEUE_SIZE
Posted by Philippe Mathieu-Daudé via 4 years ago
On 16/1/22 21:44, Warner Losh wrote:
> It's been unused for 7 years since 907f5fddaa67 when linux-user stopped
> queueing any signals.
> 
> Signed-off-by: Warner Losh <imp@bsdimp.com>
> ---
>   linux-user/qemu.h | 2 --
>   1 file changed, 2 deletions(-)

Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

Re: [PATCH 1/2] linx-user: Remove MAX_SIGQUEUE_SIZE
Posted by Peter Maydell 4 years ago
On Sun, 16 Jan 2022 at 20:44, Warner Losh <imp@bsdimp.com> wrote:
>
> It's been unused for 7 years since 907f5fddaa67 when linux-user stopped
> queueing any signals.
>
> Signed-off-by: Warner Losh <imp@bsdimp.com>
> ---

Typo in subject (should be 'linux-user'); otherwise
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>

thanks
-- PMM

Re: [PATCH 1/2] linx-user: Remove MAX_SIGQUEUE_SIZE
Posted by Laurent Vivier 4 years ago
Le 16/01/2022 à 21:44, Warner Losh a écrit :
> It's been unused for 7 years since 907f5fddaa67 when linux-user stopped
> queueing any signals.
> 
> Signed-off-by: Warner Losh <imp@bsdimp.com>
> ---
>   linux-user/qemu.h | 2 --
>   1 file changed, 2 deletions(-)
> 
> diff --git a/linux-user/qemu.h b/linux-user/qemu.h
> index 5c713fa8ab2..7910ce59cc8 100644
> --- a/linux-user/qemu.h
> +++ b/linux-user/qemu.h
> @@ -89,8 +89,6 @@ struct vm86_saved_state {
>   #include "nwfpe/fpa11.h"
>   #endif
>   
> -#define MAX_SIGQUEUE_SIZE 1024
> -
>   struct emulated_sigtable {
>       int pending; /* true if signal is pending */
>       target_siginfo_t info;

Applied to my trivial-patches branch.
(typo fixed)

Thanks,
Laurent