Re: Feedback on preemptible kernel patch

Arjan Filius
2001-09-09T21:24:40Z

Hi,

After my succes report i _do_ noticed something unusual:

I'm not sure it's preempt related, but you wanted feedback :)

Sep 9 23:08:02 sjoerd kernel: __alloc_pages: 0-order allocation failed (gfp=0x70/1).
Sep 9 23:08:02 sjoerd last message repeated 93 times
Sep 9 23:08:02 sjoerd kernel: cation failed (gfp=0x70/1).
Sep 9 23:08:02 sjoerd kernel: __alloc_pages: 0-order allocation failed (gfp=0x70/1).
Sep 9 23:08:02 sjoerd last message repeated 281 times

This is at the very moment i make a ppp connection to internet, and
get/set the time with netdate (for the first time after a reboot).
I didn't see this a second time (yet).

Btw this is 2.4.10-pre4+preempt-patch+pacht-below.

Greetings,

On 9 Sep 2001, Robert Love wrote:

> Arjan,
>
> the following patch was written by Manfred Spraul to fix your highmem
> bug. I haven't had a chance to go over it, but I would like it if you
> could test it. It can't hurt. Patch it on top of the preempt patch and
> enable CONFIG_PREEMPT, CONFIG_HIGHMEM, and CONFIG_HIGHMEM_DEBUG.
>
> let me know what happens...any relevant messages, etc. please pass
> along. if it does work, id be curious if they are any slowdowns
>
>
> --- highmem.h.prev Sun Sep 9 08:59:04 2001
> +++ highmem.h Sun Sep 9 09:00:07 2001
> @@ -88,6 +88,7 @@
> if (page < highmem_start_page)
> return page_address(page);
>
> + ctx_sw_off();
> idx = type + KM_TYPE_NR*smp_processor_id();
> vaddr = __fix_to_virt(FIX_KMAP_BEGIN + idx);
> #if HIGHMEM_DEBUG
> @@ -119,6 +120,7 @@
> pte_clear(kmap_pte-idx);
> __flush_tlb_one(vaddr);
> #endif
> + ctx_sw_on();
> }
>
> #endif /* __KERNEL__ */
>
>
>
>

--
Arjan Filius
mailto:iafilius@xs4all.nl