cvs commit: src/sys/dev/msk if_msk.c

Pyun YongHyeon pyunyh at gmail.com
Mon Jan 8 16:51:30 PST 2007


On Mon, Jan 08, 2007 at 11:35:59AM -0500, John Baldwin wrote:
 > On Sunday 07 January 2007 19:19, Pyun YongHyeon wrote:
 > > yongari     2007-01-08 00:19:53 UTC
 > > 
 > >   FreeBSD src repository
 > > 
 > >   Modified files:
 > >     sys/dev/msk          if_msk.c 
 > >   Log:
 > >   Initialize legacy SYS_RES_IRQ resources before attempting to use MSI.
 > >   This fixes legacy SYS_RES_IRQ resource allocation failure when MSI is
 > >   disabled.
 > >   
 > >   Reported by:    rrs
 > >   Tested by:      rrs
 > 
 > Gah, my bad.  I would prefer to set it down before the 'if (msic == 2...)' so 
 > all the logic for it is in one place, but anyways, you should remove the 

At first I thought that too. But mskc_attach() checks chip id and
then exits if the chip is not supported one. If that happens
mskc_detach() will call bus_release_resources() with invalid
SYS_RES_IRQ resources.

 > now-duplicate setting here:
 > 
 >                 } else {
 >                         pci_release_msi(dev);
 >                         sc->msk_irq_spec = msk_irq_spec_legacy;

Indeed, I'll do. Thanks a lot!

 >                 }
 > 
 > -- 
 > John Baldwin

-- 
Regards,
Pyun YongHyeon


More information about the cvs-src mailing list