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

John Baldwin jhb at freebsd.org
Tue Jan 9 09:22:14 PST 2007


On Monday 08 January 2007 19:24, Pyun YongHyeon wrote:
> 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.

Ah, ok.

-- 
John Baldwin


More information about the cvs-src mailing list