----- Original Message -----
> From: "CAI Qian" <caiq...@redhat.com>
> To: "Nicholas Bellinger" <n...@linux-iscsi.org>
> Cc: stable@vger.kernel.org, "Sebastian Andrzej Siewior"
> <bige...@linutronix.de>
> Sent: Friday, January 18, 2013 11:34:29 AM
> Subject: [PATCH] target: Add link_magic for fabric allow_link destination
>
> Hey Nicholas,
>
> This is a back-port patch for stable-3.4.y and stable-3.7.y
> (stable-3.0.y failed
> to apply) that adding dev->dev_link_magic = SE_DEV_LINK_MAGIC
> assignment for both
> normal se_device setup and the special core_dev_setup_virtual_lun0()
> case. Please
> help review it.
>
> Thanks,
> CAI Qian
>
> From 0ff8754981261a80f4b77db2536dfea92c2d4539 Mon Sep 17 00:00:00
> 2001
> From: Nicholas Bellinger <n...@linux-iscsi.org>
> Date: Tue, 4 Dec 2012 23:43:57 -0800
> Subject: [PATCH] target: Add link_magic for fabric allow_link
> destination
> target_items
>
> This patch adds [dev,lun]_link_magic value assignment + checks within
> generic
> target_fabric_port_link() and target_fabric_mappedlun_link() code to
> ensure
> destination config_item *target_item sent from configfs_symlink() ->
> config_item_operations->allow_link() is the underlying
> se_device->dev_group
> and se_lun->lun_group that we expect to symlink.
>
> Reported-by: Sebastian Andrzej Siewior <bige...@linutronix.de>
> Cc: Sebastian Andrzej Siewior <bige...@linutronix.de>
> Cc: stable@vger.kernel.org
> Signed-off-by: Nicholas Bellinger <n...@linux-iscsi.org>
Missing myself :)
Signed-off-by: CAI Qian <caiq...@redhat.com>
>
> diff --git a/drivers/target/target_core_device.c
> b/drivers/target/target_core_device.c
> index 9abef9f..0943ff0 100644
> --- a/drivers/target/target_core_device.c
> +++ b/drivers/target/target_core_device.c
> @@ -1624,6 +1624,7 @@ int core_dev_setup_virtual_lun0(void)
> ret = PTR_ERR(dev);
> goto out;
> }
> + dev->dev_link_magic = SE_DEV_LINK_MAGIC;
> se_dev->se_dev_ptr = dev;
> g_lun0_dev = dev;
>
> diff --git a/drivers/target/target_core_fabric_configfs.c
> b/drivers/target/target_core_fabric_configfs.c
> index bca737b..a55f91a 100644
> --- a/drivers/target/target_core_fabric_configfs.c
> +++ b/drivers/target/target_core_fabric_configfs.c
> @@ -71,6 +71,12 @@ static int target_fabric_mappedlun_link(
> struct se_portal_group *se_tpg;
> struct config_item *nacl_ci, *tpg_ci, *tpg_ci_s, *wwn_ci,
> *wwn_ci_s;
> int ret = 0, lun_access;
> +
> + if (lun->lun_link_magic != SE_LUN_LINK_MAGIC) {
> + pr_err("Bad lun->lun_link_magic, not a valid lun_ci pointer:"
> + " %p to struct lun: %p\n", lun_ci, lun);
> + return -EFAULT;
> + }
> /*
> * Ensure that the source port exists
> */
> @@ -745,6 +751,12 @@ static int target_fabric_port_link(
> struct target_fabric_configfs *tf;
> int ret;
>
> + if (dev->dev_link_magic != SE_DEV_LINK_MAGIC) {
> + pr_err("Bad dev->dev_link_magic, not a valid se_dev_ci pointer:"
> + " %p to struct se_device: %p\n", se_dev_ci, dev);
> + return -EFAULT;
> + }
> +
> tpg_ci = &lun_ci->ci_parent->ci_group->cg_item;
> se_tpg = container_of(to_config_group(tpg_ci),
> struct se_portal_group, tpg_group);
> diff --git a/drivers/target/target_core_tpg.c
> b/drivers/target/target_core_tpg.c
> index a531fe2..4c34665 100644
> --- a/drivers/target/target_core_tpg.c
> +++ b/drivers/target/target_core_tpg.c
> @@ -672,6 +672,7 @@ int core_tpg_register(
> for (i = 0; i < TRANSPORT_MAX_LUNS_PER_TPG; i++) {
> lun = se_tpg->tpg_lun_list[i];
> lun->unpacked_lun = i;
> + lun->lun_link_magic = SE_LUN_LINK_MAGIC;
> lun->lun_status = TRANSPORT_LUN_STATUS_FREE;
> atomic_set(&lun->lun_acl_count, 0);
> init_completion(&lun->lun_shutdown_comp);
> diff --git a/drivers/target/target_core_transport.c
> b/drivers/target/target_core_transport.c
> index a392e05..13fe16c 100644
> --- a/drivers/target/target_core_transport.c
> +++ b/drivers/target/target_core_transport.c
> @@ -1071,6 +1071,7 @@ struct se_device
> *transport_add_device_to_core_hba(
> dev->se_hba = hba;
> dev->se_sub_dev = se_dev;
> dev->transport = transport;
> + dev->dev_link_magic = SE_DEV_LINK_MAGIC;
> INIT_LIST_HEAD(&dev->dev_list);
> INIT_LIST_HEAD(&dev->dev_sep_list);
> INIT_LIST_HEAD(&dev->dev_tmr_list);
> diff --git a/include/target/target_core_base.h
> b/include/target/target_core_base.h
> index 5be8937..fca8bbe 100644
> --- a/include/target/target_core_base.h
> +++ b/include/target/target_core_base.h
> @@ -734,6 +734,8 @@ struct se_subsystem_dev {
> };
>
> struct se_device {
> +#define SE_DEV_LINK_MAGIC 0xfeeddeef
> + u32 dev_link_magic;
> /* RELATIVE TARGET PORT IDENTIFER Counter */
> u16 dev_rpti_counter;
> /* Used for SAM Task Attribute ordering */
> @@ -820,6 +822,8 @@ struct se_port_stat_grps {
> };
>
> struct se_lun {
> +#define SE_LUN_LINK_MAGIC 0xffff7771
> + u32 lun_link_magic;
> /* See transport_lun_status_table */
> enum transport_lun_status_table lun_status;
> u32 lun_access;
> --
> To unsubscribe from this list: send the line "unsubscribe stable" in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html