Whoops. You're right... Thanks. I'll give it another try.
On Thu, Apr 21, 2005 at 07:24:46PM +0200, Frans Pop said: > On Thursday 21 April 2005 14:53, Mike Edwards wrote: > > unsigned long __get_free_pages(unsigned int gfp_mask, unsigned int > > order) { > > struct page * page; > > > > page = alloc_pages(gfp_mask, order); > > if (!page) { > ^^^ > > if (order > 0) > > printk("DEBUG: __get_free_pages() order %d > > failed " "called by %p\n", > > order, __builtin_return_address(0)); > > return 0; > } > ^^^ > > return (unsigned long) page_address(page); > > } > > You've forgotten to add the brackets for the if (!page) statement so now > the return 0 is always executed... > I've added them above. -- Mike Edwards <[EMAIL PROTECTED]> System Administrator Psychology Department, Rutgers University, Newark campus -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]