Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <eb773959-b9f1-8fb0-305b-b9318d3d0055@amd.com>
Date:   Mon, 4 Jul 2022 15:09:35 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.9.1
Subject: Re: [PATCH 00/10] drm: use idr_init_base() over idr_init() if
 applicable
Content-Language: en-US
To:     dakr@redhat.com, airlied@linux.ie, daniel@ffwll.ch, emma@anholt.net
Cc:     dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org
References: <20220701185303.284082-1-dakr@redhat.com>
From:   =?UTF-8?Q?Christian_K=c3=b6nig?= <christian.koenig@amd.com>
In-Reply-To: <20220701185303.284082-1-dakr@redhat.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?MHBJRFFmUWN5YWRobzNONUVubEVjbTV4NnR0dkxvd3UzZDY0MEpLazVWNW1T?=
 =?utf-8?B?VmlFNE5Jbk96b1d0UmdMOGdkY29OdktvbnF5amZwMnBSbXMvL0ZTVlpwYlpv?=
 =?utf-8?B?SWdjR2JONDRhanFiM1paWjlMNEhNaXJEQU1wRG5aa2MyMnM5NWFJYTIzSkE5?=
 =?utf-8?B?a2FsV1Z6K0taN2tPME1jQ2F6UU9YUFY5LzVWSWtWc3loeW8wNStPekc5OHNB?=
 =?utf-8?B?bXV1RVZucVd1TlJEZ2dhdnJJY2hUZEdLYUU0Q25Xb2hROTRpc2VhT1R2ekxC?=
 =?utf-8?B?a2M4Y1dwWHVZNlM3WmlDWHdHOGZlaWp3dVZZeWtna09oajI2VFlSOGM2UWpM?=
 =?utf-8?B?bllzYjV5OEtrTzlySmRHdUdjME81SWNqMitjcFdLYW00dnNqU1E0MllyaVZ1?=
 =?utf-8?B?c2NIcm9jZlRRMlRoTU5aYUJidE1NMjVzZ3BvZmZ0RmZkYzJHYVdpOXNpVWxo?=
 =?utf-8?B?SGhJclhhY1dmVkFvU2x2ZExSYTc3THMrMnJJSnBsWFZaNU03RGczdFUxWTlX?=
 =?utf-8?B?akNZTWdKNUUxbUNVMjF2RFRFKzRYUVlHZldvVmZKdnpFRHBVSFJLYUVWTk5t?=
 =?utf-8?B?V1dOTUhYeTU3d1drVzk2YVRTMkMxbXIyNEtwSER4SktvZWNod01zVU4zTCt1?=
 =?utf-8?B?SEVEVTNBU3ljVGVsTTBVYk5jdGE4amV6ckI0ek05eGo2UDVYR05LUTNFMjdl?=
 =?utf-8?B?ZENaZC82Zi9ORUVNblRFTlNQMHNOTW1jcGgrcndJd05GcGoyQmIxak5BcWV4?=
 =?utf-8?B?S3RMNmt6UEdjM1ErYkQ2WlV5eW5kMlJiUCs1QzBkR0V2Y1B3emJlSWVXVXlG?=
 =?utf-8?B?eDcwOWZwUTRWeWM1T05COWd2WmNBSzJqOVNtMTdlYTNMZTZuK1hPY1RHZHor?=
 =?utf-8?B?eDVVYVZTUTgxZTE5UHYxVkNCSUxrc2xSTWdQK2VPQmhpbkZyRXNDeGxSV1ha?=
 =?utf-8?B?SVBFRStIK25KNXhNY1kyNEw0bE5wejlJWVNvdTYxSG96YStLdU5IVFVERXJK?=
 =?utf-8?B?UVNDVWNtSEtRb3E5TG5KcHhvWitXRTc4TjVkaFJzUEpYZlBHanEyblJzTjVj?=
 =?utf-8?B?aHFWdEZnSE1TUS9Ib2crZDh4RzNxMmpwQ3JXR2hQbHNWemxrQndLck45aW5j?=
 =?utf-8?B?NEtCK1JmblUvMkVSUXIrOFVYZjFncWNicElzd1pVb3VsTXhLdDZ1QTNmUkNG?=
 =?utf-8?B?SXRmcmpIV1M3RGQrZDBueEorNjNuakhFUkhYRmJrTkpwM3pud2RGSUtOT3JF?=
 =?utf-8?B?WXFLSGJuMUlvWmk0a0twckZzV0JtRFJtYk5WUWZ2aWlOQTBCd2JlMnFLL2J4?=
 =?utf-8?B?NUUxL3hiTHdLZC9OWm94K2lOYVlkNjFZbmFISkd3WWFsdnI3Zjd0bW5CMk1C?=
 =?utf-8?B?WlhZSDNzNnRxNHJ1UlBZQnQ1QTVuVWozd3NFZ0tsL0RpZVUyUVBEemFsZm9w?=
 =?utf-8?B?WCsyckcrK3AyeGhmb0J0dkd3VzRzajc5NlpJT1R3TDhRZW11QmdZK0xGVDdh?=
 =?utf-8?B?WEcwS2NNRlpBRHpKKzBYSDNPRFdUL2QwWFBocFlVWWNKdEZrL1R6RkkvMSt6?=
 =?utf-8?B?MFZwZk02UWFVTlQwejZHSXcxQTNJMzZ4RkdhTjJhSTZmcXBaVlMzWkhUM2VV?=
 =?utf-8?B?cCtXaC95VFRHSDQ4eGprOGpJZkNXMGdxdHdGenlUSVNNdkMvMUJKVGtYd0ds?=
 =?utf-8?B?b05BdVhnRGUyUnpxYXBWc3RxckZwZ09uMTVrcFFNWjlrVTJCN2NCa3ZJaFRh?=
 =?utf-8?B?enV4ZmVaeE1ONUlRMVByWWxNZGZ4c0U3MjhnNVNnRkJvWVMydGJ6cURlUnp0?=
 =?utf-8?B?K1MwOVhZbFVYM2FzV2w3cWRidkl0S2pJSEhGTXAvMlBLWHFjYUpBRktrai9Z?=
 =?utf-8?B?MTNhSktFZjBqSzVuUFBBOWZmMjBLL0NDWGJNdG16YnlYd2czc0s1c3pjMjBk?=
 =?utf-8?B?THV3UTdWZ1NlOTdmR2U3NjRxa0xLdVdmdUY5Y1gyeFY5QnIxMUc3MzFkbHJB?=
 =?utf-8?B?R3JBeThkWlNwWVRGTnZoamphZk9mVG1mUFMwdnZTRTM0ZWhiMmFXN1BjR01q?=
 =?utf-8?B?dncyVlQvYTZtYitaV3IreGxWVmhGOWRQN2lkRDRnNWdDYXVnaUV2a2QwTDNX?=
 =?utf-8?B?Z2VtNFc0ZmdrWW9Ja052SjViZ0VkM0ZRVVJzeU9JZEdGbHJwWVRpNHluZlZW?=
 =?utf-8?Q?WvwwY/ZRtutYWgvrD/FdDaHu2CPAVNN4czNA5YnBjsny?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 690b7e16-2963-4717-ab79-08da5dbe754c
X-MS-Exchange-CrossTenant-AuthSource: BN8PR12MB3587.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Jul 2022 13:09:42.1818
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 1RxAK6qRuqOis52eVD7R+35FH3ruQCjTmV+zzMSmEIe8iosgALSAUAAwJdBGD+tb
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY5PR12MB3716
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE
        autolearn=ham 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

Am 01.07.22 um 20:52 schrieb dakr@redhat.com:
> From: Danilo Krummrich <dakr@redhat.com>
>
> This patch series initializes IDRs with idr_init_base(&idr, 1) rather than
> idr_init(&idr) in case for the particular IDR no IDs < 1 are ever requested -
> this avoids unnecessary tree walks.

Feel free to add Reviewed-by: Christian König <christian.koenig@amd.com> 
for the two amdgpu patches.

And an Acked-by: Christian König <christian.koenig@amd.com> for the rest.

Regards,
Christian.

>
> Danilo Krummrich (10):
>    drm/amdgpu: use idr_init_base() to initialize mgr->ctx_handles
>    drm/amdgpu: use idr_init_base() to initialize fpriv->bo_list_handles
>    drm: use idr_init_base() to initialize master->magic_map
>    drm: use idr_init_base() to initialize master->lessee_idr
>    drm: use idr_init_base() to initialize mode_config.object_idr
>    drm: use idr_init_base() to initialize mode_config.tile_idr
>    drm/sis: use idr_init_base() to initialize dev_priv->object_idr
>    drm/v3d: use idr_init_base() to initialize v3d_priv->perfmon.idr
>    drm/via: use idr_init_base() to initialize dev_priv->object_idr
>    drm/todo: remove task for idr_init_base()
>
>   Documentation/gpu/todo.rst              | 12 ------------
>   drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c |  2 +-
>   drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c |  2 +-
>   drivers/gpu/drm/drm_auth.c              |  4 ++--
>   drivers/gpu/drm/drm_mode_config.c       |  4 ++--
>   drivers/gpu/drm/sis/sis_drv.c           |  2 +-
>   drivers/gpu/drm/v3d/v3d_perfmon.c       |  2 +-
>   drivers/gpu/drm/via/via_map.c           |  2 +-
>   8 files changed, 9 insertions(+), 21 deletions(-)
>