From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id E8AB22EC09B for ; Wed, 15 Apr 2026 12:42:25 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.140.110.172 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1776256947; cv=none; b=Hf5TavG1auBJNjx9Ca8TEa6Wkgn1OX01ozxF7DSUJRqjzqxBMfdZmxy/Y60orZAJ4ZwGLcTlxTevd0+pvy8XUKULrnUmZLFNQIds4w+LJl0NeAEUv6WxxldZdbRkmcW41YWe9JORIcuanBpPDGLbwToDRGWZYmD0C5zZ+r6DQfU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1776256947; c=relaxed/simple; bh=AA/4BL2Th9GaGSXx0JWYhGRyELvEkyDn0qoQqHwlRY4=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=eQTSLxZOOkuFquSFZ4MXnQOL9dd/vyCUTYsEF+gdb90PcJAD4ULIeZ+ZNBmudKRRBxSklR4eutaZssNo7GlxA/yQmGXsQ5gvmXRVu8/g1Bg4hx367Q60WMkfFHLZD/OlJotQ/5Az31H59pWw99nse63GmcW1zjkbiWB9pNw2v5Q= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com; spf=pass smtp.mailfrom=arm.com; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b=I/gkplXr; arc=none smtp.client-ip=217.140.110.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=arm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b="I/gkplXr" Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 9076B4FAA; Wed, 15 Apr 2026 05:42:19 -0700 (PDT) Received: from [10.1.196.46] (e134344.arm.com [10.1.196.46]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id C98C43F86F; Wed, 15 Apr 2026 05:42:21 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=arm.com; s=foss; t=1776256945; bh=AA/4BL2Th9GaGSXx0JWYhGRyELvEkyDn0qoQqHwlRY4=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=I/gkplXrcgeTdBvVd6kRCSikjhqvwwM4q84/lOBKhgxihlVMOMmBb5ZLx9I7Sb/5K hMJCAIymxYL/kAM1vgnoE7exD1it2+8DovdyPmpHgby/48WcBtKREqipp8mPR22DOb XLrCFphUsSKzt4UMCstFSdS7NbTCUvD8t8dHt8qA= Message-ID: Date: Wed, 15 Apr 2026 13:42:20 +0100 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Thunderbird Daily Subject: Re: [Question mpam mpam/snapshot+extras/v6.18-rc1] Question with Configuring iommu_group in 'task' To: Zeng Heng , James Morse Cc: Qinxin Xia , amitsinght@marvell.com, baisheng.gao@unisoc.com, baolin.wang@linux.alibaba.com, carl@os.amperecomputing.com, dave.martin@arm.com, david@kernel.org, dfustini@baylibre.com, fenghuay@nvidia.com, gshan@redhat.com, jonathan.cameron@huawei.com, kobak@nvidia.com, lcherian@marvell.com, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, peternewman@google.com, punit.agrawal@oss.qualcomm.com, quic_jiles@quicinc.com, reinette.chatre@intel.com, rohit.mathew@arm.com, scott@os.amperecomputing.com, sdonthineni@nvidia.com, xhao@linux.alibaba.com, Linuxarm References: <08aacacc-e3e5-4e97-858b-cbcbdb9a1fcf@arm.com> <7b2681c3-9ef7-4882-ba57-2503934e2759@huawei.com> <0d578c61-1341-4149-9e8c-936ea0ce0c50@arm.com> Content-Language: en-US From: Ben Horgan In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Hi Zeng, On 4/15/26 02:27, Zeng Heng wrote: > Hi Ben, > > On 2026/4/13 23:02, Ben Horgan wrote: >> Hi Qinxin, >> >> On 4/3/26 03:44, Qinxin Xia wrote: >>> >>> >>> On 2026/3/27 18:47:49, Ben Horgan wrote: >>>> Hi Qinxin, >>>> >>>> On 3/27/26 10:21, Qinxin Xia wrote: >>>>> >>>>> Hello everyone! >>>>> >>>>> In earlier versions, mpam supports the configuration of iommu_groups. >>>>> >>>>>    823 static ssize_t rdtgroup_tasks_write(struct kernfs_open_file *of, >>>>>    824                                     char *buf, size_t nbytes, >>>>> loff_t off) >>>>>    825 { >>>>>    826         struct rdtgroup *rdtgrp; >>>>>    827         int iommu_group_id; >>>>>    828         bool is_iommu; >>>>>    829         char *pid_str; >>>>>    830         int ret = 0; >>>>>    831         pid_t pid; >>>>>    832 >>>>>    833         rdtgrp = rdtgroup_kn_lock_live(of->kn); >>>>>    834         if (!rdtgrp) { >>>>>    835                 rdtgroup_kn_unlock(of->kn); >>>>>    836                 return -ENOENT; >>>>>    837         } >>>>>    838         rdt_last_cmd_clear(); >>>>>    839 >>>>>    840         if (rdtgrp->mode == RDT_MODE_PSEUDO_LOCKED || >>>>>    841             rdtgrp->mode == RDT_MODE_PSEUDO_LOCKSETUP) { >>>>>    842                 ret = -EINVAL; >>>>>    843                 rdt_last_cmd_puts("Pseudo-locking in progress\n"); >>>>>    844                 goto unlock; >>>>>    845         } >>>>>    846 >>>>>    847         while (buf && buf[0] != '\0' && buf[0] != '\n') { >>>>>    848                 pid_str = strim(strsep(&buf, ",")); >>>>>    849 >>>>>    850                 is_iommu = string_is_iommu_group(pid_str, &iommu_group_id); >>>>> >>>>> What puzzles me is why we would put it under 'task'—this seems a little >>>>>    strange to users.It seems they are not related.Why don't we add a new >>>>> interface like 'iommu'? >>>> >>>> I think it is likely that this interface would change if upstream support is added. >>>> >>> >>> I have done some work in this direction before, and I will release an >>> RFC later for further discussion.:-) >> >> Looking forward to seeing it. >> >> Ben >> > > Following the current SMMU approach, I've submitted several bugfix > patches for the MPAM driver, but haven't received any review feedback > yet. > > To avoid these being overlooked, I'd like to kindly remind to take a > look: > v2: https://lore.kernel.org/all/20260414032610.1523958-1-zengheng4@huawei.com/ > v1: https://lore.kernel.org/all/20251107063300.1580046-1-zengheng4@huawei.com/ > > Additionally, I'd like to check on the status of this branch — is it > still actively maintained? It would be helpful to understand the future > plans for MPAM development. The MPAM snapshot and extras branches are no longer maintained. Work on these has stopped so that we can focus on upstream. Apologies for not making this clear earlier. Thanks, Ben > > > Thanks for your time, > Zeng Heng