From patchwork Tue Jan 9 12:14:55 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Nilesh Javali X-Patchwork-Id: 13514872 Received: from mx0b-0016f401.pphosted.com (mx0a-0016f401.pphosted.com [67.231.148.174]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id BC94B38FA9 for ; Tue, 9 Jan 2024 12:15:22 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=marvell.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=marvell.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=marvell.com header.i=@marvell.com header.b="gxhB+NHZ" Received: from pps.filterd (m0045849.ppops.net [127.0.0.1]) by mx0a-0016f401.pphosted.com (8.17.1.24/8.17.1.24) with ESMTP id 409BvuD2018301; Tue, 9 Jan 2024 04:15:19 -0800 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=marvell.com; h= from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding:content-type; s=pfpt0220; bh=tQD6St9U KiFhVj5S3XNTIcRA4fiTzb0DsqZez8OsXyU=; b=gxhB+NHZDR24yzdoAk3Gp5B8 CsF9SWUL98Xm2cfCe0+rzQI7CR01v3/TUF+rlyzd9cXGTqYIo0ppBZ8EKSGfNqEZ DW53+C0OIEQ9jG9g1Bo/seEqacJGAcgvuJkHUHK5JotONnQb2f03BYIB96luKWvL TjTELHOTQ1CrRO/pNndzXErZq7hgNMAI89+qIQgwjfBs7IVuDtnClRhNh1xohvuj dULWD/QuxSdzwdjDj/ruRyH87F/Zcw501dp5kwVzh+sU/9yQ8UCodbyaMziidv9n weZcprfnnT2TNQO2512K3OEj9yRO88GNCKyWisfsTbTj6R3qG+dnI6Aan8SULw== Received: from dc5-exch01.marvell.com ([199.233.59.181]) by mx0a-0016f401.pphosted.com (PPS) with ESMTPS id 3vh5qt02t9-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-SHA384 bits=256 verify=NOT); Tue, 09 Jan 2024 04:15:18 -0800 (PST) Received: from DC5-EXCH02.marvell.com (10.69.176.39) by DC5-EXCH01.marvell.com (10.69.176.38) with Microsoft SMTP Server (TLS) id 15.0.1497.48; Tue, 9 Jan 2024 04:15:17 -0800 Received: from maili.marvell.com (10.69.176.80) by DC5-EXCH02.marvell.com (10.69.176.39) with Microsoft SMTP Server id 15.0.1497.48 via Frontend Transport; Tue, 9 Jan 2024 04:15:17 -0800 Received: from stgdev-a5u16.punelab.marvell.com (stgdev-a5u16.punelab.marvell.com [10.31.33.187]) by maili.marvell.com (Postfix) with ESMTP id 4898E3F70B2; Tue, 9 Jan 2024 04:15:15 -0800 (PST) From: Nilesh Javali To: , , CC: , , Subject: [PATCH v3 0/3] UIO_MEM_DMA_COHERENT for cnic/bnx2/bnx2x Date: Tue, 9 Jan 2024 17:44:55 +0530 Message-ID: <20240109121458.26475-1-njavali@marvell.com> X-Mailer: git-send-email 2.23.1 Precedence: bulk X-Mailing-List: linux-scsi@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-Proofpoint-ORIG-GUID: YwUqbuSnj_EUTX34MWGOHxoNZBjw5IkO X-Proofpoint-GUID: YwUqbuSnj_EUTX34MWGOHxoNZBjw5IkO X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.272,Aquarius:18.0.997,Hydra:6.0.619,FMLib:17.11.176.26 definitions=2023-12-09_02,2023-12-07_01,2023-05-22_02 During bnx2i iSCSI testing we ran into page refcounting issues in the uio mmaps exported from cnic to the iscsiuio process, and bisected back to the removal of the __GFP_COMP flag from dma_alloc_coherent calls. In order to fix these drivers to be able to mmap dma coherent memory via a uio device, without resorting to hacks and working with an iommu enabled, introduce a new uio mmap type backed by dma_mmap_coherent. While converting the uio interface, I also noticed that not all of these allocations were PAGE_SIZE aligned. Particularly the bnx2/bnx2x status block mapping was much smaller than any architecture page size, and I was concerned that it could be unintentionally exposing kernel memory. v3: - fix warnings reported by kernel test robot and added base commit v2: - expose only the dma_addr within uio and cnic. - Cleanup newly added unions comprising virtual_addr and struct device Chris Leech (3): uio: introduce UIO_MEM_DMA_COHERENT type cnic,bnx2,bnx2x: use UIO_MEM_DMA_COHERENT cnic,bnx2,bnx2x: page align uio mmap allocations drivers/net/ethernet/broadcom/bnx2.c | 2 + .../net/ethernet/broadcom/bnx2x/bnx2x_main.c | 10 +++-- drivers/net/ethernet/broadcom/cnic.c | 20 +++++++--- drivers/net/ethernet/broadcom/cnic.h | 1 + drivers/net/ethernet/broadcom/cnic_if.h | 1 + drivers/uio/uio.c | 40 +++++++++++++++++++ include/linux/uio_driver.h | 2 + 7 files changed, 66 insertions(+), 10 deletions(-) base-commit: ed340d13aa1db6773667ed4bf907738df203fbda