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 665A319E79 for ; Wed, 21 Jun 2023 19:31:33 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 6ED7AC433C0; Wed, 21 Jun 2023 19:31:33 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1687375893; bh=aSUfMQ+z77ChqbqenDG0JSS6hcwr+vm4UXaeb8jHlQg=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=jPdLqHqn54FUNrjYTKlMEhYzJ2+3WHygEcp/pUrvVCrm6TzKFKRnEfBnXscl5Tfqd Bx+63S5vnrPzLODXe88YpmryxodiMFU9kZrGgbgmACzJs3mWRXuiw7n7V4jcqbyKkm vjBo9u2m/KxD7S5gEsohpjaxj4gy5wdyeXTcO3JNN3PI0sLycmjATqbY5455tacJvq HgXoQHkE18uqbc0qfadsJzUyARsKtBUvyZoYwZ2+mOgHhw+Ez3heqiFJANWoA1YLIF +IoTejpDR8Ov9HzTLufaWzgvWjUBuX+85eTHAvNuLbLROMGzWwtPRbn9FApqsR3Jsg 1O1z+mT2SvLEw== Date: Wed, 21 Jun 2023 12:31:32 -0700 From: Jakub Kicinski To: Sagi Grimberg Cc: Hannes Reinecke , Christoph Hellwig , Keith Busch , linux-nvme@lists.infradead.org, Eric Dumazet , Paolo Abeni , netdev@vger.kernel.org, Boris Pismenny Subject: Re: [PATCH 4/4] net/tls: implement ->read_sock() Message-ID: <20230621123132.2e0b487b@kernel.org> In-Reply-To: <476d2cd9-ae32-a4e6-4549-52c3863d4049@grimberg.me> References: <20230620102856.56074-1-hare@suse.de> <20230620102856.56074-5-hare@suse.de> <5bbb6ce4-a251-a357-3efc-9e899e470b9c@grimberg.me> <20230620100843.19569d60@kernel.org> <35f5f82c-0a25-37aa-e017-99e6739fa307@grimberg.me> <476d2cd9-ae32-a4e6-4549-52c3863d4049@grimberg.me> 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-Transfer-Encoding: 7bit On Wed, 21 Jun 2023 12:49:21 +0300 Sagi Grimberg wrote: > > Good suggestion. > > Will be including it in the next round. > > Maybe more appropriate helper names would be > tls_rx_reader_enter / tls_rx_reader_exit. > > Whatever Jakub prefers... I was thinking along the same lines but with __ in front of the names of the factored out code. Your naming as suggested in the diff is better.