2007-01-31 04:58:04

by Kumar Gala

[permalink] [raw]
Subject: Reducing warning output from pci_get_subsys()

Greg,

There was some code added to warn if pci_get_subsys() is called and the
pci_devices is empty.

I'm wondering if there is some point at which we know its ok for the
pci_devices list be empty if there are no devices on the bus so we can
stop printing the message.

On an embedded PPC reference system I see this message 6 times when I've
got no cards in the PCI slots.

- k


2007-01-31 05:11:59

by Andrew Morton

[permalink] [raw]
Subject: Re: Reducing warning output from pci_get_subsys()

On Tue, 30 Jan 2007 22:55:38 -0600 (CST)
Kumar Gala <[email protected]> wrote:

> Greg,
>
> There was some code added to warn if pci_get_subsys() is called and the
> pci_devices is empty.
>
> I'm wondering if there is some point at which we know its ok for the
> pci_devices list be empty if there are no devices on the bus so we can
> stop printing the message.
>
> On an embedded PPC reference system I see this message 6 times when I've
> got no cards in the PCI slots.
>

I'd suggest we just remove the warning. Also the one in pci_find_subsys().

I let them go through because I was curious to know what would cause it to
trigger - it might be an indication of other bugs. But nothing very
interesting happened and they're of no use.

2007-01-31 05:19:18

by Kumar Gala

[permalink] [raw]
Subject: Re: Reducing warning output from pci_get_subsys()


On Jan 30, 2007, at 11:11 PM, Andrew Morton wrote:

> On Tue, 30 Jan 2007 22:55:38 -0600 (CST)
> Kumar Gala <[email protected]> wrote:
>
>> Greg,
>>
>> There was some code added to warn if pci_get_subsys() is called
>> and the
>> pci_devices is empty.
>>
>> I'm wondering if there is some point at which we know its ok for the
>> pci_devices list be empty if there are no devices on the bus so we
>> can
>> stop printing the message.
>>
>> On an embedded PPC reference system I see this message 6 times
>> when I've
>> got no cards in the PCI slots.
>>
>
> I'd suggest we just remove the warning. Also the one in
> pci_find_subsys().
>
> I let them go through because I was curious to know what would
> cause it to
> trigger - it might be an indication of other bugs. But nothing very
> interesting happened and they're of no use.

Works for me.

- k

2007-01-31 06:13:53

by Greg KH

[permalink] [raw]
Subject: Re: Reducing warning output from pci_get_subsys()

On Tue, Jan 30, 2007 at 11:18:21PM -0600, Kumar Gala wrote:
>
> On Jan 30, 2007, at 11:11 PM, Andrew Morton wrote:
>
> >On Tue, 30 Jan 2007 22:55:38 -0600 (CST)
> >Kumar Gala <[email protected]> wrote:
> >
> >>Greg,
> >>
> >>There was some code added to warn if pci_get_subsys() is called
> >>and the
> >>pci_devices is empty.
> >>
> >>I'm wondering if there is some point at which we know its ok for the
> >>pci_devices list be empty if there are no devices on the bus so we
> >>can
> >>stop printing the message.
> >>
> >>On an embedded PPC reference system I see this message 6 times
> >>when I've
> >>got no cards in the PCI slots.
> >>
> >
> >I'd suggest we just remove the warning. Also the one in
> >pci_find_subsys().
> >
> >I let them go through because I was curious to know what would
> >cause it to
> >trigger - it might be an indication of other bugs. But nothing very
> >interesting happened and they're of no use.
>
> Works for me.

No objection from me either.

thanks,

greg k-h