Received: by 2002:a25:868d:0:0:0:0:0 with SMTP id z13csp2797082ybk; Mon, 18 May 2020 08:09:01 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwku+vsczFm3aJWx0cfyY70lLFleHRPn/X0Q3CH6QwkANDAx6xKdjYvpMsVCmoJK/OG0tOs X-Received: by 2002:a05:6402:3128:: with SMTP id dd8mr14155218edb.156.1589814540908; Mon, 18 May 2020 08:09:00 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1589814540; cv=none; d=google.com; s=arc-20160816; b=ZUU3rEvKVNL0oRUKeLaa+6zWE3W8Mx0dWcba79nj71ijI3OyyqteljMRpVHUMR9dyf hCSgCbrwDHRGOE2mXIjYYY+px3dWxupEV6KGmlPteOkJTMZih1KO80DMzDPSQaSKn0xw EgXBrRHiFPKMcKc5eTKy1NG3ysDmQkizcb5Rr/8Gm33V4j0kXmCucHegJ2WaKLefgJMJ h/z3AqBrldv6Gtuco6TVY0FCKtx4sQLK90WqshG7buNvZNdOCJLQWyA1yYUEY+HFgEzg /c7XV9sYyfvLzN2krxzytxbEeGuJMv84oY2vzgbCH2GCQsUnQlKOzqDfvLzfwlHsNprG 7Akg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject:dkim-signature; bh=zhBCLV6Pq+PIRNumJDFC+hT6dCYyurG69WuSPlFC+DE=; b=UpaQkncOq5cDmq8Cn3eKBuApO0awktQ7qV1N1HEK8JOiXll1we/rmPfKHdkDrbKCVN CaHZ9mUsfVQ87te2/xjt4UrSdOy+wqYgvlZp8oubJk5Po7U686I1qQVj6U3Ju+dqvXPS axMFdhUMn28JW3mAUmHAx1flqlGJDloOEiJI1vtG7sVU9J6neQJjHKItmQJQI0zcs8BU r1J9QOZkSm/kjIMZDCD9T0a17AHmK9IvjTOhrs9nUP5F0sS557jCerYPvsfjOg98DJSj leNFYjM8mU1JygHZF/o5XMaCudLW1clOpzWRrmt6bZ29zwsY9e7qklk9RTAlNzXwtdTh tesg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@ieee.org header.s=google header.b="SW4u/tpY"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=ieee.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id o1si5868495edv.414.2020.05.18.08.08.36; Mon, 18 May 2020 08:09:00 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@ieee.org header.s=google header.b="SW4u/tpY"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=ieee.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728289AbgERPD5 (ORCPT + 99 others); Mon, 18 May 2020 11:03:57 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:34884 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726998AbgERPD4 (ORCPT ); Mon, 18 May 2020 11:03:56 -0400 Received: from mail-io1-xd43.google.com (mail-io1-xd43.google.com [IPv6:2607:f8b0:4864:20::d43]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C07D0C05BD09 for ; Mon, 18 May 2020 08:03:56 -0700 (PDT) Received: by mail-io1-xd43.google.com with SMTP id k18so10962003ion.0 for ; Mon, 18 May 2020 08:03:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ieee.org; s=google; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=zhBCLV6Pq+PIRNumJDFC+hT6dCYyurG69WuSPlFC+DE=; b=SW4u/tpYmJKb5hF00ZiUK7OHKeHyg8BR91taO3uS78PK3g2ArFVd9a62KZv6xuy8ox /kg1mA67VfJUMDPBwmNigfm8eIhITbGuopGqaY1CWn85xlwmQVq71zmFMKFTU6s06wDW ycU30OfwVZ0ye2/wpGID46Ev86K9JMmJHXpTE= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=zhBCLV6Pq+PIRNumJDFC+hT6dCYyurG69WuSPlFC+DE=; b=DjBi7mFsFW93xRZE25a3FvulmirdPcxibuji3/CR/IX9qG5Mhgrxp4xr+wc68axlZI AL2FuiuSURDh+AbbPQM+fxjThDhOVaoIHl7bHOWhBiOxI3gp5dOgc9K2L8sLanekPXB+ hAddpRpjFyTLNuYtdpNbzi845oZ/qL3wNxHth5gl9C6li1nH68XeaxApXsFgO76iBiRW +dNFwds2vuRFM6vNh2y+5xe0JEuSfTfcxINb/Qof2DoUMXh+IOVnypnxdNWVvQgAVyZH Uot7kAgKDfT4ectVvhEQjd1d+9/0jAemCRrssW4tmMeCT9ijn4AICvnj+rxMpnyTqvZ6 vXMw== X-Gm-Message-State: AOAM533QgdS6w6fBnzwkktQhincbv/y8chWJ1tIFWLB7xGEMrcqfG/AI a105a5JaP+qF8Ipg945sHJA7YA== X-Received: by 2002:a02:a895:: with SMTP id l21mr15553885jam.82.1589814234911; Mon, 18 May 2020 08:03:54 -0700 (PDT) Received: from [172.22.22.4] (c-73-185-129-58.hsd1.mn.comcast.net. [73.185.129.58]) by smtp.googlemail.com with ESMTPSA id q4sm4383395ilk.12.2020.05.18.08.03.53 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 18 May 2020 08:03:54 -0700 (PDT) Subject: Re: [PATCH] drivers: ipa: use devm_kzalloc for simplicity To: David Miller , kuba@kernel.org Cc: wenhu.wang@vivo.com, elder@kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, kernel@vivo.com References: <20200514035520.2162-1-wenhu.wang@vivo.com> <20200514101516.0b2ccda2@kicinski-fedora-pc1c0hjn.dhcp.thefacebook.com> <20200514.124700.630104670938763588.davem@davemloft.net> From: Alex Elder Message-ID: Date: Mon, 18 May 2020 10:03:52 -0500 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.7.0 MIME-Version: 1.0 In-Reply-To: <20200514.124700.630104670938763588.davem@davemloft.net> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 5/14/20 2:47 PM, David Miller wrote: > From: Jakub Kicinski > Date: Thu, 14 May 2020 10:15:16 -0700 > >> On Wed, 13 May 2020 20:55:20 -0700 Wang Wenhu wrote: >>> Make a substitution of kzalloc with devm_kzalloc to simplify the >>> ipa_probe() process. >>> >>> Signed-off-by: Wang Wenhu >> >> The code is perfectly fine as is. What problem are you trying to solve? > > I agree, these kinds of transformations are kind of excessive. I have considered using the devm_*() functions, but if I were to make such a switch it would be done comprehensively, throughout the driver. It is not something I plan to do in the near term though. -Alex