Re: [PATCH] gfp_t

[Rasmus Andersen]

--orO6xySwJI16pVnm
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Thu, Sep 26, 2002 at 09:29:32PM +0200, Rasmus Andersen wrote:
> On Thu, Sep 26, 2002 at 08:59:24PM +0200, Rasmus Andersen wrote:
> > On Thu, Sep 26, 2002 at 02:35:30PM +1000, Rusty Russell wrote:
> > > This creates a mythical gfp_t for passing gfp states, and conversion
> > > macros __GFP() and __UNGFP(), to give warnings, It's 55k, so
> > > compressed and attached.
> >=20
> > This breaks ntfs/malloc.h which is doing the following:
> > 49:       return __vmalloc(size, GFP_NOFS | __GFP_HIGHMEM, PAGE_KERNEL);
> [...]
>=20
> After having had a bit more caffeine, I guess I would like to
> change my previous mail to: These two patches for mm/numa.c and
> ntfs/malloc.h needs to be in your patchset as well.

Another one:

--- linux-2.5.38/drivers/char/synclink.c	Sun Sep 22 06:25:06 2002
+++ linux-2.5.38-gfp/drivers/char/synclink.c	Thu Sep 26 23:01:57 2002
@@ -3975,7 +3975,7 @@
 		/* inspect portions of the buffer while other portions are being */
 		/* updated by the adapter using Bus Master DMA. */
=20
-		info->buffer_list =3D kmalloc(BUFFERLISTSIZE, GFP_KERNEL | GFP_DMA);
+		info->buffer_list =3D kmalloc(BUFFERLISTSIZE, gfp_sub(GFP_KERNEL, __UNGF=
P(GFP_DMA)));
 		if ( info->buffer_list =3D=3D NULL )
 			return -ENOMEM;
 		=09
@@ -4087,7 +4087,7 @@
 		} else {
 			/* ISA adapter uses system memory. */
 			BufferList[i].virt_addr =3D=20
-				kmalloc(DMABUFFERSIZE, GFP_KERNEL | GFP_DMA);
+				kmalloc(DMABUFFERSIZE, gfp_sub(GFP_KERNEL, __UNGFP(GFP_DMA)));
 			if ( BufferList[i].virt_addr =3D=3D NULL )
 				return -ENOMEM;
 			phys_addr =3D isa_virt_to_bus(BufferList[i].virt_addr);
@@ -4159,7 +4159,7 @@
  */
 int mgsl_alloc_intermediate_rxbuffer_memory(struct mgsl_struct *info)
 {
-	info->intermediate_rxbuffer =3D kmalloc(info->max_frame_size, GFP_KERNEL =
| GFP_DMA);
+	info->intermediate_rxbuffer =3D kmalloc(info->max_frame_size, gfp_sub(GFP=
_KERNEL, __UNGFP(GFP_DMA)));
 	if ( info->intermediate_rxbuffer =3D=3D NULL )
 		return -ENOMEM;
=20
Regards,
  Rasmus

--orO6xySwJI16pVnm
Content-Type: application/pgp-signature
Content-Disposition: inline

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

iD8DBQE9k3jYlZJASZ6eJs4RAhoZAJ9IP4qAmTOxdEM+d/4z6cp+chAqogCeNxP9
488+gq3JKQRuqd2G6dShN9Q=
=xYvN
-----END PGP SIGNATURE-----

--orO6xySwJI16pVnm--