Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) client-ip=23.128.96.37;
Feedback-ID: ifd894703:Fastmail
User-Agent: Cyrus-JMAP/3.9.0-alpha0-1048-g9229b632c5-fm-20231019.001-g9229b632
MIME-Version: 1.0
Message-Id: <82076999-9346-473d-8841-1480cd70b527@app.fastmail.com>
Date:   Tue, 31 Oct 2023 21:41:26 +0000
From:   "Jiaxun Yang" <jiaxun.yang@flygoat.com>
To:     linux-arch@vger.kernel.org
Cc:     x86@kernel.org, linux-kernel@vger.kernel.org, arnd@arndb.de,
        bhe@redhat.com
Subject: Overhead of io{read,write}{8,16,32,64} on x86
Content-Type: text/plain
Precedence: bulk

Hi all,

I'm trying to improve Kernel's support of devices that have ioports
mapped into MMIO, that involves converting existing driver which is
using {in,out}{l,w,b} to use io{read,write}{8,16,32,64}, so they can
benefit from ioport_map and pci_iomap.

However, the problem is io{read,write}{8,16,32,64} will incur penalty
on x86 by introducing extra function calls (they are not inlined) and
having extra condition judgment on MMIO vs PIO.

x86 folks, do you think this kind of overhead is acceptable? I do think
most of PCI/ISA drivers will need to be converted.

linux-arch folks, do you think it will be better if we introduce a
variant of io{read,write}{8,16,32,64} that direct to PIO on x86 but
remains the same functionality on other architectures?

Thanks
- Jiaxun