{"id":2215475,"url":"http://patchwork.ozlabs.org/api/patches/2215475/?format=json","web_url":"http://patchwork.ozlabs.org/project/linux-pci/patch/20260324165633.4583-3-ilpo.jarvinen@linux.intel.com/","project":{"id":28,"url":"http://patchwork.ozlabs.org/api/projects/28/?format=json","name":"Linux PCI development","link_name":"linux-pci","list_id":"linux-pci.vger.kernel.org","list_email":"linux-pci@vger.kernel.org","web_url":null,"scm_url":null,"webscm_url":null,"list_archive_url":"","list_archive_url_format":"","commit_url_format":""},"msgid":"<20260324165633.4583-3-ilpo.jarvinen@linux.intel.com>","list_archive_url":null,"date":"2026-03-24T16:56:25","name":"[02/10] resource: Pass full extent of empty space to resource_alignf CB","commit_ref":null,"pull_url":null,"state":"new","archived":false,"hash":"22973c3173f209f4306312093ce9076c47b802d0","submitter":{"id":83553,"url":"http://patchwork.ozlabs.org/api/people/83553/?format=json","name":"Ilpo Järvinen","email":"ilpo.jarvinen@linux.intel.com"},"delegate":null,"mbox":"http://patchwork.ozlabs.org/project/linux-pci/patch/20260324165633.4583-3-ilpo.jarvinen@linux.intel.com/mbox/","series":[{"id":497318,"url":"http://patchwork.ozlabs.org/api/series/497318/?format=json","web_url":"http://patchwork.ozlabs.org/project/linux-pci/list/?series=497318","date":"2026-03-24T16:56:23","name":"PCI: Improve head free space usage","version":1,"mbox":"http://patchwork.ozlabs.org/series/497318/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/patches/2215475/comments/","check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/2215475/checks/","tags":{},"related":[],"headers":{"Return-Path":"\n ","X-Original-To":["incoming@patchwork.ozlabs.org","linux-pci@vger.kernel.org"],"Delivered-To":"patchwork-incoming@legolas.ozlabs.org","Authentication-Results":["legolas.ozlabs.org;\n\tdkim=pass (2048-bit key;\n unprotected) header.d=intel.com header.i=@intel.com header.a=rsa-sha256\n header.s=Intel header.b=HQQGzlch;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=vger.kernel.org\n (client-ip=172.234.253.10; helo=sea.lore.kernel.org;\n envelope-from=linux-pci+bounces-50931-incoming=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)","smtp.subspace.kernel.org;\n\tdkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com\n header.b=\"HQQGzlch\"","smtp.subspace.kernel.org;\n arc=none smtp.client-ip=198.175.65.13","smtp.subspace.kernel.org;\n dmarc=pass (p=none dis=none) header.from=linux.intel.com","smtp.subspace.kernel.org;\n spf=pass smtp.mailfrom=linux.intel.com"],"Received":["from sea.lore.kernel.org (sea.lore.kernel.org [172.234.253.10])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519 server-signature ECDSA (secp384r1) server-digest SHA384)\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4fgGcC1Bz0z1xy1\n\tfor ; Wed, 25 Mar 2026 04:04:43 +1100 (AEDT)","from smtp.subspace.kernel.org (conduit.subspace.kernel.org\n [100.90.174.1])\n\tby sea.lore.kernel.org (Postfix) with ESMTP id D4ABC310BA28\n\tfor ; Tue, 24 Mar 2026 16:59:09 +0000 (UTC)","from localhost.localdomain (localhost.localdomain [127.0.0.1])\n\tby smtp.subspace.kernel.org (Postfix) with ESMTP id 560B939FCDD;\n\tTue, 24 Mar 2026 16:59:08 +0000 (UTC)","from mgamail.intel.com (mgamail.intel.com [198.175.65.13])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby smtp.subspace.kernel.org (Postfix) with ESMTPS id B44F7364044;\n\tTue, 24 Mar 2026 16:59:06 +0000 (UTC)","from orviesa007.jf.intel.com ([10.64.159.147])\n by orvoesa105.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 24 Mar 2026 09:59:06 -0700","from ijarvine-mobl1.ger.corp.intel.com (HELO localhost)\n ([10.245.244.217])\n by orviesa007-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 24 Mar 2026 09:58:46 -0700"],"ARC-Seal":"i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116;\n\tt=1774371548; cv=none;\n b=ZbwBLLmbm9ZQf/7/O/NzPIpSpxvJFgiLPIMBk5PbQmTdi62jX/+ooOL8ABxC3zPd8aREM+pFOjangQhPjINxvhWtBOES8H3mVAEm4N0UBQXbAXxiTF79O8jSGWxR4I/PfzvOrILkpXCbWwfxhIsQV2DXNAX3zqm1Psphf4J1/Wo=","ARC-Message-Signature":"i=1; a=rsa-sha256; d=subspace.kernel.org;\n\ts=arc-20240116; t=1774371548; c=relaxed/simple;\n\tbh=WCR3Q80GUH+QkmAMmzfR2ddGucv9DgzL69WXspRX/jg=;\n\th=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References:\n\t MIME-Version:Content-Type;\n b=Ty+Lsgk8bDfe38Ijm0DCW59i87eReDJ/tD+y6Kfja8J1gK/+I92si/EXEhQCkjjji6q4y/Y+3jdBai69+zDRnwIk90rh2xSbNhi+JDwsKdYbaQH8NnW54NM7lk3n+gqVF2ai0mNyaJSW1qAu+yHNdD+uL+vHlUkj/SYJ0SdURnQ=","ARC-Authentication-Results":"i=1; smtp.subspace.kernel.org;\n dmarc=pass (p=none dis=none) header.from=linux.intel.com;\n spf=pass smtp.mailfrom=linux.intel.com;\n dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com\n header.b=HQQGzlch; arc=none smtp.client-ip=198.175.65.13","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple;\n d=intel.com; i=@intel.com; q=dns/txt; s=Intel;\n t=1774371547; x=1805907547;\n h=from:to:cc:subject:date:message-id:in-reply-to:\n references:mime-version:content-transfer-encoding;\n bh=WCR3Q80GUH+QkmAMmzfR2ddGucv9DgzL69WXspRX/jg=;\n b=HQQGzlchPcReU4bpAPfEAG9HmWlOrv2ed3s2F3drHqt7R4ux8WQjalJH\n 7g+a2DvovYg7NIxq+6lfeQDVit61w1J2cG1xh1e7JU1w18xSbgB/6s6oT\n vBMq2z5hhCeB6zkqVeABaBoegJ8Mnb0nKbADZEGVyp6EMdS3EkvX6wJCP\n lSFjD9Ez/C+VC1zvabqLanVpZYud9rmG2o0i2NfcN7Cw2zXUGJ0L/SxIM\n hqsuY6qsUPXfheqcQGA1iv1YZAI8cFrajGeU76hMaxMy7rgV51/65CPFI\n dyUJh0/msTAXfK6dGoVdbV5YQWvhselEAnGSxczmzNnqOpo6HPoiT7Gxt\n g==;","X-CSE-ConnectionGUID":["PtaXiS0PSIynHKh3BPkKNw==","WtnV/loaRsCO4KMKean0ww=="],"X-CSE-MsgGUID":["uxU80DllS261HL+9njNKVQ==","ZxEF/K58SR25QVPpKDYMzA=="],"X-IronPort-AV":["E=McAfee;i=\"6800,10657,11739\"; a=\"86471700\"","E=Sophos;i=\"6.23,138,1770624000\";\n d=\"scan'208\";a=\"86471700\"","E=Sophos;i=\"6.23,138,1770624000\";\n d=\"scan'208\";a=\"224666153\""],"X-ExtLoop1":"1","From":"=?utf-8?q?Ilpo_J=C3=A4rvinen?= ","To":"linux-pci@vger.kernel.org,\n\tBjorn Helgaas ,\n\tGuenter Roeck ,\n\tlinux-alpha@vger.kernel.org,\n\tlinux-arm-kernel@lists.infradead.org,\n\tlinux-m68k@lists.linux-m68k.org,\n\tlinux-mips@vger.kernel.org,\n\tlinux-parisc@vger.kernel.org,\n\tlinuxppc-dev@lists.ozlabs.org,\n\tlinux-s390@vger.kernel.org,\n\tlinux-sh@vger.kernel.org,\n\tRussell King ,\n\tGeert Uytterhoeven ,\n\tThomas Bogendoerfer ,\n\t\"James E.J. Bottomley\" ,\n\tHelge Deller ,\n\tMichael Ellerman ,\n\tThomas Gleixner ,\n\tIngo Molnar ,\n\tBorislav Petkov ,\n\tDave Hansen ,\n\t\"H. Peter Anvin\" ,\n\tChris Zankel ,\n\tMax Filippov ,\n\tMadhavan Srinivasan ,\n\tYoshinori Sato ,\n\tRich Felker ,\n\tJohn Paul Adrian Glaubitz ,\n\tRichard Henderson ,\n\tMatt Turner ,\n\tMagnus Lindholm ,\n\tNicholas Piggin ,\n\t\"Christophe Leroy (CS GROUP)\" ,\n\tNiklas Schnelle ,\n\tGerald Schaefer ,\n\tHeiko Carstens ,\n\tVasily Gorbik ,\n\tAlexander Gordeev ,\n\tChristian Borntraeger ,\n\tSven Schnelle ,\n\tx86@kernel.org,\n\tDominik Brodowski ,\n\tlinux-kernel@vger.kernel.org","Cc":"=?utf-8?q?Ilpo_J=C3=A4rvinen?= ","Subject":"[PATCH 02/10] resource: Pass full extent of empty space to\n resource_alignf CB","Date":"Tue, 24 Mar 2026 18:56:25 +0200","Message-Id":"<20260324165633.4583-3-ilpo.jarvinen@linux.intel.com>","X-Mailer":"git-send-email 2.39.5","In-Reply-To":"<20260324165633.4583-1-ilpo.jarvinen@linux.intel.com>","References":"<20260324165633.4583-1-ilpo.jarvinen@linux.intel.com>","Precedence":"bulk","X-Mailing-List":"linux-pci@vger.kernel.org","List-Id":"","List-Subscribe":"","List-Unsubscribe":"","MIME-Version":"1.0","Content-Type":"text/plain; charset=UTF-8","Content-Transfer-Encoding":"8bit"},"content":"__find_resource_space() calculates the full extent of empty space but\nonly passes the aligned space to resource_alignf callback. In some\nsituations, the callback may choose take advantage of the free space\nbefore the requested alignment.\n\nPass the full extent of the calculated empty space to resource_alignf\ncallback as an additional parameter.\n\nSigned-off-by: Ilpo Järvinen \n---\n arch/alpha/kernel/pci.c | 1 +\n arch/arm/kernel/bios32.c | 4 +++-\n arch/m68k/kernel/pcibios.c | 4 +++-\n arch/mips/pci/pci-generic.c | 3 ++-\n arch/mips/pci/pci-legacy.c | 1 +\n arch/parisc/kernel/pci.c | 4 +++-\n arch/powerpc/kernel/pci-common.c | 4 +++-\n arch/s390/pci/pci.c | 1 +\n arch/sh/drivers/pci/pci.c | 4 +++-\n arch/x86/pci/i386.c | 3 ++-\n arch/xtensa/kernel/pci.c | 1 +\n drivers/pci/setup-res.c | 3 ++-\n drivers/pcmcia/rsrc_nonstatic.c | 3 ++-\n include/linux/ioport.h | 2 ++\n include/linux/pci.h | 7 ++++---\n kernel/resource.c | 3 ++-\n 16 files changed, 35 insertions(+), 13 deletions(-)","diff":"diff --git a/arch/alpha/kernel/pci.c b/arch/alpha/kernel/pci.c\nindex 51a8a4c4572a..11df411b1d18 100644\n--- a/arch/alpha/kernel/pci.c\n+++ b/arch/alpha/kernel/pci.c\n@@ -125,6 +125,7 @@ DECLARE_PCI_FIXUP_FINAL(PCI_ANY_ID, PCI_ANY_ID, pcibios_fixup_final);\n \n resource_size_t\n pcibios_align_resource(void *data, const struct resource *res,\n+\t\t const struct resource *empty_res,\n \t\t resource_size_t size, resource_size_t align)\n {\n \tstruct pci_dev *dev = data;\ndiff --git a/arch/arm/kernel/bios32.c b/arch/arm/kernel/bios32.c\nindex b5793e8fbdc1..5b9b4fcd0e54 100644\n--- a/arch/arm/kernel/bios32.c\n+++ b/arch/arm/kernel/bios32.c\n@@ -560,7 +560,9 @@ char * __init pcibios_setup(char *str)\n * which might be mirrored at 0x0100-0x03ff..\n */\n resource_size_t pcibios_align_resource(void *data, const struct resource *res,\n-\t\t\t\tresource_size_t size, resource_size_t align)\n+\t\t\t\t const struct resource *empty_res,\n+\t\t\t\t resource_size_t size,\n+\t\t\t\t resource_size_t align)\n {\n \tstruct pci_dev *dev = data;\n \tresource_size_t start = res->start;\ndiff --git a/arch/m68k/kernel/pcibios.c b/arch/m68k/kernel/pcibios.c\nindex e6ab3f9ff5d8..1415f6e4e5ce 100644\n--- a/arch/m68k/kernel/pcibios.c\n+++ b/arch/m68k/kernel/pcibios.c\n@@ -27,7 +27,9 @@\n * which might be mirrored at 0x0100-0x03ff..\n */\n resource_size_t pcibios_align_resource(void *data, const struct resource *res,\n-\tresource_size_t size, resource_size_t align)\n+\t\t\t\t const struct resource *empty_res,\n+\t\t\t\t resource_size_t size,\n+\t\t\t\t resource_size_t align)\n {\n \tresource_size_t start = res->start;\n \ndiff --git a/arch/mips/pci/pci-generic.c b/arch/mips/pci/pci-generic.c\nindex d2d68bac3d25..f4957c26efc7 100644\n--- a/arch/mips/pci/pci-generic.c\n+++ b/arch/mips/pci/pci-generic.c\n@@ -22,7 +22,8 @@\n * which might have be mirrored at 0x0100-0x03ff..\n */\n resource_size_t pcibios_align_resource(void *data, const struct resource *res,\n-\t\t\t\tresource_size_t size, resource_size_t align)\n+\t\t\t\t const struct resource *empty_res,\n+\t\t\t\t resource_size_t size, resource_size_t align)\n {\n \tstruct pci_dev *dev = data;\n \tresource_size_t start = res->start;\ndiff --git a/arch/mips/pci/pci-legacy.c b/arch/mips/pci/pci-legacy.c\nindex d04b7c1294b6..817e97402afe 100644\n--- a/arch/mips/pci/pci-legacy.c\n+++ b/arch/mips/pci/pci-legacy.c\n@@ -52,6 +52,7 @@ unsigned long pci_address_to_pio(phys_addr_t address)\n */\n resource_size_t\n pcibios_align_resource(void *data, const struct resource *res,\n+\t\t const struct resource *empty_res,\n \t\t resource_size_t size, resource_size_t align)\n {\n \tstruct pci_dev *dev = data;\ndiff --git a/arch/parisc/kernel/pci.c b/arch/parisc/kernel/pci.c\nindex cf285b17a5ae..f99b20795d5a 100644\n--- a/arch/parisc/kernel/pci.c\n+++ b/arch/parisc/kernel/pci.c\n@@ -196,7 +196,9 @@ void __ref pcibios_init_bridge(struct pci_dev *dev)\n * than res->start.\n */\n resource_size_t pcibios_align_resource(void *data, const struct resource *res,\n-\t\t\t\tresource_size_t size, resource_size_t alignment)\n+\t\t\t\t const struct resource *empty_res,\n+\t\t\t\t resource_size_t size,\n+\t\t\t\t resource_size_t alignment)\n {\n \tresource_size_t mask, align, start = res->start;\n \ndiff --git a/arch/powerpc/kernel/pci-common.c b/arch/powerpc/kernel/pci-common.c\nindex a7a2fb605971..e7bfa15da043 100644\n--- a/arch/powerpc/kernel/pci-common.c\n+++ b/arch/powerpc/kernel/pci-common.c\n@@ -1132,7 +1132,9 @@ static int skip_isa_ioresource_align(struct pci_dev *dev)\n * which might have be mirrored at 0x0100-0x03ff..\n */\n resource_size_t pcibios_align_resource(void *data, const struct resource *res,\n-\t\t\t\tresource_size_t size, resource_size_t align)\n+\t\t\t\t const struct resource *empty_res,\n+\t\t\t\t resource_size_t size,\n+\t\t\t\t resource_size_t align)\n {\n \tstruct pci_dev *dev = data;\n \tresource_size_t start = res->start;\ndiff --git a/arch/s390/pci/pci.c b/arch/s390/pci/pci.c\nindex 2a430722cbe4..39bd2adfc240 100644\n--- a/arch/s390/pci/pci.c\n+++ b/arch/s390/pci/pci.c\n@@ -266,6 +266,7 @@ static int zpci_cfg_store(struct zpci_dev *zdev, int offset, u32 val, u8 len)\n }\n \n resource_size_t pcibios_align_resource(void *data, const struct resource *res,\n+\t\t\t\t const struct resource *empty_res,\n \t\t\t\t resource_size_t size,\n \t\t\t\t resource_size_t align)\n {\ndiff --git a/arch/sh/drivers/pci/pci.c b/arch/sh/drivers/pci/pci.c\nindex a3903304f33f..7a0522316ee3 100644\n--- a/arch/sh/drivers/pci/pci.c\n+++ b/arch/sh/drivers/pci/pci.c\n@@ -168,7 +168,9 @@ subsys_initcall(pcibios_init);\n * modulo 0x400.\n */\n resource_size_t pcibios_align_resource(void *data, const struct resource *res,\n-\t\t\t\tresource_size_t size, resource_size_t align)\n+\t\t\t\t const struct resource *empty_res,\n+\t\t\t\t resource_size_t size,\n+\t\t\t\t resource_size_t align)\n {\n \tstruct pci_dev *dev = data;\n \tstruct pci_channel *hose = dev->sysdata;\ndiff --git a/arch/x86/pci/i386.c b/arch/x86/pci/i386.c\nindex c4ec39ad276b..6fbd4b34c3f7 100644\n--- a/arch/x86/pci/i386.c\n+++ b/arch/x86/pci/i386.c\n@@ -153,7 +153,8 @@ skip_isa_ioresource_align(struct pci_dev *dev) {\n */\n resource_size_t\n pcibios_align_resource(void *data, const struct resource *res,\n-\t\t\tresource_size_t size, resource_size_t align)\n+\t\t const struct resource *empty_res,\n+\t\t resource_size_t size, resource_size_t align)\n {\n \tstruct pci_dev *dev = data;\n \tresource_size_t start = res->start;\ndiff --git a/arch/xtensa/kernel/pci.c b/arch/xtensa/kernel/pci.c\nindex 62c900e400d6..64ccb7e0d92f 100644\n--- a/arch/xtensa/kernel/pci.c\n+++ b/arch/xtensa/kernel/pci.c\n@@ -39,6 +39,7 @@\n */\n resource_size_t\n pcibios_align_resource(void *data, const struct resource *res,\n+\t\t const struct resource *empty_res,\n \t\t resource_size_t size, resource_size_t align)\n {\n \tstruct pci_dev *dev = data;\ndiff --git a/drivers/pci/setup-res.c b/drivers/pci/setup-res.c\nindex bb2aef373d6f..c375e255c509 100644\n--- a/drivers/pci/setup-res.c\n+++ b/drivers/pci/setup-res.c\n@@ -251,10 +251,11 @@ static int pci_revert_fw_address(struct resource *res, struct pci_dev *dev,\n */\n resource_size_t __weak pcibios_align_resource(void *data,\n \t\t\t\t\t const struct resource *res,\n+\t\t\t\t\t const struct resource *empty_res,\n \t\t\t\t\t resource_size_t size,\n \t\t\t\t\t resource_size_t align)\n {\n- return res->start;\n+\treturn res->start;\n }\n \n static int __pci_assign_resource(struct pci_bus *bus, struct pci_dev *dev,\ndiff --git a/drivers/pcmcia/rsrc_nonstatic.c b/drivers/pcmcia/rsrc_nonstatic.c\nindex 0679dd434719..949e69921fe9 100644\n--- a/drivers/pcmcia/rsrc_nonstatic.c\n+++ b/drivers/pcmcia/rsrc_nonstatic.c\n@@ -602,7 +602,8 @@ static resource_size_t pcmcia_common_align(struct pcmcia_align_data *align_data,\n \n static resource_size_t\n pcmcia_align(void *align_data, const struct resource *res,\n-\tresource_size_t size, resource_size_t align)\n+\t const struct resource *empty_res,\n+\t resource_size_t size, resource_size_t align)\n {\n \tstruct pcmcia_align_data *data = align_data;\n \tstruct resource_map *m;\ndiff --git a/include/linux/ioport.h b/include/linux/ioport.h\nindex 19d5e04564d9..3c73c9c0d4f7 100644\n--- a/include/linux/ioport.h\n+++ b/include/linux/ioport.h\n@@ -202,6 +202,7 @@ enum {\n * typedef resource_alignf - Resource alignment callback\n * @data:\tPrivate data used by the callback\n * @res:\tResource candidate range (an empty resource space)\n+ * @empty_res:\tEmpty resource range without alignment applied\n * @size:\tThe minimum size of the empty space\n * @align:\tAlignment from the constraints\n *\n@@ -212,6 +213,7 @@ enum {\n */\n typedef resource_size_t (*resource_alignf)(void *data,\n \t\t\t\t\t const struct resource *res,\n+\t\t\t\t\t const struct resource *empty_res,\n \t\t\t\t\t resource_size_t size,\n \t\t\t\t\t resource_size_t align);\n \ndiff --git a/include/linux/pci.h b/include/linux/pci.h\nindex 1c270f1d5123..ac332ff9da9f 100644\n--- a/include/linux/pci.h\n+++ b/include/linux/pci.h\n@@ -1206,9 +1206,10 @@ int __must_check pcibios_enable_device(struct pci_dev *, int mask);\n char *pcibios_setup(char *str);\n \n /* Used only when drivers/pci/setup.c is used */\n-resource_size_t pcibios_align_resource(void *, const struct resource *,\n-\t\t\t\tresource_size_t,\n-\t\t\t\tresource_size_t);\n+resource_size_t pcibios_align_resource(void *data, const struct resource *res,\n+\t\t\t\t const struct resource *empty_res,\n+\t\t\t\t resource_size_t size,\n+\t\t\t\t resource_size_t align);\n \n /* Generic PCI functions used internally */\n \ndiff --git a/kernel/resource.c b/kernel/resource.c\nindex 1e2f1dfc0edd..1b8d3101bdc6 100644\n--- a/kernel/resource.c\n+++ b/kernel/resource.c\n@@ -759,7 +759,8 @@ static int __find_resource_space(struct resource *root, struct resource *old,\n \t\t\talloc.flags = avail.flags;\n \t\t\tif (alignf) {\n \t\t\t\talloc.start = alignf(constraint->alignf_data,\n-\t\t\t\t\t\t &avail, size, constraint->align);\n+\t\t\t\t\t\t &avail, &tmp,\n+\t\t\t\t\t\t size, constraint->align);\n \t\t\t} else {\n \t\t\t\talloc.start = avail.start;\n \t\t\t}\n","prefixes":["02/10"]}