[...]
>>> @@ -662,6 +661,9 @@ static int zipl_load_segment(ComponentEntry *entry)
>>> */
>>> break;
>>> }
>>> +
>>> +comp_len += bprs->size * (bprs[i].blockct + 1);
>>> +
>>
>> I'm confused by the arithmetic here. Why is size multiplied
On 7/14/25 6:10 PM, Collin Walling wrote:
> On 7/11/25 5:10 PM, Zhuoying Cai wrote:
>> Make the address variable a parameter of zipl_load_segment and return
>> segment length.
>
> There's mixed use of the term "comp_len" and "segment length". Since
> the context here is "zipl_load_segment", perha
On 7/11/25 5:10 PM, Zhuoying Cai wrote:
> Make the address variable a parameter of zipl_load_segment and return
> segment length.
There's mixed use of the term "comp_len" and "segment length". Since
the context here is "zipl_load_segment", perhaps the variable should be
"seg_len"?
>
> Modify th
Make the address variable a parameter of zipl_load_segment and return
segment length.
Modify this function for reuse in the next patch, which allows
loading segment or signature data to the destination memory address.
Add a comp_len variable to store the length of a segment and return this
variab
