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  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Z9fxfV9jAGJ51fcn@shikoro>
Date: Mon, 17 Mar 2025 10:55:09 +0100
From: Wolfram Sang <wsa+renesas@...g-engineering.com>
To: Geert Uytterhoeven <geert@...ux-m68k.org>
Cc: Thierry Bultel <thierry.bultel.yh@...renesas.com>,
	thierry.bultel@...atsea.fr, linux-renesas-soc@...r.kernel.org,
	paul.barker.ct@...renesas.com, linux-kernel@...r.kernel.org,
	linux-serial@...r.kernel.org, Ulrich Hecht <uli@...nd.eu>,
	Linux-sh list <linux-sh@...r.kernel.org>
Subject: Re: [PATCH v4 08/13] serial: sh-sci: Introduced function pointers

Hi all,

sorry for missing this series so far and thanks to Geert for pulling me
into the loop.

> While most rough edges have been polished by now (thanks!), and the
> driver seems to still work on a variety of platforms, I am still
> worried about the impact of this change:
>   - Maintainability and future bug fixing?

I hate to see development work going to waste, yet I have to say I am
also concerned about the maintainability of this driver after this very
intrusive changeset. The driver is already quite complex. Adding another
layer of complexity (function pointers) will make proper bugfixing for
all supported instances quite harder, I'd think.

Has it been discussed to have this as a separate driver? Were there
reasons against it? This is really an open question. Maybe it is
justified to do it like this if we have reasons for it.

Seeing that SCI core needs 800+ lines changed and we still have a
seperate driver with 460 lines driver, I do wonder if copying the logic
from SCI core to a seperate driver would make sense. I am aware that the
core has currently 3500+ lines currently. I'd estimate it would shrink
quite a bit when copying because you won't need to handle all the
differences to other SCI entries.

Again, this is not a request to follow my suggestion, it is an open
question to make sure all paths have been considered.

Thanks and happy hacking,

   Wolfram


Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ