Date:   Fri, 10 Mar 2023 13:18:29 -0400
From:   Jason Gunthorpe <jgg@nvidia.com>
To:     Longfang Liu <liulongfang@huawei.com>
Cc:     alex.williamson@redhat.com, shameerali.kolothum.thodi@huawei.com,
        jonathan.cameron@huawei.com, cohuck@redhat.com,
        linux-kernel@vger.kernel.org, linuxarm@openeuler.org
Subject: Re: [PATCH v8 2/5] hisi_acc_vfio_pci: extract public functions for
 container_of
Message-ID: <ZAtmZZp7YrbEl15j@nvidia.com>
References: <20230217084831.35783-1-liulongfang@huawei.com>
 <20230217084831.35783-3-liulongfang@huawei.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20230217084831.35783-3-liulongfang@huawei.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?VjEXBygXy1aSce81X3xIxiASOA27OX/NH8kGWwHRYo2NSCyDyL6MlK6o5YVF?=
 =?us-ascii?Q?ke4DYAgYznaBBnv5P3FeZ8QDr0FpYx+d+aiWGOUzPD+hL0NujR5rXoYaKdB5?=
 =?us-ascii?Q?DYAuf3rn6W5x7vhJqT8tAujcJy3H2ZHSwHsRaPjeurnw5nDVurEqeGinWh5x?=
 =?us-ascii?Q?h+NRGiFLOMZMBdJGZmIIGaxQQKIdrzjqz6K/2ZWLEY1NHpKobw2x6R3hEPhg?=
 =?us-ascii?Q?wJjTuxelHEHxNdZpj5m0nENYxVmJ3BkXDC0jX6Qq0DPZ2r+e98/+9zFHtUUn?=
 =?us-ascii?Q?KBU2itaDsk1/mDT1PcK2ZSVlPO1oX32auRReLW5Qf3R6MbXtZI5J/3J5dDHM?=
 =?us-ascii?Q?sFrG8bYLf6RrarBEn3eCT0ZJW5tETButX18hzkm6kHZG7Qb5TLMtdgDwRGOG?=
 =?us-ascii?Q?HFMRbdt3YYUAv7d9ZUOiUtjRGD9Dve6segMJRPAw9h/hO5vREv/7gQWZy3gi?=
 =?us-ascii?Q?wAY7UoJgz4mvK79ReSjjJLad8U0Bt/FA9DDCqNyWdQHyJAlUxihqOn2g0fIj?=
 =?us-ascii?Q?/UT9t+c3PsKV7RKNZ+utVKkg9ybY0yIpPKVGTGAQXvzFdY5pZQAeG88te0gJ?=
 =?us-ascii?Q?sEC7hP9RVzxh8vBtaGGXShVXjh2mNgggFKswnFt0vmGnRfD87AfXN74W1Pxx?=
 =?us-ascii?Q?VAKv+S34K4oI2ysfxOeXFr6TQG2m2cHpKmy90OtdtXal/34+s7w4eW2mzvwA?=
 =?us-ascii?Q?k7Y7m1ODqXP/Mj1AQ6CbNd8syVBAfr8MVCSGrQLx67NQgAVGaNiqxHRJNn2v?=
 =?us-ascii?Q?hSoRsD1GkKXAJMEQifSWZUdt9lt0XZh6ElPtw7rfFEsXgY9aPOVbJhf5ROaz?=
 =?us-ascii?Q?9JqsNpp4f7QIcFn4mA/oDT8Xzf713bEIUMP1RQ2F/Ctiw9lOqNmxwspmdPDp?=
 =?us-ascii?Q?2aNSFMqy+RigU1yirv+AUTEXpgqvjIJ0n+Kg0n9+UjmlFXfOJlpSzgoaCr1K?=
 =?us-ascii?Q?z6qE2BGGCgKQfKi8vyIfJxPhdRChzG9An3Uboyy5QtoyuTbVUgGUhAfVoO0r?=
 =?us-ascii?Q?xX0CL1YDuBPxoB8eL3TP4tlzzHu9lAwwEAmEW+vZzVlqQgrBNOLMKZxFvkfA?=
 =?us-ascii?Q?gUzF6Ne7NMFDzLV7EyLtTaK68MEPELIE5VtMflXku2IzYMDs6akSooso9sI2?=
 =?us-ascii?Q?hqM48iPlPdPwe9O0Y2qSEBtm6hyBOAdlGQ8Wvnlm9OnAYEsfkDJy9J8jW7Ix?=
 =?us-ascii?Q?xuIYfC1YLobI8WACF75HCvkLv20fH8NVhudewigwAmovhNtdCQXhgCVCwJG6?=
 =?us-ascii?Q?yU2Ep6YPvGMmj6Ba5sbpGnHR3wo63PymBsKF3rLpNtMNZDd4Kc+2ac6au3XJ?=
 =?us-ascii?Q?VRVhSS3vYEXXH9QYi+pCf4qKBL1FE5RmYtPj9xz68CzgCnZF8qdhENDz3x2q?=
 =?us-ascii?Q?i+Qkpw4HouQp+DsHJGvB+7ijNLENJIUR4lqtfuatzcaFEgbscBT6MKOnH1Cy?=
 =?us-ascii?Q?WcdmF/DeTLvIn5ReJ4ni/AjnLRv/oxd2fOP5QOBJNyu1P2Zn+nLeCTtWhM0t?=
 =?us-ascii?Q?15zkifguTNWM2BNPvK2zOc9CXZgU5JYiIm2gLqgJTuXlcur1p9f6ZAVPOMOm?=
 =?us-ascii?Q?2j+YYgA2BRnXmOOhagAJFm/6UT49ScE6aeTfxUWb?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 824946c9-2650-44ab-33a2-08db218b7b2b
X-MS-Exchange-CrossTenant-AuthSource: LV2PR12MB5869.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 Mar 2023 17:18:35.5843
 (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: ZnSmkm+KIqRBVQK64+fMVN9m5e8qOQiWQHB8A3/RSoY4puXu4LFTMjhOzEG/hPZ8
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW4PR12MB6949
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Fri, Feb 17, 2023 at 04:48:28PM +0800, Longfang Liu wrote:
> In the current driver, vdev is obtained from struct
> hisi_acc_vf_core_device through the container_of function.
> This method is used in many places in the driver. In order to
> reduce this repetitive operation, I extracted a public function
> to replace it.
> 
> Signed-off-by: Longfang Liu <liulongfang@huawei.com>
> ---
>  .../vfio/pci/hisilicon/hisi_acc_vfio_pci.c    | 21 ++++++++++---------
>  1 file changed, 11 insertions(+), 10 deletions(-)

Reviewed-by: Jason Gunthorpe <jgg@nvidia.com>

Jason