2024-02-21 09:18:04

by Jiapeng Chong

[permalink] [raw]
Subject: [PATCH 8/8] dm vdo: Modify mismatched function name

No functional modification involved.

drivers/md/dm-vdo/admin-state.c:310: warning: expecting prototype for vdo_drain_operation(). Prototype was for assert_vdo_drain_operation() instead.

Reported-by: Abaci Robot <[email protected]>
Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=8280
Signed-off-by: Jiapeng Chong <[email protected]>
---
drivers/md/dm-vdo/admin-state.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/md/dm-vdo/admin-state.c b/drivers/md/dm-vdo/admin-state.c
index 8663c1e91616..3f9dba525154 100644
--- a/drivers/md/dm-vdo/admin-state.c
+++ b/drivers/md/dm-vdo/admin-state.c
@@ -300,7 +300,7 @@ static bool check_code(bool valid, const struct admin_state_code *code, const ch
}

/**
- * vdo_drain_operation() - Check that an operation is a drain.
+ * assert_vdo_drain_operation() - Check that an operation is a drain.
* @waiter The completion to finish with an error if the operation is not a drain.
*
* Return: true if the specified operation is a drain.
--
2.20.1.7.g153144c



2024-02-21 09:18:34

by Jiapeng Chong

[permalink] [raw]
Subject: [PATCH 5/8] dm vdo: Modify mismatched function name

No functional modification involved.

drivers/md/dm-vdo/encodings.c:1178: warning: expecting prototype for vdo_decode_component(). Prototype was for decode_vdo_component() instead.

Reported-by: Abaci Robot <[email protected]>
Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=8277
Signed-off-by: Jiapeng Chong <[email protected]>
---
drivers/md/dm-vdo/encodings.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/md/dm-vdo/encodings.c b/drivers/md/dm-vdo/encodings.c
index 42ffd793deeb..f4fee2c1d656 100644
--- a/drivers/md/dm-vdo/encodings.c
+++ b/drivers/md/dm-vdo/encodings.c
@@ -1170,7 +1170,7 @@ static struct vdo_component unpack_vdo_component_41_0(struct packed_vdo_componen
}

/**
- * vdo_decode_component() - Decode the component data for the vdo itself out of the super block.
+ * decode_vdo_component() - Decode the component data for the vdo itself out of the super block.
*
* Return: VDO_SUCCESS or an error.
*/
--
2.20.1.7.g153144c


2024-02-21 09:18:52

by Jiapeng Chong

[permalink] [raw]
Subject: [PATCH 6/8] dm vdo: Modify mismatched function name

No functional modification involved.

drivers/md/dm-vdo/flush.c:97: warning: expecting prototype for waiter_as_flush(). Prototype was for vdo_waiter_as_flush() instead.

Reported-by: Abaci Robot <[email protected]>
Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=8278
Signed-off-by: Jiapeng Chong <[email protected]>
---
drivers/md/dm-vdo/flush.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/md/dm-vdo/flush.c b/drivers/md/dm-vdo/flush.c
index e4be450aa12c..57e87f0d7069 100644
--- a/drivers/md/dm-vdo/flush.c
+++ b/drivers/md/dm-vdo/flush.c
@@ -88,7 +88,7 @@ static inline struct vdo_flush *completion_as_vdo_flush(struct vdo_completion *c
}

/**
- * waiter_as_flush() - Convert a vdo_flush's generic wait queue entry back to the vdo_flush.
+ * vdo_waiter_as_flush() - Convert a vdo_flush's generic wait queue entry back to the vdo_flush.
* @waiter: The wait queue entry to convert.
*
* Return: The wait queue entry as a vdo_flush.
--
2.20.1.7.g153144c


2024-02-21 09:19:28

by Jiapeng Chong

[permalink] [raw]
Subject: [PATCH 3/8] dm vdo: Modify mismatched function name

No functional modification involved.

drivers/md/dm-vdo/physical-zone.c:268: warning: expecting prototype for vdo_free_pbn_lock_pool(). Prototype was for free_pbn_lock_pool() instead.

Reported-by: Abaci Robot <[email protected]>
Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=8275
Signed-off-by: Jiapeng Chong <[email protected]>
---
drivers/md/dm-vdo/physical-zone.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/md/dm-vdo/physical-zone.c b/drivers/md/dm-vdo/physical-zone.c
index b80c4d3a9bde..14d502ea77c9 100644
--- a/drivers/md/dm-vdo/physical-zone.c
+++ b/drivers/md/dm-vdo/physical-zone.c
@@ -258,7 +258,7 @@ static int make_pbn_lock_pool(size_t capacity, struct pbn_lock_pool **pool_ptr)
}

/**
- * vdo_free_pbn_lock_pool() - Free a PBN lock pool.
+ * free_pbn_lock_pool() - Free a PBN lock pool.
* @pool: The lock pool to free.
*
* This also frees all the PBN locks it allocated, so the caller must ensure that all locks have
--
2.20.1.7.g153144c


2024-02-21 09:19:49

by Jiapeng Chong

[permalink] [raw]
Subject: [PATCH 2/8] dm vdo: Modify mismatched function name

No functional modification involved.

drivers/md/dm-vdo/slab-depot.c:2042: warning: expecting prototype for vdo_find_free_block(). Prototype was for find_free_block() instead.
drivers/md/dm-vdo/slab-depot.c:5036: warning: expecting prototype for get_depot_slab_journal_statistics(). Prototype was for get_slab_journal_statistics() instead.

Reported-by: Abaci Robot <[email protected]>
Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=8274
Signed-off-by: Jiapeng Chong <[email protected]>
---
drivers/md/dm-vdo/slab-depot.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/md/dm-vdo/slab-depot.c b/drivers/md/dm-vdo/slab-depot.c
index b11972a8a08b..a85d2983800d 100644
--- a/drivers/md/dm-vdo/slab-depot.c
+++ b/drivers/md/dm-vdo/slab-depot.c
@@ -2029,7 +2029,7 @@ static inline slab_block_number find_zero_byte_in_word(const u8 *word_ptr,
}

/**
- * vdo_find_free_block() - Find the first block with a reference count of zero in the specified
+ * find_free_block() - Find the first block with a reference count of zero in the specified
* range of reference counter indexes.
* @slab: The slab counters to scan.
* @index_ptr: A pointer to hold the array index of the free block.
@@ -5026,7 +5026,7 @@ get_ref_counts_statistics(const struct slab_depot *depot)
}

/**
- * get_depot_slab_journal_statistics() - Get the aggregated slab journal statistics for the depot.
+ * get_slab_journal_statistics() - Get the aggregated slab journal statistics for the depot.
* @depot: The slab depot.
*
* Return: The aggregated statistics for all slab journals in the depot.
--
2.20.1.7.g153144c


2024-02-21 09:23:12

by Jiapeng Chong

[permalink] [raw]
Subject: [PATCH 4/8] dm vdo: Modify mismatched function name

No functional modification involved.

drivers/md/dm-vdo/logical-zone.c:303: warning: expecting prototype for void attempt_generation_complete_notification()(). Prototype was for attempt_generation_complete_notification() instead.

Reported-by: Abaci Robot <[email protected]>
Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=8276
Signed-off-by: Jiapeng Chong <[email protected]>
---
drivers/md/dm-vdo/logical-zone.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/md/dm-vdo/logical-zone.c b/drivers/md/dm-vdo/logical-zone.c
index a040fe9c0779..131161e0c152 100644
--- a/drivers/md/dm-vdo/logical-zone.c
+++ b/drivers/md/dm-vdo/logical-zone.c
@@ -295,7 +295,7 @@ static void notify_flusher(struct vdo_completion *completion)
}

/**
- * void attempt_generation_complete_notification() - Notify the flusher if some generation no
+ * attempt_generation_complete_notification() - Notify the flusher if some generation no
* longer has active VIOs.
* @completion: The zone completion.
*/
--
2.20.1.7.g153144c


2024-02-21 09:25:34

by Jiapeng Chong

[permalink] [raw]
Subject: [PATCH 7/8] dm vdo dedupe: Modify mismatched function name

No functional modification involved.

drivers/md/dm-vdo/dedupe.c:2792: warning: expecting prototype for vdo_dump_hash_zone(). Prototype was for dump_hash_zone() instead.

Reported-by: Abaci Robot <[email protected]>
Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=8279
Signed-off-by: Jiapeng Chong <[email protected]>
---
drivers/md/dm-vdo/dedupe.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/md/dm-vdo/dedupe.c b/drivers/md/dm-vdo/dedupe.c
index a9b189395592..a2535e909a80 100644
--- a/drivers/md/dm-vdo/dedupe.c
+++ b/drivers/md/dm-vdo/dedupe.c
@@ -2785,7 +2785,7 @@ static const char *index_state_to_string(struct hash_zones *zones,
}

/**
- * vdo_dump_hash_zone() - Dump information about a hash zone to the log for debugging.
+ * dump_hash_zone() - Dump information about a hash zone to the log for debugging.
* @zone: The zone to dump.
*/
static void dump_hash_zone(const struct hash_zone *zone)
--
2.20.1.7.g153144c


2024-02-21 09:26:23

by Jiapeng Chong

[permalink] [raw]
Subject: [PATCH 1/8] dm vdo: Modify mismatched function name

No functional modification involved.

drivers/md/dm-vdo/vdo.c:276: warning: expecting prototype for vdo_read_geometry_block(). Prototype was for read_geometry_block() instead.

Reported-by: Abaci Robot <[email protected]>
Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=8273
Signed-off-by: Jiapeng Chong <[email protected]>
---
drivers/md/dm-vdo/vdo.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/md/dm-vdo/vdo.c b/drivers/md/dm-vdo/vdo.c
index 23a4f1d1b6dc..411c58341173 100644
--- a/drivers/md/dm-vdo/vdo.c
+++ b/drivers/md/dm-vdo/vdo.c
@@ -266,7 +266,7 @@ static int __must_check initialize_thread_config(struct thread_count_config coun
}

/**
- * vdo_read_geometry_block() - Synchronously read the geometry block from a vdo's underlying block
+ * read_geometry_block() - Synchronously read the geometry block from a vdo's underlying block
* device.
* @vdo: The vdo whose geometry is to be read.
*
--
2.20.1.7.g153144c


2024-02-22 17:17:19

by Matthew Sakai

[permalink] [raw]
Subject: Re: [PATCH 1/8] dm vdo: Modify mismatched function name



On 2/21/24 04:17, Jiapeng Chong wrote:
> No functional modification involved.
>
> drivers/md/dm-vdo/vdo.c:276: warning: expecting prototype for vdo_read_geometry_block(). Prototype was for read_geometry_block() instead.
>
> Reported-by: Abaci Robot <[email protected]>
> Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=8273
> Signed-off-by: Jiapeng Chong <[email protected]>
> ---
> drivers/md/dm-vdo/vdo.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/md/dm-vdo/vdo.c b/drivers/md/dm-vdo/vdo.c
> index 23a4f1d1b6dc..411c58341173 100644
> --- a/drivers/md/dm-vdo/vdo.c
> +++ b/drivers/md/dm-vdo/vdo.c
> @@ -266,7 +266,7 @@ static int __must_check initialize_thread_config(struct thread_count_config coun
> }
>
> /**
> - * vdo_read_geometry_block() - Synchronously read the geometry block from a vdo's underlying block
> + * read_geometry_block() - Synchronously read the geometry block from a vdo's underlying block
> * device.
> * @vdo: The vdo whose geometry is to be read.
> *

Thanks for the reports! I'll apply these soon.

Matt