Messages in this thread | | | Date | Sat, 15 Dec 2012 11:30:24 -0800 | From | "H. Peter Anvin" <> | Subject | Re: [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
| |