linux-kernel - RE: [PATCH v10 1/6] iommu: Change kconfig around IOMMU

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <MW5PR11MB5881463F11D86198291EB52989DCA@MW5PR11MB5881.namprd11.prod.outlook.com>
Date:   Fri, 27 Oct 2023 09:14:17 +0000
From:   "Zhang, Tina" <tina.zhang@...el.com>
To:     Joerg Roedel <joro@...tes.org>
CC:     "iommu@...ts.linux.dev" <iommu@...ts.linux.dev>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "David Woodhouse" <dwmw2@...radead.org>,
        Lu Baolu <baolu.lu@...ux.intel.com>,
        "Will Deacon" <will@...nel.org>,
        Robin Murphy <robin.murphy@....com>,
        "Jason Gunthorpe" <jgg@...pe.ca>,
        "Tian, Kevin" <kevin.tian@...el.com>,
        Nicolin Chen <nicolinc@...dia.com>,
        Michael Shavit <mshavit@...gle.com>,
        Vasant Hegde <vasant.hegde@....com>,
        Jason Gunthorpe <jgg@...dia.com>
Subject: RE: [PATCH v10 1/6] iommu: Change kconfig around IOMMU_SVA

Hi Joerg and Jason,

> -----Original Message-----
> From: Zhang, Tina
> Sent: Friday, October 27, 2023 3:37 PM
> To: Joerg Roedel <joro@...tes.org>
> Cc: iommu@...ts.linux.dev; linux-kernel@...r.kernel.org; David Woodhouse
> <dwmw2@...radead.org>; Lu Baolu <baolu.lu@...ux.intel.com>; Will Deacon
> <will@...nel.org>; Robin Murphy <robin.murphy@....com>; Jason
> Gunthorpe <jgg@...pe.ca>; Tian, Kevin <kevin.tian@...el.com>; Nicolin Chen
> <nicolinc@...dia.com>; Michael Shavit <mshavit@...gle.com>; Vasant
> Hegde <vasant.hegde@....com>; Jason Gunthorpe <jgg@...dia.com>
> Subject: RE: [PATCH v10 1/6] iommu: Change kconfig around IOMMU_SVA
> 
> Hi Joerg,
> 
> > -----Original Message-----
> > From: Joerg Roedel <joro@...tes.org>
> > Sent: Friday, October 27, 2023 3:12 PM
> > To: Zhang, Tina <tina.zhang@...el.com>
> > Cc: iommu@...ts.linux.dev; linux-kernel@...r.kernel.org; David
> > Woodhouse <dwmw2@...radead.org>; Lu Baolu
> <baolu.lu@...ux.intel.com>;
> > Will Deacon <will@...nel.org>; Robin Murphy <robin.murphy@....com>;
> > Jason Gunthorpe <jgg@...pe.ca>; Tian, Kevin <kevin.tian@...el.com>;
> > Nicolin Chen <nicolinc@...dia.com>; Michael Shavit
> > <mshavit@...gle.com>; Vasant Hegde <vasant.hegde@....com>; Jason
> > Gunthorpe <jgg@...dia.com>
> > Subject: Re: [PATCH v10 1/6] iommu: Change kconfig around IOMMU_SVA
> >
> > Hi Tina,
> >
> > On Fri, Oct 27, 2023 at 08:05:20AM +0800, Tina Zhang wrote:
> > > From: Jason Gunthorpe <jgg@...dia.com>
> > >
> > > Linus suggested that the kconfig here is confusing:
> >
> > While this patch looks decent to me, you forgot to Cc Linus on it. In
> > general, given that it touches a couple of core files, I'd like to
> > wait for some more people to have a look at it and not rushing anything in.
> Make sense. I'll CC Linus. Comments are welcome.
Can we separate this patch? I'm thinking about CC more people for review. At least, we need to CC all the reviewers of https://lore.kernel.org/all/20230506133134.1492395-1-jacob.jun.pan@linux.intel.com/ to this patch. So, it seems more reasonable to separate this patch. What do you think?

Regards,
-Tina