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: <07c740c3-434a-c88f-2ae4-268351bc3005@intel.com>
Date:   Tue, 22 Feb 2022 00:23:40 +0100
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101
 Firefox/91.0 Thunderbird/91.6.1
Subject: Re: [PATCH v2 00/11] drm/i915: use ref_tracker library for tracking
 wakerefs
Content-Language: en-US
To:     <linux-kernel@vger.kernel.org>, <intel-gfx@lists.freedesktop.org>,
        <dri-devel@lists.freedesktop.org>, netdev <netdev@vger.kernel.org>
CC:     Jani Nikula <jani.nikula@linux.intel.com>,
        Daniel Vetter <daniel@ffwll.ch>,
        Lucas De Marchi <lucas.demarchi@intel.com>,
        Chris Wilson <chris.p.wilson@intel.com>,
        Eric Dumazet <edumazet@google.com>,
        Dmitry Vyukov <dvyukov@google.com>,
        Jakub Kicinski <kuba@kernel.org>
References: <20220221231705.1481059-1-andrzej.hajda@intel.com>
From:   Andrzej Hajda <andrzej.hajda@intel.com>
Organization: Intel Technology Poland sp. z o.o. - ul. Slowackiego 173, 80-298
 Gdansk - KRS 101882 - NIP 957-07-52-316
In-Reply-To: <20220221231705.1481059-1-andrzej.hajda@intel.com>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?c2NpandPWCtjYWJwQUlDelhwbGZJbExyK1l3TFFvTVR3YU11MU9pUU03NEhP?=
 =?utf-8?B?ejkyZHBXS3ZCb0Rzb3VKakRoYkdZL0FOTjQ3a2gxbGtQbDRCT0VIM1ZlbFBn?=
 =?utf-8?B?c2VZR0xrUzByc3FVSVg4Qmk2T21qamNlMHdoNHRGWklKdEpwQ2hyL2VqZ0Vw?=
 =?utf-8?B?RW45VFVzcU1RTExyY3FlejJrWkNVZXlqMkljbE9LY1p5cXZSRmFkQ2ZPSnhU?=
 =?utf-8?B?LzdNLzNndllrSkFSQnNReVdmZEFvR2hLamdMa1JqYXBxTVRaOFZVR2hTTWY1?=
 =?utf-8?B?WkpMbnp0M05DL2J1c091YkFDYWR4dFBSbE1RS1JrM3ozZ3RyRTVCNWFVUk14?=
 =?utf-8?B?ZkdQYThNemhIc0hsci9nc296SDI3U3ordlFJNTYyUFRrSzZHNlpQN2IrbXV2?=
 =?utf-8?B?Wlp0dVFqRHE3MXNXVmExMXdoZWN4aTk3SDdOVUs4anNnZ1hhdlR1d2xOZnpZ?=
 =?utf-8?B?Z3p6OEhFVlVDdk45cDhDekJ1ZStUQ3hYSFpuSFF4UXE5RlpxUmdwR2ZjQUxV?=
 =?utf-8?B?SkdSK051c1UwYklUTDI4OWxibXRGbEdncUxFYk0vZ0VHSjI4SkdTOTFoZkVH?=
 =?utf-8?B?S2luTnFqS1VzUkNDMjVCc2N5dHN1dEdSS3lycERDWEZmSlBtM2JTYkhkeG9C?=
 =?utf-8?B?MWluTWNuQ2tEcDh5bHd1VHRnbVlrazQyTm1ScDJFQnp6dy8xdnBEZ2l1OWNv?=
 =?utf-8?B?WHlCcGpJRFd2QmVJWjFOODYyYnNRZ0txcnFyOHN2YXFFZzY4RzcrNnQzNm9Q?=
 =?utf-8?B?NEljeUZQMENTY0Rvb2NKMkRGRjYwZUNIOEhKM2FNOTl4dnp6L2xtdGJpSElF?=
 =?utf-8?B?WTRqK01KQ3J3cktVcklIL3cwb2NQQUxwaTQxbWlZeTMrVGhEdXJuS2xlbHR2?=
 =?utf-8?B?RVdwM25JTWFMalVodG53UjI3RXRYcC9sbXFqV2lkZmVGLzdRK2ovZFByK21J?=
 =?utf-8?B?TzMzM2UwV0lsYWNOVkZhdllaWXZWZk9CZ2Z0aWNJNURBV3NPc0RqaVlJTk9H?=
 =?utf-8?B?RDN1YU1mYmVWSzh3dFVqUmczK2cvdzQvUExuNUpWOUloWEFoS3o5c2YraGZW?=
 =?utf-8?B?Z0xZU0RzZ3dKUVNWdjhNdkJYcXBZMWh1RnArK243ZmxxMklnWEZERldxTGRD?=
 =?utf-8?B?OVVPdWU4WmtHZGlFQkE5T3E3enpla0ozNkJEQ1FsK3crNnhTdmY5K3NZTWwz?=
 =?utf-8?B?NWVxZ2ZRMkVCOTNmQnVUMzB2T2hpUmwvczN2TW8vY0ZFYjlya25MeFIvRnZU?=
 =?utf-8?B?a0MyK3M4V1ArUEFGTXJDYWNyNzFxYUVpTFo3Y0UxZUFmWmlscG5DV0V1eENT?=
 =?utf-8?B?VzFYSjNSWFpxYm84Q3NNK3FRYUE2SEhyVzEwclArMnhDM3FTWDNreGhvUEg2?=
 =?utf-8?B?Y3ZNTUhYY0kvandZY01yclhqV3ZzK3g4Ry8zV0pUaVkvWTEzOFdNQitqM3JK?=
 =?utf-8?B?aEIyR1hYUU5LTkFmM2Z3enpvTHNtbUlhY3RSQUhBRmlCZkFsS0pkSUR3Y0gw?=
 =?utf-8?B?ZTRZR3E0WHl4aU5KUkFhSDhwc1I2SWFOWHlyVy9TRExzaVlnbGhaSTlZVTNw?=
 =?utf-8?B?TFFxRkZOVlJKV1cyU1hOMHRGVnVldXhPZC9Tb1FGOStoSzdTWk10NDRUWkhM?=
 =?utf-8?B?azBWQmNNaCszcDhVUHpzUVBkdk5JT2dRWTNxY1g0b2dPVjBaODR5Vld1R2ls?=
 =?utf-8?B?U09DUXBGZUljYk56YzZRa3dJN2UrZ04yRmQ1Ynd4QkNvK3JlMDQ5aU1TdWVL?=
 =?utf-8?B?b3N3bHRSMDZSV0l1QmxPUXp1M0tKVm5wSVpOYlNFdjJDUmVvZFZnWkR0VUFH?=
 =?utf-8?B?cmlqWTNwdGJNbWZLOWswNmt5MGlFYkV0Y2h4WDVZZUlnTVhlc3lub0U2ZUo3?=
 =?utf-8?B?YVBwdEV1dUFMQUV0ejQ3L01UQWx2YjhEVVYvM0VuRGY3eTBBMTJtblpxK1Z0?=
 =?utf-8?B?alkydXloZzNsTU5EN3E0MXlzRHRDTHRQMHFmbXd5L0RJOEx2TFpCQlVtYWlz?=
 =?utf-8?B?SEYvQnBlcGNPbktwRzA5WE5kSUZaLzJSZnV3TlhqakJnaGEyaEhmajMzMXBR?=
 =?utf-8?B?QzA4djZGS3E4L2pXNG50Qmt5Zm1NL3FUQ1gwV0hCVGJzY2I0YkFubEx4V25O?=
 =?utf-8?B?VXM2ODhJUE1QakoxWFB2Tmlkck9lb1hRR3hTcTdvV2lYNy9pODJ2WHpaQStU?=
 =?utf-8?Q?aGUMuOv9h/j5Xeg9BY7k0GU=3D?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 077247ad-8cd1-43ee-372f-08d9f5913606
X-MS-Exchange-CrossTenant-AuthSource: DM6PR11MB3180.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 21 Feb 2022 23:23:47.7731
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: MgFfDmbSTYZotsnYp1vkZOXY7LD9fkNFUrXVNvd+5KBdIyE8KpdS6TH7MPBraLOQw7vOZ5a0ySoDlLSo0YWmSw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM5PR11MB1593
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-7.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,
        RCVD_IN_DNSWL_HI,RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,
        SPF_NONE,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


On 22.02.2022 00:16, Andrzej Hajda wrote:
> Hi,
>
> Appearance of ref_tracker library allows to drop custom solution for wakeref
> tracking used in i915 and reuse the library.
> For this few adjustements has been made to ref_tracker, details in patches.
> I hope changes are OK for original author.
>
> The patchset has been rebased on top of drm-tip to allow test changes by CI.
> Patches marked "[DO NOT MERGE]" are cherry-picked from linux-next (they are
> not yet in drm-tip), to allow build and run CI on the patchset (it works only
> on drm-tip tree).
>
> Added CC to netdev as the only user of the library atm.
>
> v2:
>    - replaced list_sort with ref_tracker_dir_stats, to avoid potentially
>      extensive sorting, if number of reports is expected to be big enough (???)
>      we can replace linear search in ref_tracker_dir_stats.stacks with binary
>      heap (min_heap),
>    - refactored gfp flags,
>    - fixed i915 handling of no-tracking flag.
>
> Regards
> Andrzej

Sorry for the mess, sth wrong happened to my scripts and I've messed 
patches, I will resend it properly.

Regards
Andrzej