lkml.org 
[lkml]   [2013]   [Jan]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v7u1 22/31] x86, boot: add fields to support load bzImage and ramdisk above 4G
On 01/13/2013 09:37 PM, Yinghai Lu wrote:
>>
>> This is kinda missing from the mechanism of the sentinel and it should
>> be documented too.
>
> No, we should have too much duplicated info.
>

That is not duplicating info... that is basic documentation. As you
show in the post further on, it took a very simple description, and it
*is* a very subtle thing that is inherently different from how the other
fields operate.

It doesn't help that you didn't, despite repeated requests, implement
what I *asked for*, which is:

If the sentinel is flagged, zero *all fields not explicitly set by the
broken versions of kexec*, not just your new "ext" fields.

Yinghai, I understand you're frustrated, but please understand that
Borislav is not in any shape, way, or form "some guys that do not know
the code well keep sending comments out to waste others time". Rather,
he has spent a huge amount of time giving you an awful lot of good
feedback A lot of them have centered on documentation and code
maintainability, both of which are vitally important part of a
long-lived codebase.

Having someone doing line-by-line review of your code is enormously
time-consuming and not something most people enjoy doing. Borislav is
doing you -- and me -- a huge favor here.

-hpa



--
H. Peter Anvin, Intel Open Source Technology Center
I work for Intel. I don't speak on their behalf.



\
 
 \ /
  Last update: 2013-01-14 19:41    [W:2.769 / U:0.648 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site