Re: [PATCH][2.5] replace flush_map() in arch/i386/mm/pageattr.c with flush_tlb_all()

• Next message: Pavel Machek: "Re: ioctl32 consolidation"
• Previous message: Nick Piggin: "Re: IO scheduler benchmarking"
• In reply to: Dave Jones: "Re: [PATCH][2.5] replace flush_map() in arch/i386/mm/pageattr.c with flush_tlb_all()"
• Next in thread: Hugh Dickins: "Re: [PATCH][2.5] replace flush_map() in arch/i386/mm/pageattr.c w ith flush_tlb_all()"
• Reply: Hugh Dickins: "Re: [PATCH][2.5] replace flush_map() in arch/i386/mm/pageattr.c w ith flush_tlb_all()"
• Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

On Thu, Feb 20, 2003 21:50, Dave Jones wrote:
> Its hinting at a possible optimisation, not saying
> that it is unneeded.

OK, sorry, than I just misunderstood the comment...

So here is a minimal change patch that should solve the preempt issue in
flush_map().

Instead of just doing a preempt_disable() before and a preempt_enable() after
the flush_kernel_map() calls I just changed the order so that the preempt
point is not between them...

  Thomas

• text/x-diff attachment: flush_map_preempt.patch

• application/pgp-signature attachment: signature

-
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/

• Next message: Pavel Machek: "Re: ioctl32 consolidation"
• Previous message: Nick Piggin: "Re: IO scheduler benchmarking"
• In reply to: Dave Jones: "Re: [PATCH][2.5] replace flush_map() in arch/i386/mm/pageattr.c with flush_tlb_all()"
• Next in thread: Hugh Dickins: "Re: [PATCH][2.5] replace flush_map() in arch/i386/mm/pageattr.c w ith flush_tlb_all()"
• Reply: Hugh Dickins: "Re: [PATCH][2.5] replace flush_map() in arch/i386/mm/pageattr.c w ith flush_tlb_all()"
• Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

This archive was generated by hypermail 2b29 : Sun Feb 23 2003 - 22:00:33 EST