From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 1784AA04DB for ; Thu, 15 Oct 2020 20:57:14 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id ECECD1C1F8; Thu, 15 Oct 2020 20:57:12 +0200 (CEST) Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id EB2061C1F8; Thu, 15 Oct 2020 20:57:09 +0200 (CEST) IronPort-SDR: fXCyX3zeT61xx4w2wKyX3ub7+XESPbMVCMdrmKsC4j6wuSCg58NU0GjmOXAonw796qKiol0orL YYAAaSI6a/xQ== X-IronPort-AV: E=McAfee;i="6000,8403,9775"; a="165652462" X-IronPort-AV: E=Sophos;i="5.77,380,1596524400"; d="scan'208";a="165652462" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga004.jf.intel.com ([10.7.209.38]) by fmsmga103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 15 Oct 2020 11:56:54 -0700 IronPort-SDR: 8Geu0QW2SAqKNCWxqs1zoz1gj7EBKjQEnpU8umOk89h685Gkum/i3XLriYz+Iuyfxp9JbrlIrX exO0EcJHPhNA== X-IronPort-AV: E=Sophos;i="5.77,380,1596524400"; d="scan'208";a="464405907" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.251.84.112]) ([10.251.84.112]) by orsmga004-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 15 Oct 2020 11:56:52 -0700 To: Tianfei zhang , dev@dpdk.org, rosen.xu@intel.com, wei.huang@intel.com Cc: stable@dpdk.org References: <1600846213-18093-1-git-send-email-tianfei.zhang@intel.com> <1601257218-6606-1-git-send-email-tianfei.zhang@intel.com> <1601257218-6606-2-git-send-email-tianfei.zhang@intel.com> From: Ferruh Yigit Message-ID: Date: Thu, 15 Oct 2020 19:56:48 +0100 MIME-Version: 1.0 In-Reply-To: <1601257218-6606-2-git-send-email-tianfei.zhang@intel.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit Subject: Re: [dpdk-stable] [dpdk-dev] [PATCH v2 1/4] raw/ifpga/base: fix bug in IRQ functions X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Sender: "stable" On 9/28/2020 2:40 AM, Tianfei zhang wrote: > From: Wei Huang > > Using a pointer instead of using a structure and point to > ifpga_irq_handle[] in register and unregister interrupt > functions. > Treat positive return value of ifpga_unregister_msix_irq() > as successful. > > Fixes: e0a1aafe ("raw/ifpga: introduce IRQ functions") > Cc: stable@dpdk.org > > Signed-off-by: Wei Huang > Signed-off-by: Tianfei zhang I suggest commit log as following: raw/ifpga/base: fix interrupt handler instance usage Interrupt handler copied to the local 'intr_handle' variable by value before passing it to IRQ functions. This leads IRQ functions update the local variable instead of 'ifpga_irq_handle'. Instead, using 'intr_handle' local variable as pointer to 'ifpga_irq_handle' as intended. Also handle unsupported interrupt type requests properly, on unsupported interrupt case: 'ifpga_unregister_msix_irq()' returns success 'ifpga_register_msix_irq()' return failure. Fixes: e0a1aafe2af9 ("raw/ifpga: introduce IRQ functions") Cc: stable@dpdk.org The "Also" part highlights that patch addressed two different issues, for next time please split different fixes to the different patches. Title "fix bug" doesn't give much information, better to give some context. And for the following part in the original commit log: " Treat positive return value of ifpga_unregister_msix_irq() as successful. " It is missing in the patch, I see that part is in the next patch :) +1 to update, since 'rte_intr_callback_unregister()' can return positive, but perhaps better to move the change too into its own patch.