From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v7 0/2] clean up interrupt handle Date: Thu, 06 Apr 2017 21:16:15 +0200 Message-ID: <3390852.FyvIhZzGCW@xps13> References: <20170404230908.15145-1-qi.z.zhang@intel.com> <20170406124222.18172-1-qi.z.zhang@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org To: Qi Zhang Return-path: Received: from mail-wr0-f179.google.com (mail-wr0-f179.google.com [209.85.128.179]) by dpdk.org (Postfix) with ESMTP id AC3D02B9B for ; Thu, 6 Apr 2017 21:16:17 +0200 (CEST) Received: by mail-wr0-f179.google.com with SMTP id t20so77249560wra.1 for ; Thu, 06 Apr 2017 12:16:17 -0700 (PDT) In-Reply-To: <20170406124222.18172-1-qi.z.zhang@intel.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 2017-04-06 20:42, Qi Zhang: > It seems its not necessary to register an intr_handle for interrupt callback > function. "void* cb_arg" shows enough when be used to pass the object that > contain the information be required to handle the interrupt event( A typical > way which is implemented by almost all driver is by passing a rte_ethdev > instance). > The patch change the prototype of rte_intr_callback_fn by removing > the uncessary intr_handle paramter. Applied, thanks