From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 97E69CD80A8 for ; Tue, 10 Oct 2023 11:24:54 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230223AbjJJLYy (ORCPT ); Tue, 10 Oct 2023 07:24:54 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:47666 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231228AbjJJLYx (ORCPT ); Tue, 10 Oct 2023 07:24:53 -0400 Received: from casper.infradead.org (casper.infradead.org [IPv6:2001:8b0:10b:1236::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id ADD4E94; Tue, 10 Oct 2023 04:24:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=casper.20170209; h=In-Reply-To:Content-Type:MIME-Version: References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=RHIty1r8jlpg4mbxrTX99MguDdmyNDYNfk7Z509OtEw=; b=AK/yhtwqX3jRAdiHHdyk4fqwYm 4LtzZ1cDNS8zXq+n2DDkrXyrY1swTguaEC1XcnfP3Xo7lqcvYvpsYay9lVCPGct8rJfX2bIt9QxS3 9eyjtchnrilqJ2j1VmM/XYgVJ04vaNaRaOLUpz6EWadnqkeEz1alsuYxGnioBz+i65//wBOp1Jukf /JOyO36Tgak878AK8KiYQOQmP1FyyYaP7sM8z9ixJaRuS/a9rqnuoaM6uwpPGrI1U3Caz4llcugJc UlyVh8gop7AUFH3aCD64gvwZcMN5j/cENwAPLfUnoAK7BZOApLF9XJgkTraYp5l0bEfoItnprootj sFDJ0krA==; Received: from j130084.upc-j.chello.nl ([24.132.130.84] helo=noisy.programming.kicks-ass.net) by casper.infradead.org with esmtpsa (Exim 4.94.2 #2 (Red Hat Linux)) id 1qqAqf-004JVi-Mh; Tue, 10 Oct 2023 11:24:25 +0000 Received: by noisy.programming.kicks-ass.net (Postfix, from userid 1000) id 5F185300392; Tue, 10 Oct 2023 13:24:25 +0200 (CEST) Date: Tue, 10 Oct 2023 13:24:25 +0200 From: Peter Zijlstra To: "Kirill A. Shutemov" Cc: Borislav Petkov , Thomas Gleixner , Ingo Molnar , Dave Hansen , x86@kernel.org, "H. Peter Anvin" , Andrey Ryabinin , Alexander Potapenko , Andrey Konovalov , Dmitry Vyukov , Vincenzo Frascino , kasan-dev@googlegroups.com, linux-kernel@vger.kernel.org, Fei Yang , stable@vger.kernel.org Subject: Re: [PATCH] x86/alternatives: Disable KASAN on text_poke_early() in apply_alternatives() Message-ID: <20231010112425.GJ377@noisy.programming.kicks-ass.net> References: <20231010053716.2481-1-kirill.shutemov@linux.intel.com> <20231010081938.GBZSUJGlSvEkFIDnES@fat_crate.local> <20231010101056.GF377@noisy.programming.kicks-ass.net> <20231010102537.qkrfcna2fwfkzgir@box.shutemov.name> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20231010102537.qkrfcna2fwfkzgir@box.shutemov.name> Precedence: bulk List-ID: X-Mailing-List: stable@vger.kernel.org On Tue, Oct 10, 2023 at 01:25:37PM +0300, Kirill A. Shutemov wrote: > > That said, I don't particularly like the patch, I think it should, at > > the veyr least, cover all of apply_alternatives, not just > > text_poke_early(). > > I can do this, if it is the only stopper. > > Do you want it disabled on caller side or inside apply_alternatives()? Inside probably, covering the whole for()-loop thingy. Ideally with a comment explaining how KASAN doesn't like partial LA57 patching.