[RFC PATCH]: merge identical code
Wang Nan
wangnan0 at huawei.com
Thu Apr 10 19:36:47 PDT 2014
On 2014/4/10 15:56, Atsushi Kumagai wrote:
> Hello Wang,
>
>> Hi Kumagai,
>>
>> I found that in makedumpfile.c, there are many code like following:
>>
>> if (info->flag_cyclic) {
>> proc_cyclic();
>> } else {
>> proc();
>> }
>>
>> The logical and code of proc_cyclic and proc are nearly identical.
>>
>> Do you want to merge them together?
>
> Yes, I think we should do this work.
>
> However, the similarity of them will be reduced since how to update
> cyclic region is going to be changed in v1.5.6.
> If you can carry out this work also for v1.5.6, I'll accept it.
>
> You can use the current devel branch because it's almost v1.5.6.
> (Formally, v1.5.6 will be released next week.)
>
> https://sourceforge.net/p/makedumpfile/code/ci/devel/tree/
>
>
> Thanks
> Atsushi Kumagai
>
What about totaly remove non-cyclic functions and use a very large cyclic?
More information about the kexec
mailing list