linux-kernel - Re: [REGRESSION 5.19.x] AMD HD-audio devices missing on 5.19

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 for Android: free password hash cracker in your pocket

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

Message-ID: <YxihMchpLRPAqNWx@nvidia.com>
Date:   Wed, 7 Sep 2022 10:48:33 -0300
From:   Jason Gunthorpe <jgg@...dia.com>
To:     Takashi Iwai <tiwai@...e.de>
Cc:     Lu Baolu <baolu.lu@...ux.intel.com>,
        Joerg Roedel <jroedel@...e.de>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Bjorn Helgaas <bhelgaas@...gle.com>,
        Robin Murphy <robin.murphy@....com>,
        Eric Auger <eric.auger@...hat.com>,
        regressions@...ts.linux.dev, linux-kernel@...r.kernel.org
Subject: Re: [REGRESSION 5.19.x] AMD HD-audio devices missing on 5.19

On Wed, Sep 07, 2022 at 03:28:31PM +0200, Takashi Iwai wrote:
> >  /**
> >   * iommu_device_use_default_domain() - Device driver wants to handle device
> >   *                                     DMA through the kernel DMA API.
> > @@ -3094,8 +3112,7 @@ int iommu_device_use_default_domain(struct device *dev)
> >  
> >  	mutex_lock(&group->mutex);
> >  	if (group->owner_cnt) {
> > -		if (group->domain != group->default_domain ||
> > -		    group->owner) {
> > +		if (group->owner || iommu_is_default_domain(group)) {
> 
> Isn't this rather 
> 		if (group->owner || !iommu_is_default_domain(group)) {
> ?
> 
> I'll rebuild the kernel with this change and ask reporters again.

Oh yes, good eyes, that probably crashes on boot too

Jason