On Tue, Mar 01, 2022 at 11:45:59AM -0800, Iouri Tarassov wrote: [...] > +void dxgadapter_remove_shared_resource(struct dxgadapter *adapter, > + struct dxgsharedresource *object) > +{ > + down_write(&adapter->shared_resource_list_lock); > + if (object->shared_resource_list_entry.next) { > + list_del(&object->shared_resource_list_entry); > + object->shared_resource_list_entry.next = NULL; > + } Again, just list_del please. I will skip pointing out this in later patches. Thanks, Wei.