Received-SPF: softfail (google.com: domain of transitioning linux-kernel-owner@vger.kernel.org does not designate 23.128.96.19 as permitted sender) client-ip=23.128.96.19;
Date:   Mon, 28 Feb 2022 13:57:58 -0400
From:   Jason Gunthorpe <jgg@nvidia.com>
To:     Yajun Deng <yajun.deng@linux.dev>
Cc:     linux-rdma@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH for-next] RDMA/core: Remove unnecessary statements
Message-ID: <20220228175758.GA608324@nvidia.com>
References: <20220223074901.201506-1-yajun.deng@linux.dev>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20220223074901.201506-1-yajun.deng@linux.dev>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?3MHiAv/NdtR9/nlYY8u1ORbMQMVpQ0lIubdPUqNr5x5E9aM9aMSnzxpaJ5iH?=
 =?us-ascii?Q?IE3HNujm3LrYqjsRKqYWyKuetJ0ULu+8msssmbQtN6rNUZOrxTZa9J0erOXK?=
 =?us-ascii?Q?MEs/omCozwokQpSXeOeKy8j4KNPFzuxAL73kOlIABPrpA1M26vBtoboUuWaa?=
 =?us-ascii?Q?IjBO3EALlicCgdi2znqKnxpvRw3TZ9prhLzC8ceoMaJ4HUh+qbfuOYuGxOFr?=
 =?us-ascii?Q?KqUvEeacweY3akhABDqYUJySJtSq7yqc1RY2zGxXXad8SVVc6SxzfUehPKN6?=
 =?us-ascii?Q?EF8G1RvjgVeYm2WxOnbbOVJGClZtf/YGYa/oBWcfJUPtvY9OYHZewDJntJ4i?=
 =?us-ascii?Q?509rLgUfMJicZo3pWxLz16Bwm7gChl74GgqrP0kV0F/QqRC4T+ZzNT8aOQIB?=
 =?us-ascii?Q?UePXp2tzi8Ay8fUwprXTYobEtvcb1XkjJCzk/xDpkqfl23qeWpVL6lox1StV?=
 =?us-ascii?Q?Kj5qaURzsnDLly/MptwYXzwtcRaGoTOMyFs0ig1LxUvBxd/A6/6cmk7wXS8y?=
 =?us-ascii?Q?Tuh3HA+YLL6LYxZAdX/9wXxwM2s6V7XuJkb3GB7IMBk1a5283MAHSTZ8u1uF?=
 =?us-ascii?Q?GmoGhgj69UaYhdMaCdtvsqncCeXlVTO9Ad+EJQtY45jn71wuILDermyYnuud?=
 =?us-ascii?Q?T44uaDXxDnbfQ1E0GWfhl+L+HTxhGejp30B42HZSFCflOywCO7dythkz3XNM?=
 =?us-ascii?Q?gGfIPbZM8jGRHmT5yCwjpLdsObOnpcmMu+0ohfsXNjMupPlXqlUYS4MztpVt?=
 =?us-ascii?Q?+laPgb8SpWrgPKQCEotv90DdL7dJ27oyuqi22BLJyAvxWoHepkeHKKUMIK+I?=
 =?us-ascii?Q?JL0KyHoV13SnWbVbr3nB30yYDV1Xk5F/MVOjrmIXPqVBCGQ6n90a4e2Qkk4N?=
 =?us-ascii?Q?3paZKmUxYS2b/8IV46iHggg+Ji7gFXyaKq6zW4SEapNIsUpel+Qbe+arsenh?=
 =?us-ascii?Q?mVTmHEDwIuPgnsxMCRmev/s6OynEi6THYYPvsOOywCK7G2DwyXKJrws++R7i?=
 =?us-ascii?Q?hIPbZM/rD8IosBCToRF5kLYWkburi/4LRy2wfgfRqjGq4Cj85g/rG3UDsGjs?=
 =?us-ascii?Q?ygmdskBWC+wq0UsIS2dbkqxeFMScMYOK+xIh1Ar78THyANSKT1yP/DiM7m6N?=
 =?us-ascii?Q?32MFZ0nTG5tKbSK3DR8sO9vzywfQM9ZkrL9D4whXG7AlaIFDf+J/qGhezsri?=
 =?us-ascii?Q?M9leWa4BM9HoNlb7aTHUk39wMoOxqBAoi52PusxvBvypASWQgUdA1GUOR1h2?=
 =?us-ascii?Q?QSWvMfXRde8TtM6AKU2NS7JKzHC3xWLlYJgOaVNFIdvL9JghYfXEY5l8l9ZH?=
 =?us-ascii?Q?rAvEjEgurAhmwgtj3A9ZA2+QMN/bpixdpkFvqW3jBnWb/qpqaQJNfjwqOwfh?=
 =?us-ascii?Q?rLKdRir2wG4AxWFgt6hsC/ok0t5jyQJfAHIO7xQSno2bevO+uW6a8BNkzb0G?=
 =?us-ascii?Q?V7VCu7fNrW9+jj5Gq0hBjKSv3nGk6fCoFB4dGsxnnmGdAP7B8QuQIF5p11j8?=
 =?us-ascii?Q?DEpGgy/LLehVu9VOaYlX5Ccb2+QV6y2VARQGVq8sxVL2qp/jeSLZWM2hjOXi?=
 =?us-ascii?Q?/ReFrkZYBb6PZlc7Lrc=3D?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: b82eed02-4dbb-42a5-df5b-08d9fae3db5f
X-MS-Exchange-CrossTenant-AuthSource: MN2PR12MB4192.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Feb 2022 17:57:59.5430
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 2SvPjamkawKxShpR4KDhNVehF2OwP00b8l/QvmhpsiAdYIa7yxZlIIMOUgXNIPo9
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW2PR12MB4667
X-Spam-Status: No, score=-2.4 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
	DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,RDNS_NONE,SPF_HELO_NONE,T_SCC_BODY_TEXT_LINE
	autolearn=unavailable autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
	lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Wed, Feb 23, 2022 at 03:49:01PM +0800, Yajun Deng wrote:
> The rdma_zalloc_drv_obj() in __ib_alloc_pd() would zero pd, it unnecessary
> add NULL to the object in struct pd.
> 
> The uverbs_free_pd() already return busy if pd->usecnt is true, there is
> no need to add a warning.
> 
> Signed-off-by: Yajun Deng <yajun.deng@linux.dev>
> Reviewed-by: Leon Romanovsky <leonro@nvidia.com>
>  drivers/infiniband/core/verbs.c | 7 -------
>  1 file changed, 7 deletions(-)

Applied to for-next, with Leon's note added

Thanks,
Jason