linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [bug report] dm: 'tried to init an initialized object' error triggered during device creation
@ 2025-07-30 12:00 Li Lingfeng
  2025-07-31  1:28 ` Hou Tao
  0 siblings, 1 reply; 3+ messages in thread
From: Li Lingfeng @ 2025-07-30 12:00 UTC (permalink / raw)
  To: agk, Mike Snitzer, Mikulas Patocka, dm-devel,
	linux-kernel@vger.kernel.org
  Cc: zhengqixing, yangerkun, zhangyi (F), Hou Tao, yukuai (C),
	chengzhihao1@huawei.com, Li Lingfeng

Hi all,

Recently, when creating dm-mpath devices based on iSCSI devices, we
encountered a 'tried to init an initialized object' error triggered
by one DM_DEV_CREATE and two DM_TABLE_LOAD operations.

// DM_DEV_CREATE
dev_create
  dm_create
   alloc_dev
    blk_alloc_disk // alloc gendisk for md->disk

// first DM_TABLE_LOAD
table_load
  dm_get_md_type // md->type is DM_TYPE_NONE
  dm_setup_md_queue
   dm_mq_init_request_queue
    add_disk // add md->disk
     device_add_disk
      add_disk_fwnode
       __add_disk
        blk_register_queue
         kobject_init
          kobject_init_internal
           // set kobj->state_initialized
  bd_link_disk_holder
   // return -ENODEV since iscsi device has been logged out
  // skip setting md->type and leave it as DM_TYPE_NONE

// second DM_TABLE_LOAD
table_load
  dm_get_md_type // md->type is DM_TYPE_NONE
  dm_setup_md_queue
   dm_mq_init_request_queue
    add_disk // add md->disk
     device_add_disk
      add_disk_fwnode
       __add_disk
        blk_register_queue
         kobject_init
          // dump_stack since kobj->state_initialized has been set

Resetting state_initialized after the first failed DM_TABLE_LOAD might
resolve this issue, but it seems weird — I think state_initialized is
designed to ​never be reset during the object lifecycle.

Any suggestions would be appreciated.

Thanks,
Lingfeng

*Reproduction steps are as follows:*
Base:
master   4b290aae788e

Diff:
diff --git a/drivers/md/dm.c b/drivers/md/dm.c
index abfe0392b5a4..b41f5a6f3e72 100644
--- a/drivers/md/dm.c
+++ b/drivers/md/dm.c
@@ -2598,6 +2598,9 @@ int dm_setup_md_queue(struct mapped_device *md, 
struct dm_table *t)
         if (r)
                 return r;

+       printk("%s sleep before link disk holder\n", __func__);
+       msleep(10 * 1000);
+       printk("%s sleep done\n", __func__);
         /*
          * Register the holder relationship for devices added before 
the disk
          * was live.

Procedures:
[root@nfs_test3 test]# lsblk
NAME MAJ:MIN RM SIZE RO TYPE MOUNTPOINT
sda    8:0    0  10G  0 disk
sdb    8:16   0  10G  0 disk
sdc    8:32   0  30G  0 disk
vda  253:0    0  20G  0 disk /
[root@nfs_test3 test]# ifconfig lo:0 127.0.0.2 up
[root@nfs_test3 test]# ifconfig lo:1 127.0.0.3 up
[root@nfs_test3 test]# service tgtd restart
Redirecting to /bin/systemctl restart tgtd.service
[root@nfs_test3 test]# service iscsid restart
Redirecting to /bin/systemctl restart iscsid.service
[root@nfs_test3 test]# iscsiadm -m discovery -p 127.0.0.2 -t st
127.0.0.2:3260,1 iqn.2019-04.jenkins.disk
[root@nfs_test3 test]# iscsiadm -m discovery -p 127.0.0.3 -t st
127.0.0.3:3260,1 iqn.2019-04.jenkins.disk
[root@nfs_test3 test]# iscsiadm -m node -l
Logging in to [iface: default, target: iqn.2020-05.com.bdqn:disk1, 
portal: 192.168.240.250,3260] (multiple)
Logging in to [iface: default, target: iqn.2019-04.jenkins.disk, portal: 
127.0.0.2,3260] (multiple)
Logging in to [iface: default, target: iqn.2019-04.jenkins.disk, portal: 
127.0.0.3,3260] (multiple)
Logging in to [iface: default, target: , portal: ,3260] (multiple)
[  275.687170][   C10] sd 3:0:0:1: Power-on or device reset occurred
[  275.708911][   C10] sd 4:0:0:1: Power-on or device reset occurred
iscsiadm: caught SIGINT, exiting...
[root@nfs_test3 test]# lsblk
NAME MAJ:MIN RM SIZE RO TYPE MOUNTPOINT
sda    8:0    0  10G  0 disk
sdb    8:16   0  10G  0 disk
sdc    8:32   0  30G  0 disk
sdd    8:48   0  10G  0 disk
sde    8:64   0  10G  0 disk
vda  253:0    0  20G  0 disk /
[root@nfs_test3 test]# ./dmtest &
[1] 2742
[root@nfs_test3 test]# DM_VERSION 4 50 0
[  294.101770][ T2742] dm_setup_md_queue sleep before link disk holder
[root@nfs_test3 test]#
[root@nfs_test3 test]# iscsiadm -m node -u
Logging out of session [sid: 1, target: iqn.2019-04.jenkins.disk, 
portal: 127.0.0.2,3260]
Logging out of session [sid: 2, target: iqn.2019-04.jenkins.disk, 
portal: 127.0.0.3,3260]
Logout of [sid: 1, target: iqn.2019-04.jenkins.disk, portal: 
127.0.0.2,3260] successful.
Logout of [sid: 2, target: iqn.2019-04.jenkins.disk, portal: 
127.0.0.3,3260] successful.
[root@nfs_test3 test]#
[root@nfs_test3 test]#
[root@nfs_test3 test]# [  304.185384][ T2742] dm_setup_md_queue sleep done
[  304.280823][ T2742] device-mapper: ioctl: unable to set up device 
queue for new table.
delay before second table_load...
[root@nfs_test3 test]#
[root@nfs_test3 test]# iscsiadm -m node -l
Logging in to [iface: default, target: iqn.2020-05.com.bdqn:disk1, 
portal: 192.168.240.250,3260] (multiple)
Logging in to [iface: default, target: iqn.2019-04.jenkins.disk, portal: 
127.0.0.2,3260] (multiple)
Logging in to [iface: default, target: iqn.2019-04.jenkins.disk, portal: 
127.0.0.3,3260] (multiple)
iscsiadm: could not read session targetname: 5
iscsiadm: could not find session info for session3
Logging in to [iface: default, target: , portal: ,3260] (multiple)
[  307.357847][   C10] sd 3:0:0:1: Power-on or device reset occurred
[  307.372301][   C10] sd 4:0:0:1: Power-on or device reset occurred
iscsiadm: caught SIGINT, exiting...
[root@nfs_test3 test]#
[root@nfs_test3 test]#
[root@nfs_test3 test]# lsblk
NAME MAJ:MIN RM SIZE RO TYPE MOUNTPOINT
sda    8:0    0  10G  0 disk
sdb    8:16   0  10G  0 disk
sdc    8:32   0  30G  0 disk
sdd    8:48   0  10G  0 disk
sde    8:64   0  10G  0 disk
vda  253:0    0  20G  0 disk /
[root@nfs_test3 test]# delay done
[  314.938868][ T2742] kobject: kobject (ffff88810c27bb90): tried to 
init an initialized object, something is seriously wrong.
[  314.940322][ T2742] CPU: 6 UID: 0 PID: 2742 Comm: dmtest Not tainted 
6.16.0-04405-g4b290aae788e-dirty #21 PREEMPT(none)
[  314.940331][ T2742] Hardware name: QEMU Standard PC (i440FX + PIIX, 
1996), BIOS 1.16.3-2.fc40 04/01/2014
[  314.940335][ T2742] Call Trace:
[  314.940339][ T2742]  <TASK>
[  314.940345][ T2742]  dump_stack_lvl+0x5b/0x80
[  314.940370][ T2742]  kobject_init.cold+0x43/0x51
[  314.940380][ T2742]  blk_register_queue+0x46/0x280
[  314.940399][ T2742]  __add_disk+0x385/0x5d0
[  314.940413][ T2742]  add_disk_fwnode+0xb5/0x280
[  314.940425][ T2742]  dm_setup_md_queue+0x194/0x1c0
[  314.940434][ T2742]  ? mempool_init_noprof+0x16/0x20
[  314.940443][ T2742]  ? __pfx_dm_setup_md_queue+0x10/0x10
[  314.940476][ T2742]  table_load+0x297/0x2d0
[  314.940484][ T2742]  ? __pfx_table_load+0x10/0x10
[  314.940489][ T2742]  ? _inline_copy_from_user+0x6c/0x90
[  314.940499][ T2742]  ? copy_params+0xa6/0x330
[  314.940513][ T2742]  ctl_ioctl+0x2a2/0x480
[  314.940520][ T2742]  ? __pfx_table_load+0x10/0x10
[  314.940528][ T2742]  ? __pfx_ctl_ioctl+0x10/0x10
[  314.940564][ T2742]  ? __free_zapped_classes+0x70/0x120
[  314.940572][ T2742]  ? exc_page_fault+0x61/0xa0
[  314.940591][ T2742]  dm_ctl_ioctl+0xe/0x20
[  314.940598][ T2742]  __x64_sys_ioctl+0xc7/0x110
[  314.940605][ T2742]  ? rcu_is_watching+0x20/0x50
[  314.940616][ T2742]  do_syscall_64+0x72/0x390
[  314.940624][ T2742]  entry_SYSCALL_64_after_hwframe+0x76/0x7e
[  314.940631][ T2742] RIP: 0033:0x7f8ce1106577
[  314.940639][ T2742] Code: b3 66 90 48 8b 05 11 89 2c 00 64 c7 00 26 
00 00 00 48 c7 c0 ff ff ff ff c3 66 2e 0f 1f 84 00 00 00 00 00 b8 10 00 
00 00 0f 05 <48> 3d 01 f0 ff ff 73 018
[  314.940645][ T2742] RSP: 002b:00007fff58708438 EFLAGS: 00000202 
ORIG_RAX: 0000000000000010
[  314.940652][ T2742] RAX: ffffffffffffffda RBX: 0000000000000000 RCX: 
00007f8ce1106577
[  314.940657][ T2742] RDX: 0000000008355010 RSI: 00000000c138fd09 RDI: 
0000000000000003
[  314.940661][ T2742] RBP: 00007fff587084d0 R08: 0000000000000000 R09: 
0000000000000000
[  314.940665][ T2742] R10: 0000000000400e00 R11: 0000000000000202 R12: 
3120312030206874
[  314.940669][ T2742] R13: 6563697672657320 R14: 5f65756575712031 R15: 
61705f6f6e5f6669
[  314.940686][ T2742]  </TASK>
[  315.026828][ T2742] dm_setup_md_queue sleep before link disk holder
[root@nfs_test3 test]# [  325.177205][ T2742] dm_setup_md_queue sleep done


dmtest.c
#include <stdio.h>
#include <stdlib.h>
#include <fcntl.h>
#include <unistd.h>
#include <string.h>
#include <errno.h>
#include <linux/dm-ioctl.h>
#include <sys/ioctl.h>

#define DM_EXISTS_FLAG 0x00000004

int main(int argc, char *argv[]) {
     int ret, fd, index;
     int in_data;

     struct dm_ioctl *param = calloc(1, sizeof(struct dm_ioctl) + 16384);
     struct dm_target_spec sp;
     char *target = (char *)(param + 1);
     char *last_param = target + sizeof(struct dm_target_spec);

     struct dm_target_msg tmsg;
     fd = open("/dev/mapper/control", O_RDWR);
     if (fd < 0) {
         printf("Failed to open device /dev/mapper/control: %s\n", 
strerror(errno));
         return -1;
     }

     // DM_VERSION
     param->version[0] = 4;
     param->version[1] = 0;
     param->version[2] = 0;
     param->data_size = 16384;
     param->flags = DM_EXISTS_FLAG;

     ret = ioctl(fd, DM_VERSION, param);
     if (ret < 0) {
         printf("Failed to send DM_VERSION: %s\n", strerror(errno));
         close(fd);
         return -1;
     }
     printf("DM_VERSION %d %d %d\n", param->version[0], 
param->version[1], param->version[2]);

     // DM_DEV_CREATE
     param->data_start = 312;
     strncpy(param->name, "multipath-dev", sizeof(param->name));

     // clear
     memset(target, 0, 16384);

     param->data_size = 16384;
     param->dev = 0;

     ret = ioctl(fd, DM_DEV_CREATE, param);
     if (ret < 0) {
         printf("Failed to send DM_DEV_CREATE: %s\n", strerror(errno));
         close(fd);
         return -1;
     }

     // DM_TABLE_LOAD
     memset(param->name, 0, sizeof(param->name));
     param->flags = DM_EXISTS_FLAG;
     param->data_start = 312;
     param->data_size = 16384;
     //param->dev = 64514; // makedev(252, 2)
     param->target_count = 1;

     sp.sector_start = 0;
     sp.length = 20971520;
     strncpy(sp.target_type, "multipath", sizeof(sp.target_type) - 1);
     sp.target_type[sizeof(sp.target_type) - 1] = '\0';

     memcpy(target, &sp, sizeof(struct dm_target_spec));
     strcpy(last_param, "1 queue_if_no_path 0 1 1 service-time 0 1 1 
8:48 1");

     ret = ioctl(fd, DM_TABLE_LOAD, param);
//    if (ret < 0) {
//        printf("Failed to send DM_TABLE_LOAD: %s\n", strerror(errno));
//        close(fd);
//        return -1;
//    }

     printf("delay before second table_load...\n");
     sleep(10);
     printf("delay done\n");
     memset(param->name, 0, sizeof(param->name));
     param->flags = DM_EXISTS_FLAG;
     param->data_start = 312;
     param->data_size = 16384;
     param->target_count = 1;

     sp.sector_start = 0;
     sp.length = 20971520;
     strncpy(sp.target_type, "multipath", sizeof(sp.target_type) - 1);
     sp.target_type[sizeof(sp.target_type) - 1] = '\0';

     memcpy(target, &sp, sizeof(struct dm_target_spec));
     strcpy(last_param, "1 queue_if_no_path 0 1 1 service-time 0 1 1 
8:48 1");

     ret = ioctl(fd, DM_TABLE_LOAD, param);
     if (ret < 0) {
         printf("Failed to send DM_TABLE_LOAD: %s\n", strerror(errno));
         close(fd);
         return -1;
     }

     // DM_DEV_SUSPEND
     param->flags = DM_EXISTS_FLAG; // continue creating
     param->event_nr = 6345265;
     param->data_size = 16384;
     param->dev = 0;
     strncpy(param->name, "multipath-dev", sizeof(param->name));

     ret = ioctl(fd, DM_DEV_SUSPEND, param);
     if (ret < 0) {
         printf("Failed to send DM_DEV_SUSPEND: %s\n", strerror(errno));
         close(fd);
         return -1;
     }
     printf("DM_DEV_SUSPEND sent successfully\n");
     close(fd);

     return 0;
}



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [bug report] dm: 'tried to init an initialized object' error triggered during device creation
  2025-07-30 12:00 [bug report] dm: 'tried to init an initialized object' error triggered during device creation Li Lingfeng
@ 2025-07-31  1:28 ` Hou Tao
  2025-07-31  1:49   ` Yu Kuai
  0 siblings, 1 reply; 3+ messages in thread
From: Hou Tao @ 2025-07-31  1:28 UTC (permalink / raw)
  To: Li Lingfeng, agk, Mike Snitzer, Mikulas Patocka, dm-devel,
	linux-kernel@vger.kernel.org, linux-block
  Cc: zhengqixing, yangerkun, zhangyi (F), yukuai (C),
	chengzhihao1@huawei.com, Li Lingfeng



On 7/30/2025 8:00 PM, Li Lingfeng wrote:
> Hi all,
>
> Recently, when creating dm-mpath devices based on iSCSI devices, we
> encountered a 'tried to init an initialized object' error triggered
> by one DM_DEV_CREATE and two DM_TABLE_LOAD operations.
>
> // DM_DEV_CREATE
> dev_create
>  dm_create
>   alloc_dev
>    blk_alloc_disk // alloc gendisk for md->disk
>
> // first DM_TABLE_LOAD
> table_load
>  dm_get_md_type // md->type is DM_TYPE_NONE
>  dm_setup_md_queue
>   dm_mq_init_request_queue
>    add_disk // add md->disk
>     device_add_disk
>      add_disk_fwnode
>       __add_disk
>        blk_register_queue
>         kobject_init
>          kobject_init_internal
>           // set kobj->state_initialized
>  bd_link_disk_holder
>   // return -ENODEV since iscsi device has been logged out
>  // skip setting md->type and leave it as DM_TYPE_NONE
>
> // second DM_TABLE_LOAD
> table_load
>  dm_get_md_type // md->type is DM_TYPE_NONE
>  dm_setup_md_queue
>   dm_mq_init_request_queue
>    add_disk // add md->disk
>     device_add_disk
>      add_disk_fwnode
>       __add_disk
>        blk_register_queue
>         kobject_init
>          // dump_stack since kobj->state_initialized has been set
>
> Resetting state_initialized after the first failed DM_TABLE_LOAD might
> resolve this issue, but it seems weird — I think state_initialized is
> designed to ​never be reset during the object lifecycle.
>

cc +linux-block

The disk->queue_kobj is initialized in add_disk() and is uninitialized
in del_disk(). And it seems that blk_unregister_queue() in del_disk()
doesn't uninitialize the queue_kobj completely, because it doesn't
expect the queue_kobj will be added again. I think the right place to
fix the problem is blk_unregister_queue(). How about just memset the
queue_kobj as zero after deleting the queue_kobj in blk_unregister_queue() ?

> Any suggestions would be appreciated.
>
> Thanks,
> Lingfeng
>
> *Reproduction steps are as follows:*
> Base:
> master   4b290aae788e
>
> Diff:
> diff --git a/drivers/md/dm.c b/drivers/md/dm.c
> index abfe0392b5a4..b41f5a6f3e72 100644
> --- a/drivers/md/dm.c
> +++ b/drivers/md/dm.c
> @@ -2598,6 +2598,9 @@ int dm_setup_md_queue(struct mapped_device *md,
> struct dm_table *t)
>         if (r)
>                 return r;
>
> +       printk("%s sleep before link disk holder\n", __func__);
> +       msleep(10 * 1000);
> +       printk("%s sleep done\n", __func__);
>         /*
>          * Register the holder relationship for devices added before
> the disk
>          * was live.
>
> Procedures:
> [root@nfs_test3 test]# lsblk
> NAME MAJ:MIN RM SIZE RO TYPE MOUNTPOINT
> sda    8:0    0  10G  0 disk
> sdb    8:16   0  10G  0 disk
> sdc    8:32   0  30G  0 disk
> vda  253:0    0  20G  0 disk /
> [root@nfs_test3 test]# ifconfig lo:0 127.0.0.2 up
> [root@nfs_test3 test]# ifconfig lo:1 127.0.0.3 up
> [root@nfs_test3 test]# service tgtd restart
> Redirecting to /bin/systemctl restart tgtd.service
> [root@nfs_test3 test]# service iscsid restart
> Redirecting to /bin/systemctl restart iscsid.service
> [root@nfs_test3 test]# iscsiadm -m discovery -p 127.0.0.2 -t st
> 127.0.0.2:3260,1 iqn.2019-04.jenkins.disk
> [root@nfs_test3 test]# iscsiadm -m discovery -p 127.0.0.3 -t st
> 127.0.0.3:3260,1 iqn.2019-04.jenkins.disk
> [root@nfs_test3 test]# iscsiadm -m node -l
> Logging in to [iface: default, target: iqn.2020-05.com.bdqn:disk1,
> portal: 192.168.240.250,3260] (multiple)
> Logging in to [iface: default, target: iqn.2019-04.jenkins.disk,
> portal: 127.0.0.2,3260] (multiple)
> Logging in to [iface: default, target: iqn.2019-04.jenkins.disk,
> portal: 127.0.0.3,3260] (multiple)
> Logging in to [iface: default, target: , portal: ,3260] (multiple)
> [  275.687170][   C10] sd 3:0:0:1: Power-on or device reset occurred
> [  275.708911][   C10] sd 4:0:0:1: Power-on or device reset occurred
> iscsiadm: caught SIGINT, exiting...
> [root@nfs_test3 test]# lsblk
> NAME MAJ:MIN RM SIZE RO TYPE MOUNTPOINT
> sda    8:0    0  10G  0 disk
> sdb    8:16   0  10G  0 disk
> sdc    8:32   0  30G  0 disk
> sdd    8:48   0  10G  0 disk
> sde    8:64   0  10G  0 disk
> vda  253:0    0  20G  0 disk /
> [root@nfs_test3 test]# ./dmtest &
> [1] 2742
> [root@nfs_test3 test]# DM_VERSION 4 50 0
> [  294.101770][ T2742] dm_setup_md_queue sleep before link disk holder
> [root@nfs_test3 test]#
> [root@nfs_test3 test]# iscsiadm -m node -u
> Logging out of session [sid: 1, target: iqn.2019-04.jenkins.disk,
> portal: 127.0.0.2,3260]
> Logging out of session [sid: 2, target: iqn.2019-04.jenkins.disk,
> portal: 127.0.0.3,3260]
> Logout of [sid: 1, target: iqn.2019-04.jenkins.disk, portal:
> 127.0.0.2,3260] successful.
> Logout of [sid: 2, target: iqn.2019-04.jenkins.disk, portal:
> 127.0.0.3,3260] successful.
> [root@nfs_test3 test]#
> [root@nfs_test3 test]#
> [root@nfs_test3 test]# [  304.185384][ T2742] dm_setup_md_queue sleep
> done
> [  304.280823][ T2742] device-mapper: ioctl: unable to set up device
> queue for new table.
> delay before second table_load...
> [root@nfs_test3 test]#
> [root@nfs_test3 test]# iscsiadm -m node -l
> Logging in to [iface: default, target: iqn.2020-05.com.bdqn:disk1,
> portal: 192.168.240.250,3260] (multiple)
> Logging in to [iface: default, target: iqn.2019-04.jenkins.disk,
> portal: 127.0.0.2,3260] (multiple)
> Logging in to [iface: default, target: iqn.2019-04.jenkins.disk,
> portal: 127.0.0.3,3260] (multiple)
> iscsiadm: could not read session targetname: 5
> iscsiadm: could not find session info for session3
> Logging in to [iface: default, target: , portal: ,3260] (multiple)
> [  307.357847][   C10] sd 3:0:0:1: Power-on or device reset occurred
> [  307.372301][   C10] sd 4:0:0:1: Power-on or device reset occurred
> iscsiadm: caught SIGINT, exiting...
> [root@nfs_test3 test]#
> [root@nfs_test3 test]#
> [root@nfs_test3 test]# lsblk
> NAME MAJ:MIN RM SIZE RO TYPE MOUNTPOINT
> sda    8:0    0  10G  0 disk
> sdb    8:16   0  10G  0 disk
> sdc    8:32   0  30G  0 disk
> sdd    8:48   0  10G  0 disk
> sde    8:64   0  10G  0 disk
> vda  253:0    0  20G  0 disk /
> [root@nfs_test3 test]# delay done
> [  314.938868][ T2742] kobject: kobject (ffff88810c27bb90): tried to
> init an initialized object, something is seriously wrong.
> [  314.940322][ T2742] CPU: 6 UID: 0 PID: 2742 Comm: dmtest Not
> tainted 6.16.0-04405-g4b290aae788e-dirty #21 PREEMPT(none)
> [  314.940331][ T2742] Hardware name: QEMU Standard PC (i440FX + PIIX,
> 1996), BIOS 1.16.3-2.fc40 04/01/2014
> [  314.940335][ T2742] Call Trace:
> [  314.940339][ T2742]  <TASK>
> [  314.940345][ T2742]  dump_stack_lvl+0x5b/0x80
> [  314.940370][ T2742]  kobject_init.cold+0x43/0x51
> [  314.940380][ T2742]  blk_register_queue+0x46/0x280
> [  314.940399][ T2742]  __add_disk+0x385/0x5d0
> [  314.940413][ T2742]  add_disk_fwnode+0xb5/0x280
> [  314.940425][ T2742]  dm_setup_md_queue+0x194/0x1c0
> [  314.940434][ T2742]  ? mempool_init_noprof+0x16/0x20
> [  314.940443][ T2742]  ? __pfx_dm_setup_md_queue+0x10/0x10
> [  314.940476][ T2742]  table_load+0x297/0x2d0
> [  314.940484][ T2742]  ? __pfx_table_load+0x10/0x10
> [  314.940489][ T2742]  ? _inline_copy_from_user+0x6c/0x90
> [  314.940499][ T2742]  ? copy_params+0xa6/0x330
> [  314.940513][ T2742]  ctl_ioctl+0x2a2/0x480
> [  314.940520][ T2742]  ? __pfx_table_load+0x10/0x10
> [  314.940528][ T2742]  ? __pfx_ctl_ioctl+0x10/0x10
> [  314.940564][ T2742]  ? __free_zapped_classes+0x70/0x120
> [  314.940572][ T2742]  ? exc_page_fault+0x61/0xa0
> [  314.940591][ T2742]  dm_ctl_ioctl+0xe/0x20
> [  314.940598][ T2742]  __x64_sys_ioctl+0xc7/0x110
> [  314.940605][ T2742]  ? rcu_is_watching+0x20/0x50
> [  314.940616][ T2742]  do_syscall_64+0x72/0x390
> [  314.940624][ T2742]  entry_SYSCALL_64_after_hwframe+0x76/0x7e
> [  314.940631][ T2742] RIP: 0033:0x7f8ce1106577
> [  314.940639][ T2742] Code: b3 66 90 48 8b 05 11 89 2c 00 64 c7 00 26
> 00 00 00 48 c7 c0 ff ff ff ff c3 66 2e 0f 1f 84 00 00 00 00 00 b8 10
> 00 00 00 0f 05 <48> 3d 01 f0 ff ff 73 018
> [  314.940645][ T2742] RSP: 002b:00007fff58708438 EFLAGS: 00000202
> ORIG_RAX: 0000000000000010
> [  314.940652][ T2742] RAX: ffffffffffffffda RBX: 0000000000000000
> RCX: 00007f8ce1106577
> [  314.940657][ T2742] RDX: 0000000008355010 RSI: 00000000c138fd09
> RDI: 0000000000000003
> [  314.940661][ T2742] RBP: 00007fff587084d0 R08: 0000000000000000
> R09: 0000000000000000
> [  314.940665][ T2742] R10: 0000000000400e00 R11: 0000000000000202
> R12: 3120312030206874
> [  314.940669][ T2742] R13: 6563697672657320 R14: 5f65756575712031
> R15: 61705f6f6e5f6669
> [  314.940686][ T2742]  </TASK>
> [  315.026828][ T2742] dm_setup_md_queue sleep before link disk holder
> [root@nfs_test3 test]# [  325.177205][ T2742] dm_setup_md_queue sleep
> done
>
>
> dmtest.c
> #include <stdio.h>
> #include <stdlib.h>
> #include <fcntl.h>
> #include <unistd.h>
> #include <string.h>
> #include <errno.h>
> #include <linux/dm-ioctl.h>
> #include <sys/ioctl.h>
>
> #define DM_EXISTS_FLAG 0x00000004
>
> int main(int argc, char *argv[]) {
>     int ret, fd, index;
>     int in_data;
>
>     struct dm_ioctl *param = calloc(1, sizeof(struct dm_ioctl) + 16384);
>     struct dm_target_spec sp;
>     char *target = (char *)(param + 1);
>     char *last_param = target + sizeof(struct dm_target_spec);
>
>     struct dm_target_msg tmsg;
>     fd = open("/dev/mapper/control", O_RDWR);
>     if (fd < 0) {
>         printf("Failed to open device /dev/mapper/control: %s\n",
> strerror(errno));
>         return -1;
>     }
>
>     // DM_VERSION
>     param->version[0] = 4;
>     param->version[1] = 0;
>     param->version[2] = 0;
>     param->data_size = 16384;
>     param->flags = DM_EXISTS_FLAG;
>
>     ret = ioctl(fd, DM_VERSION, param);
>     if (ret < 0) {
>         printf("Failed to send DM_VERSION: %s\n", strerror(errno));
>         close(fd);
>         return -1;
>     }
>     printf("DM_VERSION %d %d %d\n", param->version[0],
> param->version[1], param->version[2]);
>
>     // DM_DEV_CREATE
>     param->data_start = 312;
>     strncpy(param->name, "multipath-dev", sizeof(param->name));
>
>     // clear
>     memset(target, 0, 16384);
>
>     param->data_size = 16384;
>     param->dev = 0;
>
>     ret = ioctl(fd, DM_DEV_CREATE, param);
>     if (ret < 0) {
>         printf("Failed to send DM_DEV_CREATE: %s\n", strerror(errno));
>         close(fd);
>         return -1;
>     }
>
>     // DM_TABLE_LOAD
>     memset(param->name, 0, sizeof(param->name));
>     param->flags = DM_EXISTS_FLAG;
>     param->data_start = 312;
>     param->data_size = 16384;
>     //param->dev = 64514; // makedev(252, 2)
>     param->target_count = 1;
>
>     sp.sector_start = 0;
>     sp.length = 20971520;
>     strncpy(sp.target_type, "multipath", sizeof(sp.target_type) - 1);
>     sp.target_type[sizeof(sp.target_type) - 1] = '\0';
>
>     memcpy(target, &sp, sizeof(struct dm_target_spec));
>     strcpy(last_param, "1 queue_if_no_path 0 1 1 service-time 0 1 1
> 8:48 1");
>
>     ret = ioctl(fd, DM_TABLE_LOAD, param);
> //    if (ret < 0) {
> //        printf("Failed to send DM_TABLE_LOAD: %s\n", strerror(errno));
> //        close(fd);
> //        return -1;
> //    }
>
>     printf("delay before second table_load...\n");
>     sleep(10);
>     printf("delay done\n");
>     memset(param->name, 0, sizeof(param->name));
>     param->flags = DM_EXISTS_FLAG;
>     param->data_start = 312;
>     param->data_size = 16384;
>     param->target_count = 1;
>
>     sp.sector_start = 0;
>     sp.length = 20971520;
>     strncpy(sp.target_type, "multipath", sizeof(sp.target_type) - 1);
>     sp.target_type[sizeof(sp.target_type) - 1] = '\0';
>
>     memcpy(target, &sp, sizeof(struct dm_target_spec));
>     strcpy(last_param, "1 queue_if_no_path 0 1 1 service-time 0 1 1
> 8:48 1");
>
>     ret = ioctl(fd, DM_TABLE_LOAD, param);
>     if (ret < 0) {
>         printf("Failed to send DM_TABLE_LOAD: %s\n", strerror(errno));
>         close(fd);
>         return -1;
>     }
>
>     // DM_DEV_SUSPEND
>     param->flags = DM_EXISTS_FLAG; // continue creating
>     param->event_nr = 6345265;
>     param->data_size = 16384;
>     param->dev = 0;
>     strncpy(param->name, "multipath-dev", sizeof(param->name));
>
>     ret = ioctl(fd, DM_DEV_SUSPEND, param);
>     if (ret < 0) {
>         printf("Failed to send DM_DEV_SUSPEND: %s\n", strerror(errno));
>         close(fd);
>         return -1;
>     }
>     printf("DM_DEV_SUSPEND sent successfully\n");
>     close(fd);
>
>     return 0;
> }
>
>
> .


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [bug report] dm: 'tried to init an initialized object' error triggered during device creation
  2025-07-31  1:28 ` Hou Tao
@ 2025-07-31  1:49   ` Yu Kuai
  0 siblings, 0 replies; 3+ messages in thread
From: Yu Kuai @ 2025-07-31  1:49 UTC (permalink / raw)
  To: Hou Tao, Li Lingfeng, agk, Mike Snitzer, Mikulas Patocka,
	dm-devel, linux-kernel@vger.kernel.org, linux-block
  Cc: zhengqixing, yangerkun, zhangyi (F), chengzhihao1@huawei.com,
	Li Lingfeng, yukuai (C)

Hi,

在 2025/07/31 9:28, Hou Tao 写道:
> cc +linux-block
> 
> The disk->queue_kobj is initialized in add_disk() and is uninitialized
> in del_disk(). And it seems that blk_unregister_queue() in del_disk()
> doesn't uninitialize the queue_kobj completely, because it doesn't
> expect the queue_kobj will be added again. I think the right place to
> fix the problem is blk_unregister_queue(). How about just memset the
> queue_kobj as zero after deleting the queue_kobj in blk_unregister_queue() ?

I think it's better to move kobject_init to allocate disk:

1) alloc disk, kobject_init
2) register q, kobject_add
3) unregister q, kobject_del
4) free disk, kobject_put

With this, it's safe to add the kobject again.

Thanks,
Kuai


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2025-07-31  1:49 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-07-30 12:00 [bug report] dm: 'tried to init an initialized object' error triggered during device creation Li Lingfeng
2025-07-31  1:28 ` Hou Tao
2025-07-31  1:49   ` Yu Kuai

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).