From: Jan Beulich <jbeulich@suse.com>
To: Juergen Gross <jgross@suse.com>,
Boris Ostrovsky <boris.ostrovsky@oracle.com>
Cc: Stefano Stabellini <sstabellini@kernel.org>,
lkml <linux-kernel@vger.kernel.org>,
"xen-devel@lists.xenproject.org" <xen-devel@lists.xenproject.org>
Subject: [PATCH 02/12] swiotlb-xen: fix late init retry
Date: Tue, 7 Sep 2021 14:04:47 +0200 [thread overview]
Message-ID: <778299d6-9cfd-1c13-026e-25ee5d14ecb3@suse.com> (raw)
In-Reply-To: <588b3e6d-2682-160c-468e-44ca4867a570@suse.com>
The commit referenced below removed the assignment of "bytes" from
xen_swiotlb_init() without - like done for xen_swiotlb_init_early() -
adding an assignment on the retry path, thus leading to excessively
sized allocations upon retries.
Fixes: 2d29960af0be ("swiotlb: dynamically allocate io_tlb_default_mem")
Signed-off-by: Jan Beulich <jbeulich@suse.com>
Cc: stable@vger.kernel.org
--- a/drivers/xen/swiotlb-xen.c
+++ b/drivers/xen/swiotlb-xen.c
@@ -211,8 +211,8 @@ error:
if (repeat--) {
/* Min is 2MB */
nslabs = max(1024UL, (nslabs >> 1));
- pr_info("Lowering to %luMB\n",
- (nslabs << IO_TLB_SHIFT) >> 20);
+ bytes = nslabs << IO_TLB_SHIFT;
+ pr_info("Lowering to %luMB\n", bytes >> 20);
goto retry;
}
pr_err("%s (rc:%d)\n", xen_swiotlb_error(m_ret), rc);
next prev parent reply other threads:[~2021-09-07 12:04 UTC|newest]
Thread overview: 66+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-09-07 12:03 [PATCH 00/12] swiotlb-xen: fixes and adjustments Jan Beulich
2021-09-07 12:04 ` [PATCH 01/12] swiotlb-xen: avoid double free Jan Beulich
2021-09-08 6:48 ` Christoph Hellwig
2021-09-10 22:55 ` Stefano Stabellini
2021-09-10 22:55 ` Stefano Stabellini
2021-09-07 12:04 ` Jan Beulich [this message]
2021-09-08 6:50 ` [PATCH 02/12] swiotlb-xen: fix late init retry Christoph Hellwig
2021-09-07 12:05 ` [PATCH 03/12] swiotlb-xen: maintain slab count properly Jan Beulich
2021-09-08 6:53 ` Christoph Hellwig
2021-09-10 23:10 ` Stefano Stabellini
2021-09-10 23:10 ` Stefano Stabellini
2021-09-07 12:05 ` [PATCH 04/12] swiotlb-xen: ensure to issue well-formed XENMEM_exchange requests Jan Beulich
2021-09-08 6:57 ` Christoph Hellwig
2021-09-08 9:16 ` Jan Beulich
2021-09-10 23:14 ` Stefano Stabellini
2021-09-10 23:14 ` Stefano Stabellini
2021-09-13 7:17 ` Jan Beulich
2021-09-13 20:31 ` Stefano Stabellini
2021-09-13 20:31 ` Stefano Stabellini
2021-09-14 9:11 ` Jan Beulich
2021-09-15 1:22 ` Stefano Stabellini
2021-09-15 1:22 ` Stefano Stabellini
2021-09-15 1:54 ` Stefano Stabellini
2021-09-15 1:54 ` Stefano Stabellini
2021-09-15 8:21 ` Jan Beulich
2021-09-07 12:05 ` [PATCH 05/12] swiotlb-xen: suppress certain init retries Jan Beulich
2021-09-08 6:58 ` Christoph Hellwig
2021-09-10 23:18 ` Stefano Stabellini
2021-09-10 23:18 ` Stefano Stabellini
2021-09-07 12:06 ` [PATCH 06/12] swiotlb-xen: limit " Jan Beulich
2021-09-08 7:00 ` Christoph Hellwig
2021-09-10 23:23 ` Stefano Stabellini
2021-09-10 23:23 ` Stefano Stabellini
2021-09-07 12:06 ` [PATCH 07/12] swiotlb-xen: drop leftover __ref Jan Beulich
2021-09-08 7:02 ` Christoph Hellwig
2021-09-10 23:24 ` Stefano Stabellini
2021-09-10 23:24 ` Stefano Stabellini
2021-09-07 12:07 ` [PATCH 08/12] swiotlb-xen: arrange to have buffer info logged Jan Beulich
2021-09-08 7:04 ` Christoph Hellwig
2021-09-10 23:26 ` Stefano Stabellini
2021-09-10 23:26 ` Stefano Stabellini
2021-09-07 12:07 ` [PATCH 09/12] swiotlb-xen: drop DEFAULT_NSLABS Jan Beulich
2021-09-08 7:06 ` Christoph Hellwig
2021-09-10 23:27 ` Stefano Stabellini
2021-09-10 23:27 ` Stefano Stabellini
2021-09-07 12:10 ` [PATCH 10/12] xen-pcifront: this module is PV-only Jan Beulich
2021-09-07 15:30 ` Bjorn Helgaas
2021-09-07 16:14 ` Jan Beulich
2021-09-07 16:38 ` Bjorn Helgaas
2021-09-10 23:34 ` Stefano Stabellini
2021-09-10 23:34 ` Stefano Stabellini
2021-09-07 12:11 ` [PATCH 11/12] xen/pci-swiotlb: reduce visibility of symbols Jan Beulich
2021-09-08 7:08 ` Christoph Hellwig
2021-09-07 12:13 ` [PATCH 12/12] swiotlb-xen: this is PV-only on x86 Jan Beulich
2021-09-08 7:14 ` Christoph Hellwig
2021-09-10 23:48 ` Stefano Stabellini
2021-09-10 23:48 ` Stefano Stabellini
2021-09-13 7:28 ` Jan Beulich
2021-09-13 20:54 ` Stefano Stabellini
2021-09-13 20:54 ` Stefano Stabellini
2021-09-13 7:51 ` Jan Beulich
2021-09-13 20:49 ` Stefano Stabellini
2021-09-13 20:49 ` Stefano Stabellini
2021-09-08 15:18 ` [PATCH 00/12] swiotlb-xen: fixes and adjustments Juergen Gross
2021-09-08 15:18 ` Juergen Gross via iommu
2021-09-14 8:30 ` Juergen Gross
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=778299d6-9cfd-1c13-026e-25ee5d14ecb3@suse.com \
--to=jbeulich@suse.com \
--cc=boris.ostrovsky@oracle.com \
--cc=jgross@suse.com \
--cc=linux-kernel@vger.kernel.org \
--cc=sstabellini@kernel.org \
--cc=xen-devel@lists.xenproject.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.