From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751632AbeCTDQ4 (ORCPT ); Mon, 19 Mar 2018 23:16:56 -0400 Received: from akagi.fsij.org ([217.70.189.144]:38114 "EHLO akagi.fsij.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751351AbeCTDQv (ORCPT ); Mon, 19 Mar 2018 23:16:51 -0400 From: NIIBE Yutaka To: linux-kernel@vger.kernel.org Cc: gniibe@fsij.org Subject: [PATCH 0/4] Unblocked by fake_signal_wake_up then real signal Date: Tue, 20 Mar 2018 11:38:53 +0900 Message-Id: <20180320023857.1885-1-gniibe@fsij.org> X-Mailer: git-send-email 2.11.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello, Background: I tried to fix my own problem in suspend-to-ram with USB devices (some device doesn't work well after suspend/resume). I know it's common problems. Well, while I identified some problems of USB devices and drivers, I did try my best. I fixed a USB device implementation of mine. Then, I also located and fixed bugs in a USB user space driver which I maintain. Now, I think I found a kernel problem. After suspend/resume, in my USB user space driver, a task sends a signal to another task, then, pselect is unblocked by the signal. In this situation, a signal handler is called correctly, but it seems that the pselect is restarted. Expected behavior is unblocked pselect returning -EINTR. Looking the code, my theory is: It is a fake signal by suspend which unblocks pselect to be frozen, and after resume, real signal is sent. When both of signals are handled in a single call of exit_to_usermode_loop, by two calls of do_signal, firstly for fake signal and secondly for real signal, the system call is restarted wrongly, where it should return -EINTR. Here is a patch series of mine for x86. I'd suggest architecture maintainers to do similar if an architecture supports suspend/resume. NIIBE Yutaka (4): signal/x86: Factor out nosig handling. signal/x86: do_signal: syscall restart should be done only once. signal/x86: Move nosig handling at the end of exit_to_usermode_loop. signal/x86: Move restore_saved_sigmask(). arch/x86/entry/common.c | 6 +++++- arch/x86/include/asm/signal.h | 3 ++- arch/x86/kernel/signal.c | 22 ++++++++++++++-------- 3 files changed, 21 insertions(+), 10 deletions(-) -- 2.11.0