Re: [PATCH V4] acpi: acpica: fix acpi parse and parseext cache leaks

[Seunghun Han]

Hello, Andy.

Thank you for your reply.

Patch V4 is the last patch which is applied all changes from original code.
Due to maintainer's advice, Patch V1, V2, V3 have reverted and I made
the latest patch, Patch V4.
(the review is here, https://github.com/acpica/acpica/pull/278)

Therefore, it seems that incremental patch is not needed.

If you have any request, please let me know.

Best regards.

ps) I am sending email again after removing HTML part.

2017-06-26 7:12 GMT+09:00 Seunghun Han <kkamagui@xxxxxxxxx>:
> Hello, Andy.
>
> Thank you for your reply.
>
> Patch V4 is the last patch which is applied all changes from original code.
> Due to reviewer's advice, Patch V1, V2, V3 has reverted and I made the
> latest patch, Patch V4.
> Therefore, it seems that incremental patch is not needed.
>
> If you have any requests, please let me know.
>
> Best regards.
>
>
> 2017ë 6ì 26ì (ì) ìì 1:13, Andy Shevchenko <andy.shevchenko@xxxxxxxxx>ëì ìì:
>>
>> On Fri, Jun 23, 2017 at 12:36 PM, Seunghun Han <kkamagui@xxxxxxxxx> wrote:
>> > I'm Seunghun Han, and I work for National Security Research Institute of
>> > South Korea.
>>
>>
>> > -                                       /* Clean up */
>> > -                                       do {
>> > -                                               if (op) {
>> > -                                                       status2 =
>> > -
>> > acpi_ps_complete_this_op
>> > -                                                           (walk_state,
>> > op);
>> > -                                                       if (ACPI_FAILURE
>> > -                                                           (status2)) {
>> > -
>> > return_ACPI_STATUS
>> > -
>> > (status2);
>> > -                                                       }
>> > -                                               }
>> > -
>> > -                                               acpi_ps_pop_scope(&
>> > -
>> > (walk_state->
>> > -
>> > parser_state),
>> > -                                                                 &op,
>> > -
>> > &walk_state->
>> > -
>> > arg_types,
>> > -
>> > &walk_state->
>> > -
>> > arg_count);
>> > -
>> > -                                       } while (op);
>>
>> I didn't get, do you send an incremental patch as a new version?!
>>
>> --
>> With Best Regards,
>> Andy Shevchenko