lkml.org 
[lkml]   [2012]   [Dec]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [tip:x86/microcode] x86/microcode_intel_early.c: Early update ucode on Intel's CPU
On 12/14/2012 11:57 PM, Yinghai Lu wrote:
>
> I tailored your patch and made use 2M page increase to replace patch
> ioremap function.
>
> [PATCH v6 12/27] x86: use io_remap to access real_mode_data
>
> and it will extend init_level4_pgt to map extra range. that will limit
> affect to even others.
>
> please check if that is ok to you.
>

What is the point of only managing 2M at a time? Now you have to have
more conditionals and you don't get any more memory efficiency.

Filling arbitrarily into the brk is not acceptable... the brk is an O(1)
area and all brk allocations need to be reserved at compile time, so the
overflow handling is still necessary.

So no, this patch is not acceptable.

-hpa




\
 
 \ /
  Last update: 2012-12-15 21:01    [W:0.156 / U:1.384 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site