Commit 2a8d6abd authored by Randy Dunlap's avatar Randy Dunlap Committed by Greg Kroah-Hartman
Browse files

devcoredump: cleanup some comments 



Correct a spello, remove an extra space between words, and fix
one kernel-doc warning:

drivers/base/devcoredump.c:292: warning: No description found for return value of 'devcd_read_from_sgtable'

Fixes: 52256637 ("devcoredump: add scatterlist support")
Fixes: 01daccf7 ("devcoredump : Serialize devcd_del work")
Signed-off-by: default avatarRandy Dunlap <rdunlap@infradead.org>
Cc: Johannes Berg <johannes@sipsolutions.net>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Rafael J. Wysocki <rafael@kernel.org>
Cc: Aviya Erenfeld <aviya.erenfeld@intel.com>
Cc: Mukesh Ojha <quic_mojha@quicinc.com>
Link: https://lore.kernel.org/r/20241130023554.538820-1-rdunlap@infradead.org


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 7685ad5f

drivers/base/devcoredump.c

+4 −2
Original line number Diff line number Diff line
@@ -186,7 +186,7 @@ static ssize_t disabled_show(const struct class *class, const struct class_attri 
 *             mutex_lock(&devcd->mutex);

 *

 *

 * In the above diagram, It looks like disabled_store() would be racing with parallely

 * In the above diagram, it looks like disabled_store() would be racing with parallelly

 * running devcd_del() and result in memory abort while acquiring devcd->mutex which

 * is called after kfree of devcd memory after dropping its last reference with

 * put_device(). However, this will not happens as fn(dev, data) runs
@@ -285,6 +285,8 @@ static void devcd_free_sgtable(void *data) 
 * @offset: start copy from @offset@ bytes from the head of the data

 *	in the given scatterlist

 * @data_len: the length of the data in the sg_table

 *

 * Returns: the number of bytes copied

 */

static ssize_t devcd_read_from_sgtable(char *buffer, loff_t offset,

				       size_t buf_len, void *data,