From mboxrd@z Thu Jan 1 00:00:00 1970 From: Geert Uytterhoeven Subject: Re: [PATCH 09/10] riscv: fix livelock in uaccess Date: Tue, 7 Feb 2023 17:11:40 +0100 Message-ID: References: Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:Cc:To:Subject:Message-ID:Date:From: In-Reply-To:References:MIME-Version:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=4Y0I/3Ei8612A1t5CDrqxn6F055QJ4kcsd2St3Gp7mg=; b=TaenCeQqY/o8LD ESOQCiOsITAEY5nK4ta5nqiiWIhFilQ0twudlS+keQlhw3bi7CJAJpH6IEW7eoyTmKv5XvcJ+dDPm ETSwEubv5+got9Wyc73evZ5nYXzVlt3ovl/xb9WGphYceigHuUAwiE3SBJ01oKuD3+B3CywDi4UWZ kuJ6hkoC1fDZyznW0uPmtcu0i3B/ArobfP2yNKhNuJ5D2qN3bcONiS3MUTqD/E37mq4LnSclTe6Nr InHcv2nqthXVfwmrkSKhn7v9tVcMtJJht29snC82QmD/HbYJ/AMzm5QLlel7V7I2qQvrKci3cdVe/ zPClLjx4XrRtuPiJyK/g==; In-Reply-To: List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-riscv" Errors-To: linux-riscv-bounces+glpr-linux-riscv=m.gmane-mx.org@lists.infradead.org To: Al Viro Cc: linux-arch@vger.kernel.org, linux-alpha@vger.kernel.org, linux-ia64@vger.kernel.org, linux-hexagon@vger.kernel.org, linux-m68k@lists.linux-m68k.org, Michal Simek , Dinh Nguyen , openrisc@lists.librecores.org, linux-parisc@vger.kernel.org, linux-riscv@lists.infradead.org, sparclinux@vger.kernel.org, Linus Torvalds On Tue, Jan 31, 2023 at 9:07 PM Al Viro wrote: > riscv equivalent of 26178ec11ef3 "x86: mm: consolidate VM_FAULT_RETRY handling" > If e.g. get_user() triggers a page fault and a fatal signal is caught, we might > end up with handle_mm_fault() returning VM_FAULT_RETRY and not doing anything > to page tables. In such case we must *not* return to the faulting insn - > that would repeat the entire thing without making any progress; what we need > instead is to treat that as failed (user) memory access. > > Signed-off-by: Al Viro Tested-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds