Re: pci_pool reap?

[Daniel Stodden]

--=-hG7i5kykPNDPzyD99zgH
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

hi.
=20
On Tue, 2002-02-12 at 03:44, David S. Miller wrote:
>    From: G=E9rard Roudier <[e-mail]>
>    Date: Sun, 10 Feb 2002 21:20:05 +0100 (CET)
>=20
>    On Mon, 11 Feb 2002, Alan Cox wrote:
>   =20
>    > This function may not be called in interrupt context.
>   =20
>    Such limitation looks poor implementation to me.
>=20
> I agree with you Gerard, and probably nobody truly even requires
> this limitation.  I do plan to remove it after I've done a thorough
> investigation of the platform implementations.

ok, i've looked through most of 2.5.4 now.
results look like this:

			pci_alloc_consistent()	pci_free_consistent()
i386:
		[1]	ok			ok

ppc:
		[1]	ok			ok

mips:
		[1]	ok			ok

sh:
		[1]	ok			ok
  stm:		[1]	ok			ok
  dc:		[3]	ok			ok

mips64:
  ip32:		[1]	ok			ok
  ip27:		[1]	ok			ok

sparc:
		[1]	GFP_KERNEL		ok
sparc64:
		[2]	ok			ok

arm:		[4]	BUG()/GFP_KERNEL	BUG()

alpha:
		[2]	ok			ok

ia64:		[5]	ok?			ok?

	=09
[1]
gfp() + __pa() (or similar)

[2]
gfp() + IOMMU

[3]
dummy, offsets only

[4]    =20
ARM does GFP_KERNEL, and then __ioremaps the underlying pages.
ugh. is that the only way to get the area coherent?
furthermore i don't see why this could not be interrupt safe.

[5]
i don't understand ia64. but it looks somewhat atomic :)

well, assuming i didn't oversee anything, there are indeed few reasons
left why the whole _consistent() machinery shouldn't be callable from
interrupts.=20

back to my original question: what were the last trees with shrinking
pools? would the original version still work or any redesigns needed?

regards,
dns

--=20
___________________________________________________________________________
 mailto:[e-mail]

--=-hG7i5kykPNDPzyD99zgH
Content-Type: application/pgp-signature; name=signature.asc
Content-Description: This is a digitally signed message part

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.6 (GNU/Linux)
Comment: For info see http://www.gnupg.org

iD8DBQA8aTaBSPSplX5M5nQRAgapAJ96NCMvx4iLkdgvMWPWHDmNpljKHgCfQhqM
7fOA6lOkDbkzBK7LfZZXP/s=
=VKiF
-----END PGP SIGNATURE-----

--=-hG7i5kykPNDPzyD99zgH--