From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id ADA3EA0C52; Wed, 24 Nov 2021 09:40:22 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 6058440A4B; Wed, 24 Nov 2021 09:40:22 +0100 (CET) Received: from out3-smtp.messagingengine.com (out3-smtp.messagingengine.com [66.111.4.27]) by mails.dpdk.org (Postfix) with ESMTP id 93A5040688 for ; Wed, 24 Nov 2021 09:40:21 +0100 (CET) Received: from compute5.internal (compute5.nyi.internal [10.202.2.45]) by mailout.nyi.internal (Postfix) with ESMTP id E13265C0183; Wed, 24 Nov 2021 03:40:20 -0500 (EST) Received: from mailfrontend1 ([10.202.2.162]) by compute5.internal (MEProxy); Wed, 24 Nov 2021 03:40:20 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h= from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding:content-type; s=fm2; bh= oUUQoYUg5ddIIxKQvSIU/GYdrQZEhxyG0nOVHJ3fmKg=; b=ByI5+f/O1emymozk QZDPOc5LCKWhNV+j0AKklqsBwbdUyxSv9XoZsjBnKlJHz5YGW8vJwqxiLmFEuu3X eVndmqGVT9g/Ix7Fb2vEz4jlkP425GUgNDVoHzPi37v8B2gR4RLWf+6VyefIVL5s WyVpaeX6MdurNEZ5wljD5DJzQJvdG8qip1OygMFPDGN0XbLB7o9OBNlpyUkLPjCz dLaPPk95b10FPANkehWZdX6zdV8y416T/AgrlFEouq8TSwRigciY3ky2iWVEnMyI UiDo6P0IOuS/G1xeXo7fOR9H2kTXDlBxeV7QPu43Ffe2WNtZLDpRUBh8A6H99eLb vDBlbw== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm1; bh=oUUQoYUg5ddIIxKQvSIU/GYdrQZEhxyG0nOVHJ3fm Kg=; b=G36MA3tUHswQ70xsmPoViaU8mD+Zuyiuj5+myLWuPQu+f/lORWT9sZB9e tgtorTFVbEpHHO0HBs0zcOjE4Dfy6mm0V1tObGKiA0hun8paz5SE2oAmqZVli87B 9V7uNJ8p1cXgb/EvDVdaGI6x/ZXmFqS5KBIKL3QqRE/iloVwZmh9pyggDXOsjcVJ m/Yqqb8Icw2F2H/eLHQb4QIVeHOzDKpNwllOBPUN6O9LX4rhGaVC2+mj2lOUOX3C to/jRqLv6dfE3FGlnnXLxaYRbNLkN08ClXT5Dr0uiYoiFuynRMRWkZ7OugRb1Kno j+BtkQXCnPGD96zJ82xI+BXLFuVtA== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvuddrgeejgdduvdefucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhephffvufffkfgjfhgggfgtsehtufertddttddvnecuhfhrohhmpefvhhhomhgr shcuofhonhhjrghlohhnuceothhhohhmrghssehmohhnjhgrlhhonhdrnhgvtheqnecugg ftrfgrthhtvghrnhepudeggfdvfeduffdtfeeglefghfeukefgfffhueejtdetuedtjeeu ieeivdffgeehnecuvehluhhsthgvrhfuihiivgeptdenucfrrghrrghmpehmrghilhhfrh homhepthhhohhmrghssehmohhnjhgrlhhonhdrnhgvth X-ME-Proxy: Received: by mail.messagingengine.com (Postfix) with ESMTPA; Wed, 24 Nov 2021 03:40:20 -0500 (EST) From: Thomas Monjalon To: Elena Agostini Cc: dev@dpdk.org Subject: Re: [PATCH v7] gpudev: manage NULL pointer Date: Wed, 24 Nov 2021 09:40:18 +0100 Message-ID: <4804769.LHGoIO6cyH@thomas> In-Reply-To: <20211123004207.24733-1-eagostini@nvidia.com> References: <20211118192802.23955-1-eagostini@nvidia.com> <20211123004207.24733-1-eagostini@nvidia.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org 23/11/2021 01:42, eagostini@nvidia.com: > From: Elena Agostini > > gpudev free and unregister functions return > gracefully if input pointer is NULL because > API doc was indicating NULL as a no-op > accepted value. > > cuda driver checks are removed because > redundant with the checks added > in gpudev library. > > Fixes: e818c4e2bf50 ("gpudev: add memory API") > > Signed-off-by: Elena Agostini Applied with updated commit message: gpudev: manage null parameters in memory functions The gpudev functions free, register and unregister return gracefully if input pointer is NULL or size 0, as API doc was indicating no-op accepted values. CUDA driver checks are removed because redundant with the checks added in gpudev library.