From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 6DA081B7DA for ; Fri, 18 Aug 2023 18:51:52 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id A867BC433C8; Fri, 18 Aug 2023 18:51:50 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1692384711; bh=pidLHaTD/OAu/s3IGdmpPPGAw+jWYE00bBcIdCAuQA0=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=BcgYQOz9wf8yakoBN6pzjaJsiLiiObwbxYfqt5PjWifELfjBCLJIk9AgNeIMyVP2U 3nt1flNNQqxxLKpSAOZGNWTObdVH4ttrsn045KBIYzAtyOLz4BstcayPTZfKU+NY48 a4S+n+mxfR11ldYct+hpEfgxEAvPUw3+9ua8MYa1H3iOb0kwLkmNb9gXWFXtd7kQ2S rN4qO1KeY9Co9yQ5bbV+zahPBwQK+8McrrdoVx+67D9gTo/4+623qCp5b7GmGFU2jL jYZgaTDAyT8kWennBoFgMadn5pud+WwdOvHT7orhPUyzZ2Tk4+hDWUx2+vPO4rZluA mYymY6nGtR1lg== Date: Fri, 18 Aug 2023 21:51:43 +0300 From: Leon Romanovsky To: Tony Nguyen Cc: davem@davemloft.net, kuba@kernel.org, pabeni@redhat.com, edumazet@google.com, netdev@vger.kernel.org, Przemek Kitszel , Jacob Keller , Simon Horman , Pucha Himasekhar Reddy Subject: Re: [PATCH net-next v2 13/15] ice: ice_aq_check_events: fix off-by-one check when filling buffer Message-ID: <20230818185143.GE22185@unreal> References: <20230817212239.2601543-1-anthony.l.nguyen@intel.com> <20230817212239.2601543-14-anthony.l.nguyen@intel.com> Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20230817212239.2601543-14-anthony.l.nguyen@intel.com> On Thu, Aug 17, 2023 at 02:22:37PM -0700, Tony Nguyen wrote: > From: Przemek Kitszel > > Allow task's event buffer to be filled also in the case that it's size > is exactly the size of the message. > > Fixes: d69ea414c9b4 ("ice: implement device flash update via devlink") > Reviewed-by: Jacob Keller > Signed-off-by: Przemek Kitszel > Reviewed-by: Simon Horman > Tested-by: Pucha Himasekhar Reddy (A Contingent worker at Intel) > Signed-off-by: Tony Nguyen > --- > drivers/net/ethernet/intel/ice/ice_main.c | 13 +++++++------ > 1 file changed, 7 insertions(+), 6 deletions(-) > Thanks, Reviewed-by: Leon Romanovsky