lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Fri, 5 Aug 2022 18:07:11 +0200 From: Daniel Lezcano <daniel.lezcano@...aro.org> To: Vadim Pasternak <vadimp@...dia.com>, "rafael@...nel.org" <rafael@...nel.org> Cc: "davem@...emloft.net" <davem@...emloft.net>, "netdev@...r.kernel.org" <netdev@...r.kernel.org>, "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, Ido Schimmel <idosch@...dia.com>, Petr Machata <petrm@...dia.com>, Eric Dumazet <edumazet@...gle.com>, Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com> Subject: Re: [PATCH 2/2] Revert "mlxsw: core: Add the hottest thermal zone detection" Hi Vadim, On 04/08/2022 14:21, Vadim Pasternak wrote: > > >> -----Original Message----- >> From: Daniel Lezcano <daniel.lezcano@...aro.org> >> Sent: Monday, August 1, 2022 12:56 PM >> To: daniel.lezcano@...aro.org; rafael@...nel.org >> Cc: Vadim Pasternak <vadimp@...dia.com>; davem@...emloft.net; >> netdev@...r.kernel.org; linux-kernel@...r.kernel.org; Ido Schimmel >> <idosch@...dia.com>; Petr Machata <petrm@...dia.com>; Eric Dumazet >> <edumazet@...gle.com>; Jakub Kicinski <kuba@...nel.org>; Paolo Abeni >> <pabeni@...hat.com> >> Subject: [PATCH 2/2] Revert "mlxsw: core: Add the hottest thermal zone >> detection" >> >> This reverts commit 6f73862fabd93213de157d9cc6ef76084311c628. >> >> As discussed in the thread: >> >> https://lore.kernel.org/all/f3c62ebe-7d59-c537-a010- >> bff366c8aeba@...aro.org/ >> >> the feature provided by commits 2dc2f760052da and 6f73862fabd93 is >> actually already handled by the thermal framework via the cooling device >> state aggregation, thus all this code is pointless. >> >> The revert conflicts with the following changes: >> - 7f4957be0d5b8: thermal: Use mode helpers in drivers >> - 6a79507cfe94c: mlxsw: core: Extend thermal module with per QSFP module >> thermal zones >> >> These conflicts were fixed and the resulting changes are in this patch. >> >> Signed-off-by: Daniel Lezcano <daniel.lezcano@...aro.org> > Tested-by: Vadim Pasternak <vadimp@...dia.com> Thanks for testing > Daniel, > Could you, please, re-base the patch on top of net-next as Jakub mentioned? > Or do you want me to do it? It is fine, I can do it. The conflict is trivial. However, I would have preferred to have the patch in my tree so I can continue the consolidation work. Is it ok if I pick the patch and the conflict being simple, that can be handle at merge time, no? > There is also redundant blank line in this patch: > &mlxsw_thermal_module_ops, > + > &mlxsw_thermal_params, Yeah, thanks. -- <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | <http://twitter.com/#!/linaroorg> Twitter | <http://www.linaro.org/linaro-blog/> Blog
Powered by blists - more mailing lists