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 kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 6D572CE79A9 for ; Tue, 19 Sep 2023 15:44:09 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id F04A56B0565; Tue, 19 Sep 2023 11:44:08 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id EAE5A6B0566; Tue, 19 Sep 2023 11:44:08 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id D4F6B6B0567; Tue, 19 Sep 2023 11:44:08 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0016.hostedemail.com [216.40.44.16]) by kanga.kvack.org (Postfix) with ESMTP id C14F06B0565 for ; Tue, 19 Sep 2023 11:44:08 -0400 (EDT) Received: from smtpin28.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay02.hostedemail.com (Postfix) with ESMTP id 95E9B12066C for ; Tue, 19 Sep 2023 15:44:08 +0000 (UTC) X-FDA: 81253768176.28.72BD2AB Received: from casper.infradead.org (casper.infradead.org [90.155.50.34]) by imf04.hostedemail.com (Postfix) with ESMTP id EB3FD40014 for ; Tue, 19 Sep 2023 15:44:05 +0000 (UTC) Authentication-Results: imf04.hostedemail.com; dkim=pass header.d=infradead.org header.s=casper.20170209 header.b=NUUqRKI9; spf=none (imf04.hostedemail.com: domain of willy@infradead.org has no SPF policy when checking 90.155.50.34) smtp.mailfrom=willy@infradead.org; dmarc=none ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1695138246; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=K4XOxDjoQc0tDKul2sAFg8dPE5RBNC5v+ER7EvKWlro=; b=nrvZECNi+9Ad7kPi5siCaK5Jy/3Yuj12vHFNrZ38Vhfbo/kV+0CjLCJJRX62YK91SJtYJ1 iNJ35QuOqiIFIZmObz8YuqDT3pnypkIAkkhvrb63TIziOrhJmOuwcxV5KMWVCEr0Dk9Mmc S8pAu9RqiS63VxkWTT4QDzSMApJyGNI= ARC-Authentication-Results: i=1; imf04.hostedemail.com; dkim=pass header.d=infradead.org header.s=casper.20170209 header.b=NUUqRKI9; spf=none (imf04.hostedemail.com: domain of willy@infradead.org has no SPF policy when checking 90.155.50.34) smtp.mailfrom=willy@infradead.org; dmarc=none ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1695138246; a=rsa-sha256; cv=none; b=Qa8lAnnUV1Nj86HTz/IyGEqUAo8olYMOfyzva9XEVRvQlYPNkz/hzKpk9FXIPKSE9GJoBA P6Odi7hrfE0s/AKSzrMFYzMda69/Zaem0hX90j6jUd64+QsmoHPrMstYVN6+vOaenhgLY7 hhezvDrDalD0of9vk4zj+pDFjHRYKpc= 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=K4XOxDjoQc0tDKul2sAFg8dPE5RBNC5v+ER7EvKWlro=; b=NUUqRKI9eBera0KC3gXkeqZtkM /GFKMZSxhT/N4WSHKiJ/LnKARML8J4fVRmJAK6vS+sngTB3qVVyL5CZ7+aYjPzEBzodj/t51kyRLt +lC5J68P8dYuCckGOdMcFiLmJTQjK6XnyMKf+KwyMI7oVjFXRLoLzSfMEgkQla6fltpCc33xk0tTD jkWzrEOofVLacRE1OsH9DD0Y3UrJuKKDfV9dv6ARDpn1HVgbpIj1e8+kk7CJygrv6/VjTWV4BPRkl RwV6MSNc1q6jXJklBhtoCEN0yr3A6DbNe1h+LuJ/9Kx0mqQ2fnY4HswW1WJ8xjBYMRuhM1tY0rMmS FoqlIc6w==; Received: from willy by casper.infradead.org with local (Exim 4.94.2 #2 (Red Hat Linux)) id 1qict2-000Qnd-Tt; Tue, 19 Sep 2023 15:43:40 +0000 Date: Tue, 19 Sep 2023 16:43:40 +0100 From: Matthew Wilcox To: Oleksandr Natalenko Cc: linux-kernel@vger.kernel.org, intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, Jani Nikula , Joonas Lahtinen , Rodrigo Vivi , Tvrtko Ursulin , David Airlie , Daniel Vetter , Andi Shyti , Andrzej Hajda , Matthew Auld , Matt Roper , Aravind Iddamsetty , Fei Yang , Thomas =?iso-8859-1?Q?Hellstr=F6m?= , Nathan Chancellor , Chris Wilson , Daniele Ceraolo Spurio , Andrew Morton , linux-mm@kvack.org Subject: Re: [REGRESSION] [BISECTED] Panic in gen8_ggtt_insert_entries() with v6.5 Message-ID: References: <4857570.31r3eYUQgx@natalenko.name> <6287208.lOV4Wx5bFT@natalenko.name> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <6287208.lOV4Wx5bFT@natalenko.name> X-Rspamd-Queue-Id: EB3FD40014 X-Rspam-User: X-Stat-Signature: 1kkhhsxic9si5xozsdjotj7cpgzr5jyf X-Rspamd-Server: rspam01 X-HE-Tag: 1695138245-981197 X-HE-Meta: U2FsdGVkX1+O9cLcgWkYr7FuuPHO5HT/gxrAxrRtMrEI4COQNLWJcnBa2bti/V/jc7PDC9zdd69JOptd3H2jgvH9IIeR6nYT1wBpNC7rQjgfxdWeikqtU7cRkUtcJtSAmcqyNbw3WZBg3mkRlU+Oxx8pptCIQoDcbrq5RmM/Bmc44d48fuiN7PZ4Q/F2dchQckfzfEzowKaZF8joybS9bZ7Zy6fs804nn0SzKgmxj+QjUQQhK7/1+JNm7BuVl3ya9YnAnv0UFxpVdGWigEyc5UmacJcMCCNIH67Cdu5JMDB5fJXkEhU9FcK+skK0zdslkWykadpyNhrI36DVLoY2dRMhpdNcrwLkwMKVQQhoyEIHRXErBWm35uwvx3cA4BS9MWTt2+3Pm5BZiGJ7yCZcps3ft1N55qHJQkDv87RPKGmF++WPxwUGvBQsnF6R4yNMYHsWkCcIwEKtZHnSRRhnhlP+f2TP7UCjEGVl5wrnYucCcEfmoSmXG80aEY/dDILYWPSJE4mUNk4PUyU85mpxUNuysDwti1OfzTyfn/1oPVGXyKh7wH0tIk97OBhO1DM0nCzR/uvPwIsHH/hfAfZem09ALuUiF9CWiR70ZALxkfF3UaiWi9jmfpC0S+ZvOJCziGJG4RgpUKVlHaAf8tniDu/jfpZsXtHeFBINqbST1/W6PNAmoVT8w5JuIl5XpfaOMFhW4Pqa7LLheC42biiht7xwh2rlKjMbc7g9+6q1C3ucPjhQkIC+vVXIEAc2pKliNs0LCD1sfMpjB3i/45eE27YNIt/MHnO8T1Ivgs2Mx7FhklGwNqRUJoVHtOEukzPu8hNt49tAEh23GZQ+z7tnFXsiWR1baEk+5r6kDFCltIUC4grgFvHUDIkkd6jgHODB2fw+cWLrbJKGTGj7JNF3avliPY+XX0ECW4B/TatxXb8XbALT5uY7idddu+U0hPr2lmLUifMbaVXhF9JGWQ9 F3UJXokg ffGHRNP9C2C2fwF+darsPBzcywcvwZckbrjOZEOosf1Qs9kO4OiZm0DMuZ2h5jceGXRAYDnpUbLkeuqGdc4FdiksUSqgPf9yX7/2H9w+vHbhcxes/V7PgG0SeTuOCoqvQXUKPxrN31IIVnNBykXqHXKhqcroOOk4xqZXJBhIAk6jkybT+l23qXNJ3cPXmZB19dn/9YFuZLnYuiPv0iUbq1XF8i53ghatXk+4E2BNGBYsJk6ILBjXOdR7mxA== X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On Tue, Sep 19, 2023 at 10:26:42AM +0200, Oleksandr Natalenko wrote: > Andrzej asked me to try to revert commits 0b62af28f249, e0b72c14d8dc and 1e0877d58b1e, and reverting those fixed the i915 crash for me. The e0b72c14d8dc and 1e0877d58b1e commits look like just prerequisites, so I assume 0b62af28f249 ("i915: convert shmem_sg_free_table() to use a folio_batch") is the culprit here. > > Could you please check this? > > Our conversation with Andrzej is available at drm-intel GitLab [1]. > > Thanks. > > [1] https://gitlab.freedesktop.org/drm/intel/-/issues/9256 Wow, that is some great debugging. Thanks for all the time & effort you and others have invested. Sorry for breaking your system. You're almost right about the "prerequisites", but it's in the other direction; 0b62af28f249 is a prerequisite for the later two cleanups, so reverting all three is necessary to test 0b62af28f249. It seems to me that you've isolated the problem to constructing overly long sg lists. I didn't realise that was going to be a problem, so that's my fault. Could I ask you to try this patch? I'll follow up with another patch later because I think I made another assumption that may not be valid. diff --git a/drivers/gpu/drm/i915/gem/i915_gem_shmem.c b/drivers/gpu/drm/i915/gem/i915_gem_shmem.c index 8f1633c3fb93..73a4a4eb29e0 100644 --- a/drivers/gpu/drm/i915/gem/i915_gem_shmem.c +++ b/drivers/gpu/drm/i915/gem/i915_gem_shmem.c @@ -100,6 +100,7 @@ int shmem_sg_alloc_table(struct drm_i915_private *i915, struct sg_table *st, st->nents = 0; for (i = 0; i < page_count; i++) { struct folio *folio; + unsigned long nr_pages; const unsigned int shrink[] = { I915_SHRINK_BOUND | I915_SHRINK_UNBOUND, 0, @@ -150,6 +151,8 @@ int shmem_sg_alloc_table(struct drm_i915_private *i915, struct sg_table *st, } } while (1); + nr_pages = min_t(unsigned long, + folio_nr_pages(folio), page_count - i); if (!i || sg->length >= max_segment || folio_pfn(folio) != next_pfn) { @@ -157,13 +160,13 @@ int shmem_sg_alloc_table(struct drm_i915_private *i915, struct sg_table *st, sg = sg_next(sg); st->nents++; - sg_set_folio(sg, folio, folio_size(folio), 0); + sg_set_folio(sg, folio, nr_pages * PAGE_SIZE, 0); } else { /* XXX: could overflow? */ - sg->length += folio_size(folio); + sg->length += nr_pages * PAGE_SIZE; } - next_pfn = folio_pfn(folio) + folio_nr_pages(folio); - i += folio_nr_pages(folio) - 1; + next_pfn = folio_pfn(folio) + nr_pages; + i += nr_pages - 1; /* Check that the i965g/gm workaround works. */ GEM_BUG_ON(gfp & __GFP_DMA32 && next_pfn >= 0x00100000UL);