user/sven/linux.git/drivers/base/devres.c, branch v5.15.141Linux Kernel
https://git.stealer.net/cgit.cgi/user/sven/linux.git/atom?h=v5.15.1412021-06-15T15:14:36Zdevres: Enable trace events2021-06-15T15:14:36ZAndy Shevchenkoandriy.shevchenko@linux.intel.com2021-05-17T12:29:46Zurn:sha1:09705dcb63d269000595284b5dd7f5c938d647b9
In some cases the printf() mechanism is too heavy and can't be used.
For example, when debugging a race condition involving devres API.
When CONFIG_DEBUG_DEVRES is enabled I can't reproduce an issue, and
otherwise it's quite visible with a useful information being collected.
Enable trace events for devres part of the driver core.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://lore.kernel.org/r/20210517122946.53161-4-andriy.shevchenko@linux.intel.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
devres: No need to call remove_nodes() when there none present2021-06-15T15:14:36ZAndy Shevchenkoandriy.shevchenko@linux.intel.com2021-05-17T12:29:45Zurn:sha1:a7f1d03b6046cf44f1dd702aeaad3b5e4d7b33a5
If a list of devres nodes is empty, no need to call remove_nodes().
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://lore.kernel.org/r/20210517122946.53161-3-andriy.shevchenko@linux.intel.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
devres: Use list_for_each_safe_from() in remove_nodes()2021-06-15T15:14:36ZAndy Shevchenkoandriy.shevchenko@linux.intel.com2021-05-17T12:29:44Zurn:sha1:c3cd0ff7aa18a60229134fb8e467d5e1d92abec3
The remove_nodes() open codes the list_for_each_safe_from().
Replace it by a generic macro.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://lore.kernel.org/r/20210517122946.53161-2-andriy.shevchenko@linux.intel.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
devres: Make locking straight forward in release_nodes()2021-06-15T15:14:36ZAndy Shevchenkoandriy.shevchenko@linux.intel.com2021-05-17T12:29:43Zurn:sha1:bbc8f3e79e9e35469ac87b0b3329729afc715885
It seems for the sake of saving stack memory of couple of pointers,
the locking in release_nodes() callers becomes interesting.
Replace this logic with a straight forward locking and unlocking scheme.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://lore.kernel.org/r/20210517122946.53161-1-andriy.shevchenko@linux.intel.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
driver core: Replace printf() specifier and drop unneeded casting2021-04-02T15:02:45ZAndy Shevchenkoandriy.shevchenko@linux.intel.com2021-04-01T17:10:42Zurn:sha1:318c3e00f13c2f6e11202a22cc302ea8c70552ea
The size_t type has very well established specifier, i.e. "%zu",
use it directly instead of casting to unsigned long with "%lu".
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://lore.kernel.org/r/20210401171042.60612-1-andriy.shevchenko@linux.intel.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
driver core: Cast to (void *) with __force for __percpu pointer2021-04-02T15:02:39ZAndy Shevchenkoandriy.shevchenko@linux.intel.com2021-04-01T17:10:30Zurn:sha1:d7aa44f5a1f86cb40659eef06035d8d92604b9d5
Sparse is not happy:
drivers/base/devres.c:1230:9: warning: cast removes address space '__percpu' of expression
Use __force attribute to make it happy.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://lore.kernel.org/r/20210401171030.60527-1-andriy.shevchenko@linux.intel.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers: base: fix some kernel-doc markups2020-11-09T17:56:49ZMauro Carvalho Chehabmchehab+huawei@kernel.org2020-10-23T16:32:56Zurn:sha1:33c0c9bdf7a59051a654cd98b7d2b48ce0080967
class_create is actually defined at the header. Fix the
markup there and add a new one at the right place.
While here, also fix some markups for functions that have
different names between their prototypes and kernel-doc
comments.
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
Link: https://lore.kernel.org/r/2fb6efd6a1f90d69ff73bf579566079cbb051e15.1603469755.git.mchehab+huawei@kernel.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
devres: provide devm_krealloc()2020-09-08T11:32:06ZBartosz Golaszewskibgolaszewski@baylibre.com2020-08-24T17:38:57Zurn:sha1:f82485722e5de5ebb08d3a1dd7302203346dbff9
Implement the managed variant of krealloc(). This function works with
all memory allocated by devm_kmalloc() (or devres functions using it
implicitly like devm_kmemdup(), devm_kstrdup() etc.).
Managed realloc'ed chunks can be manually released with devm_kfree().
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://lore.kernel.org/r/20200824173859.4910-2-brgl@bgdev.pl
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
devres: handle zero size in devm_kmalloc()2020-07-02T12:36:02ZBartosz Golaszewskibgolaszewski@baylibre.com2020-06-29T06:50:06Zurn:sha1:cad064f1bd5237481d15b72b481809e3f0907dd9
Make devm_kmalloc() behave similarly to non-managed kmalloc(): return
ZERO_SIZE_PTR when requested size is 0. Update devm_kfree() to handle
this case.
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
Link: https://lore.kernel.org/r/20200629065008.27620-5-brgl@bgdev.pl
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
devres: move the size check from alloc_dr() into a separate function2020-07-02T12:36:02ZBartosz Golaszewskibgolaszewski@baylibre.com2020-06-29T06:50:04Zurn:sha1:dc2a633ccb608a1a8109ad0d488c98cc776bef72
We will perform the same size check in devm_krealloc(). Move the relevant
code into a separate helper.
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
Link: https://lore.kernel.org/r/20200629065008.27620-3-brgl@bgdev.pl
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>