Vivek Goyal <[EMAIL PROTECTED]> writes:

> On Wed, May 24, 2006 at 01:40:41PM +0900, Magnus Damm wrote:
>>  
>> @@ -170,45 +151,16 @@ void machine_kexec_cleanup(struct kimage
>>  NORET_TYPE void machine_kexec(struct kimage *image)
>>  {
>>      unsigned long page_list;
>> -    unsigned long reboot_code_buffer;
>> -
>> +    unsigned long control_code;
>> +    unsigned long page_table_a;
>>      relocate_new_kernel_t rnk;
>>  
>> -    /* Interrupts aren't acceptable while we reboot */
>> -    local_irq_disable();
>
> Why are you getting rid of this call? Looks sane to disable interrupts
> early.

Agreed, this and changing the segment and idt handling is gratuitous.
If you are going to make multiple unrelated changes place do them
as separate patches.

Eric
_______________________________________________
fastboot mailing list
[email protected]
https://lists.osdl.org/mailman/listinfo/fastboot

Reply via email to