Re: [BK PATCH] PCI reorg fix

Martin Dalecki (dalecki@evision-ventures.com)
Thu, 09 May 2002 21:45:04 +0200


Uz.ytkownik Greg KH napisa?:
> Linus,
>
> James pointed out that pci_alloc_consistent() and pci_free_consistent()
> are allowed to be called, even if CONFIG_PCI is not enabled. So this
> changeset moves these calls back into the arch/i386/kernel directory.
>
> Pull from: bk://linuxusb.bkbits.net/linux-2.5-pci
>
> As a side note, I don't think that any pci_* function should be able to
> be called by non-pci drivers. Is it worth spending the time now in 2.5
> to make these two functions not rely on 'struct pci_dev' and fix up all
> of the drivers and architectures and documentation to reflect this?
> Possible names would be alloc_consistent() and free_consistent()?

If your are at it: I was always itching my had what
pci_alloc_inconsistent and pci_free_inconsistent is supposed to be?
Negating semantically the consistent attribute shows nicely
that the _consistent is a bad nomenclatures. Perhaps something
more related to the purpose of it would help. Like

ioalloc() and iofree()

Could be even abstracted from the bus implementation.

And of course much less typing...

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/