[PATCH] block: export disk_part_iter_* helpers

Subsystems: block layer, the rest

STALE1980d

3 messages, 2 authors, 2021-03-26 · open the first message on its own page

[PATCH] block: export disk_part_iter_* helpers

From: <hidden>
Date: 2021-03-26 12:12:17

From: Yanfei Xu <redacted>

disk_part_iter_* helpers might be used by other external modules, like
lttng-modules. But it was unexport in 'commit bc359d03c7ec ("block: add
a disk_uevent helper")'. Here export them again.

Signed-off-by: Yanfei Xu <redacted>
---
 block/genhd.c | 3 +++
 1 file changed, 3 insertions(+)
diff --git a/block/genhd.c b/block/genhd.c
index 8c8f543572e6..4eac897baffe 100644
--- a/block/genhd.c
+++ b/block/genhd.c
@@ -183,6 +183,7 @@ void disk_part_iter_init(struct disk_part_iter *piter, struct gendisk *disk,
 		piter->idx = 1;
 	piter->flags = flags;
 }
+EXPORT_SYMBOL_GPL(disk_part_iter_init);
 
 /**
  * disk_part_iter_next - proceed iterator to the next partition and return it
@@ -219,6 +220,7 @@ struct block_device *disk_part_iter_next(struct disk_part_iter *piter)
 
 	return piter->part;
 }
+EXPORT_SYMBOL_GPL(disk_part_iter_next);
 
 /**
  * disk_part_iter_exit - finish up partition iteration
@@ -235,6 +237,7 @@ void disk_part_iter_exit(struct disk_part_iter *piter)
 		bdput(piter->part);
 	piter->part = NULL;
 }
+EXPORT_SYMBOL_GPL(disk_part_iter_exit);
 
 /*
  * Can be deleted altogether. Later.
-- 
2.27.0

Re: [PATCH] block: export disk_part_iter_* helpers

From: Christoph Hellwig <hch@lst.de>
Date: 2021-03-26 12:14:57

On Fri, Mar 26, 2021 at 08:10:59PM +0800, yanfei.xu@windriver.com wrote:
From: Yanfei Xu <redacted>

disk_part_iter_* helpers might be used by other external modules, like
lttng-modules. But it was unexport in 'commit bc359d03c7ec ("block: add
a disk_uevent helper")'. Here export them again.
Err, no.  We never export things for out of tree modules.  And any kind
of driver code has absolutely no business looking at the partition tables
to start with, modular or not.

Re: [PATCH] block: export disk_part_iter_* helpers

From: Xu, Yanfei <hidden>
Date: 2021-03-26 13:02:12


On 3/26/21 8:13 PM, Christoph Hellwig wrote:
[Please note: This e-mail is from an EXTERNAL e-mail address]

On Fri, Mar 26, 2021 at 08:10:59PM +0800, yanfei.xu@windriver.com wrote:
quoted
From: Yanfei Xu <redacted>

disk_part_iter_* helpers might be used by other external modules, like
lttng-modules. But it was unexport in 'commit bc359d03c7ec ("block: add
a disk_uevent helper")'. Here export them again.
Err, no.  We never export things for out of tree modules.  And any kind
of driver code has absolutely no business looking at the partition tables
to start with, modular or not.
I see. Thanks.
//Yanfei
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help