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 gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id D547FC64ED6 for ; Tue, 28 Feb 2023 10:42:06 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id AE4AE10E03C; Tue, 28 Feb 2023 10:42:06 +0000 (UTC) Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by gabe.freedesktop.org (Postfix) with ESMTPS id D8F1E10E687 for ; Tue, 28 Feb 2023 10:42:03 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1677580923; x=1709116923; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=H5hq6OvhYilcGFXpXrRc7rWTMuBQXMcrld4dE47kU08=; b=duEQfA+cUOahq+oE+UioW7NW1fZ7fcZPcMnJchRFrNVC2GRHUx5kXAEC Fp/JJNfWPyI7hKplVeMsW4COcEl2UR6Q8iVPbCYgAC1nRuUlBOf3B1GFr yMAs+Dp5Q0iYMXqG9Xh+AtT/DpLYzR8QCi5U5fhWRt0CVqrKejNFGERdo nyXB7OGM0B55myrTzAjzLtSsrJvpdhXNQlnWMWmClqlP6uVRg8J0nPu3J RXP1DZlgCw137EpS8nNDJ0F03nOm/tnRszSLBi0wgVOT/6XKuT6zdGPSW qhCXKBr4VNN+2v52vfxSwb9SNMg6hwOvxe/rss5XCRSllHThQkys/odBZ g==; X-IronPort-AV: E=McAfee;i="6500,9779,10634"; a="398884689" X-IronPort-AV: E=Sophos;i="5.98,221,1673942400"; d="scan'208";a="398884689" Received: from fmsmga008.fm.intel.com ([10.253.24.58]) by orsmga105.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 28 Feb 2023 02:42:03 -0800 X-IronPort-AV: E=McAfee;i="6500,9779,10634"; a="738108960" X-IronPort-AV: E=Sophos;i="5.98,221,1673942400"; d="scan'208";a="738108960" Received: from mistoan-mobl.ger.corp.intel.com (HELO mwauld-desk1.intel.com) ([10.252.9.93]) by fmsmga008-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 28 Feb 2023 02:42:02 -0800 From: Matthew Auld To: intel-xe@lists.freedesktop.org Date: Tue, 28 Feb 2023 10:41:28 +0000 Message-Id: <20230228104137.80965-6-matthew.auld@intel.com> X-Mailer: git-send-email 2.39.2 In-Reply-To: <20230228104137.80965-1-matthew.auld@intel.com> References: <20230228104137.80965-1-matthew.auld@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [Intel-xe] [PATCH v2 05/14] drm/xe/mmio: s/lmem/vram/ X-BeenThere: intel-xe@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel Xe graphics driver List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Lucas De Marchi Errors-To: intel-xe-bounces@lists.freedesktop.org Sender: "Intel-xe" This seems to be the preferred nomenclature in xe. Signed-off-by: Matthew Auld Cc: Lucas De Marchi --- drivers/gpu/drm/xe/xe_mmio.c | 40 +++++++++++++++++----------------- drivers/gpu/drm/xe/xe_module.c | 6 ++--- drivers/gpu/drm/xe/xe_module.h | 2 +- 3 files changed, 24 insertions(+), 24 deletions(-) diff --git a/drivers/gpu/drm/xe/xe_mmio.c b/drivers/gpu/drm/xe/xe_mmio.c index 65b0df9bb579..e5bd4609aaee 100644 --- a/drivers/gpu/drm/xe/xe_mmio.c +++ b/drivers/gpu/drm/xe/xe_mmio.c @@ -68,7 +68,7 @@ _resize_bar(struct xe_device *xe, int resno, resource_size_t size) return 1; } -static int xe_resize_lmem_bar(struct xe_device *xe, resource_size_t lmem_size) +static int xe_resize_vram_bar(struct xe_device *xe, resource_size_t vram_size) { struct pci_dev *pdev = to_pci_dev(xe->drm.dev); struct pci_bus *root = pdev->bus; @@ -78,31 +78,31 @@ static int xe_resize_lmem_bar(struct xe_device *xe, resource_size_t lmem_size) u32 pci_cmd; int i; int ret; - u64 force_lmem_bar_size = xe_force_lmem_bar_size; + u64 force_vram_bar_size = xe_force_vram_bar_size; current_size = roundup_pow_of_two(pci_resource_len(pdev, GEN12_LMEM_BAR)); - if (force_lmem_bar_size) { + if (force_vram_bar_size) { u32 bar_sizes; - rebar_size = force_lmem_bar_size * (resource_size_t)SZ_1M; + rebar_size = force_vram_bar_size * (resource_size_t)SZ_1M; bar_sizes = pci_rebar_get_possible_sizes(pdev, GEN12_LMEM_BAR); if (rebar_size == current_size) return 0; if (!(bar_sizes & BIT(pci_rebar_bytes_to_size(rebar_size))) || - rebar_size >= roundup_pow_of_two(lmem_size)) { - rebar_size = lmem_size; + rebar_size >= roundup_pow_of_two(vram_size)) { + rebar_size = vram_size; drm_info(&xe->drm, "Given bar size is not within supported size, setting it to default: %llu\n", - (u64)lmem_size >> 20); + (u64)vram_size >> 20); } } else { rebar_size = current_size; - if (rebar_size != roundup_pow_of_two(lmem_size)) - rebar_size = lmem_size; + if (rebar_size != roundup_pow_of_two(vram_size)) + rebar_size = vram_size; else return 0; } @@ -117,7 +117,7 @@ static int xe_resize_lmem_bar(struct xe_device *xe, resource_size_t lmem_size) } if (!root_res) { - drm_info(&xe->drm, "Can't resize LMEM BAR - platform support is missing\n"); + drm_info(&xe->drm, "Can't resize VRAM BAR - platform support is missing\n"); return -1; } @@ -168,7 +168,7 @@ int xe_mmio_total_vram_size(struct xe_device *xe, u64 *vram_size, u64 *usable_si if (usable_size) { reg = xe_gt_mcr_unicast_read_any(gt, XEHP_FLAT_CCS_BASE_ADDR); *usable_size = (u64)REG_FIELD_GET(GENMASK(31, 8), reg) * SZ_64K; - drm_info(&xe->drm, "lmem_size: 0x%llx usable_size: 0x%llx\n", + drm_info(&xe->drm, "vram_size: 0x%llx usable_size: 0x%llx\n", *vram_size, *usable_size); } @@ -180,7 +180,7 @@ int xe_mmio_probe_vram(struct xe_device *xe) struct pci_dev *pdev = to_pci_dev(xe->drm.dev); struct xe_gt *gt; u8 id; - u64 lmem_size; + u64 vram_size; u64 original_size; u64 current_size; u64 usable_size; @@ -207,29 +207,29 @@ int xe_mmio_probe_vram(struct xe_device *xe) gt = xe_device_get_gt(xe, 0); original_size = pci_resource_len(pdev, GEN12_LMEM_BAR); - err = xe_mmio_total_vram_size(xe, &lmem_size, &usable_size); + err = xe_mmio_total_vram_size(xe, &vram_size, &usable_size); if (err) return err; - resize_result = xe_resize_lmem_bar(xe, lmem_size); + resize_result = xe_resize_vram_bar(xe, vram_size); current_size = pci_resource_len(pdev, GEN12_LMEM_BAR); xe->mem.vram.io_start = pci_resource_start(pdev, GEN12_LMEM_BAR); - xe->mem.vram.size = min(current_size, lmem_size); + xe->mem.vram.size = min(current_size, vram_size); if (!xe->mem.vram.size) return -EIO; if (resize_result > 0) - drm_info(&xe->drm, "Successfully resize LMEM from %lluMiB to %lluMiB\n", + drm_info(&xe->drm, "Successfully resize VRAM from %lluMiB to %lluMiB\n", (u64)original_size >> 20, (u64)current_size >> 20); - else if (xe->mem.vram.size < lmem_size && !xe_force_lmem_bar_size) + else if (xe->mem.vram.size < vram_size && !xe_force_vram_bar_size) drm_info(&xe->drm, "Using a reduced BAR size of %lluMiB. Consider enabling 'Resizable BAR' support in your BIOS.\n", (u64)xe->mem.vram.size >> 20); - if (xe->mem.vram.size < lmem_size) + if (xe->mem.vram.size < vram_size) drm_warn(&xe->drm, "Restricting VRAM size to PCI resource size (0x%llx->0x%llx)\n", - lmem_size, (u64)xe->mem.vram.size); + vram_size, (u64)xe->mem.vram.size); xe->mem.vram.mapping = ioremap_wc(xe->mem.vram.io_start, xe->mem.vram.size); xe->mem.vram.size = min_t(u64, xe->mem.vram.size, usable_size); @@ -360,7 +360,7 @@ int xe_mmio_init(struct xe_device *xe) * and we should not continue with driver initialization. */ if (IS_DGFX(xe) && !(xe_mmio_read32(gt, GU_CNTL.reg) & LMEM_INIT)) { - drm_err(&xe->drm, "LMEM not initialized by firmware\n"); + drm_err(&xe->drm, "VRAM not initialized by firmware\n"); return -ENODEV; } diff --git a/drivers/gpu/drm/xe/xe_module.c b/drivers/gpu/drm/xe/xe_module.c index 5a51a9959eff..6860586ce7f8 100644 --- a/drivers/gpu/drm/xe/xe_module.c +++ b/drivers/gpu/drm/xe/xe_module.c @@ -22,9 +22,9 @@ bool enable_display = true; module_param_named(enable_display, enable_display, bool, 0444); MODULE_PARM_DESC(enable_display, "Enable display"); -u32 xe_force_lmem_bar_size; -module_param_named(lmem_bar_size, xe_force_lmem_bar_size, uint, 0600); -MODULE_PARM_DESC(lmem_bar_size, "Set the lmem bar size(in MiB)"); +u32 xe_force_vram_bar_size; +module_param_named(vram_bar_size, xe_force_vram_bar_size, uint, 0600); +MODULE_PARM_DESC(vram_bar_size, "Set the vram bar size(in MiB)"); int xe_guc_log_level = 5; module_param_named(guc_log_level, xe_guc_log_level, int, 0600); diff --git a/drivers/gpu/drm/xe/xe_module.h b/drivers/gpu/drm/xe/xe_module.h index 2c6ee46f5595..86916c176382 100644 --- a/drivers/gpu/drm/xe/xe_module.h +++ b/drivers/gpu/drm/xe/xe_module.h @@ -8,6 +8,6 @@ /* Module modprobe variables */ extern bool enable_guc; extern bool enable_display; -extern u32 xe_force_lmem_bar_size; +extern u32 xe_force_vram_bar_size; extern int xe_guc_log_level; extern char *xe_param_force_probe; -- 2.39.2