On 13.05.2022 11:38:31, Vincent Mailhol wrote: > > +static void elm327_parse_line(struct can327 *elm, size_t len) > > +{ > > + lockdep_assert_held(&elm->lock); > > You are using many of those lockdep_assert_held(&elm->lock); > > I guess you put them for debug purpose and probaly some of those can > be removed (if you see a genuine risk at some places, then OK to keep > as a safety net, but a bit of clean up can be done here, I think). > > This comment applies to all use of lockdep_assert_held(). These statements document that the code must be called with the lock held. They optimize away if lockdep is not enabled. Better keep them in place. regards, Marc -- Pengutronix e.K. | Marc Kleine-Budde | Embedded Linux | https://www.pengutronix.de | Vertretung West/Dortmund | Phone: +49-231-2826-924 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
Attachment:
signature.asc
Description: PGP signature