Messages in this thread | | | Date | Sat, 9 Dec 2000 16:04:03 +0100 | From | Martin Mares <> | Subject | Re: pdev_enable_device no longer used ? |
| |
Hi!
> I noticed a lot of drivers are setting the PCI_CACHE_LINE_SIZE > themselves, some to L1_CACHE_BYTES/sizeof(u32), others > to arbitrary values (4, 8, 16). > > Then I spotted that we have a routine in the PCI subsystem > (pdev_enable_device) that sets all these to L1_CACHE_BYTES/sizeof(u32) > Further digging revealed that this routine was not getting called. > > On my Athlon box, I've noticed some devices are getting their > default values set to 8 (where (I think) it should be 16 ?) > > Questions: > 1. Is there reason for the drivers to be setting this themselves > to hardcoded values ?
Definitely not unless the devices are buggy and need a work-around.
For PC's, we've until now relied on the BIOS setting up cache line sizes correctly. Are the "8"'s you've spotted due to drivers messing with the cache line register or do they come from the BIOS?
> 2. Why is pdev_device_enable no longer used ?
I haven't written this function, but I guess it's expected to be called from the architecture specific device enabling code and that some architectures have to call it, some don't.
Have a nice fortnight -- Martin `MJ' Mares <mj@ucw.cz> <mj@suse.cz> http://atrey.karlin.mff.cuni.cz/~mj/ bug, n: A son of a glitch. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org Please read the FAQ at http://www.tux.org/lkml/
| |