Re: [RFC PATCH v2 07/16] bcache: add comments for kobj release callback routine

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 4/19/19 6:05 PM, Coly Li wrote:
Bcache has several routines to release resources in implicit way, they
are called when the associated kobj released. This patch adds code
comments to notice when and which release callback will be called,
- When dc->disk.kobj released:
   void bch_cached_dev_release(struct kobject *kobj)
- When d->kobj released:
   void bch_flash_dev_release(struct kobject *kobj)
- When c->kobj released:
   void bch_cache_set_release(struct kobject *kobj)
- When ca->kobj released
   void bch_cache_release(struct kobject *kobj)

Signed-off-by: Coly Li <colyli@xxxxxxx>
---
  drivers/md/bcache/super.c | 4 ++++
  1 file changed, 4 insertions(+)

diff --git a/drivers/md/bcache/super.c b/drivers/md/bcache/super.c
index 036bffad0bfe..400af446c372 100644
--- a/drivers/md/bcache/super.c
+++ b/drivers/md/bcache/super.c
@@ -1174,6 +1174,7 @@ int bch_cached_dev_attach(struct cached_dev *dc, struct cache_set *c,
  	return 0;
  }
+/* when dc->disk.kobj released */
  void bch_cached_dev_release(struct kobject *kobj)
  {
  	struct cached_dev *dc = container_of(kobj, struct cached_dev,
@@ -1326,6 +1327,7 @@ static void register_bdev(struct cache_sb *sb, struct page *sb_page,
/* Flash only volumes */ +/* When d->kobj released */
  void bch_flash_dev_release(struct kobject *kobj)
  {
  	struct bcache_device *d = container_of(kobj, struct bcache_device,
@@ -1496,6 +1498,7 @@ bool bch_cache_set_error(struct cache_set *c, const char *fmt, ...)
  	return true;
  }
+/* When c->kobj released */
  void bch_cache_set_release(struct kobject *kobj)
  {
  	struct cache_set *c = container_of(kobj, struct cache_set, kobj);
@@ -2023,6 +2026,7 @@ static const char *register_cache_set(struct cache *ca)
/* Cache device */ +/* When ca->kobj released */
  void bch_cache_release(struct kobject *kobj)
  {
  	struct cache *ca = container_of(kobj, struct cache, kobj);

This could be inferred from the 'container_of' statement for each relase function, but if you want ...

Reviewed-by: Hannes Reinecke <hare@xxxxxxxx>

Cheers,

Hannes
--
Dr. Hannes Reinecke		   Teamlead Storage & Networking
hare@xxxxxxx			               +49 911 74053 688
SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: Felix Imendörffer, Mary Higgins, Sri Rasiah
HRB 21284 (AG Nürnberg)



[Index of Archives]     [Linux RAID]     [Linux SCSI]     [Linux ATA RAID]     [IDE]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Device Mapper]

  Powered by Linux