Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1750917AbdFZF0L (ORCPT ); Mon, 26 Jun 2017 01:26:11 -0400 Received: from mail-wr0-f171.google.com ([209.85.128.171]:34667 "EHLO mail-wr0-f171.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750725AbdFZF0C (ORCPT ); Mon, 26 Jun 2017 01:26:02 -0400 MIME-Version: 1.0 In-Reply-To: References: <1498210597-112293-1-git-send-email-kkamagui@gmail.com> From: Seunghun Han Date: Mon, 26 Jun 2017 14:26:00 +0900 Message-ID: Subject: Re: [PATCH V4] acpi: acpica: fix acpi parse and parseext cache leaks To: Andy Shevchenko Cc: "linux-acpi@vger.kernel.org" , "linux-kernel@vger.kernel.org" Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by mail.home.local id v5Q5RP9c008309 Content-Length: 2347 Lines: 78 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 : > 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 님이 작성: >> >> On Fri, Jun 23, 2017 at 12:36 PM, Seunghun Han 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