From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.0 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 962E6C433E0 for ; Mon, 21 Dec 2020 13:59:52 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 53B1322BED for ; Mon, 21 Dec 2020 13:59:52 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726268AbgLUN7w (ORCPT ); Mon, 21 Dec 2020 08:59:52 -0500 Received: from mail.kernel.org ([198.145.29.99]:44868 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726127AbgLUN7v (ORCPT ); Mon, 21 Dec 2020 08:59:51 -0500 Date: Mon, 21 Dec 2020 19:29:05 +0530 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1608559151; bh=3ZAmEitTCsexx/51HtE9t/c0rky6LhobvkdB5sVl5sM=; h=From:To:Cc:Subject:References:In-Reply-To:From; b=ZjONlJTHHCY2wgcC5QY1lJ/xQ6TrR4vG6z/pBhsTGSONkemgNGHUfBiBVY0cCuiHW EvCPC/KZ7MAo6brN/1afejx8nFHPmWzpzOh9s4Nkp7oxbbdodq5rLgZB/DwediBmvq YmrsLX2qo7x4BrhJZFoQ0QdRYiGex02Imsxx5ws4Xt70kTyMp0Oa6sDbuu9SgQLAcS 5ivrj5rJ9eMhKJL3+io/zQDe9U0vf54wPeIdlrO5SlG58x15bI4HJc/YBIWmyXzzTo LVhbEtAfb19UY7cwsFEZ+sdAD8wi8gLeDU/miXge9pHM3lBjaJNzaN3ORM2G3AlqTY Gi3/k/Fnk1Bwg== From: Vinod Koul To: Dan Carpenter Cc: Dave Jiang , Dan Williams , dmaengine@vger.kernel.org, kernel-janitors@vger.kernel.org Subject: Re: [PATCH] dmaengine: idxd: off by one in cleanup code Message-ID: <20201221135905.GB3323@vkoul-mobl> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: dmaengine@vger.kernel.org On 16-12-20, 11:29, Dan Carpenter wrote: > The clean up is off by one so this will start at "i" and it should start > with "i - 1" and then it doesn't unregister the zeroeth elements in the > array. Applied, thanks -- ~Vinod