Re: [PATCH 25/32] target: Call remove_lun instead of del_lun in fabric_port_unlink

2013-12-16 Thread Nicholas A. Bellinger 
On Fri, 2013-12-13 at 15:59 -0800, Andy Grover wrote:
 We want to be freeing the port here, not freeing the lun.
 
 Signed-off-by: Andy Grover agro...@redhat.com
 ---
  drivers/target/target_core_fabric_configfs.c |2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)
 

Skipping this one as well.

--nab

 diff --git a/drivers/target/target_core_fabric_configfs.c 
 b/drivers/target/target_core_fabric_configfs.c
 index 4715836..fe940d4 100644
 --- a/drivers/target/target_core_fabric_configfs.c
 +++ b/drivers/target/target_core_fabric_configfs.c
 @@ -808,7 +808,7 @@ static int target_fabric_port_unlink(
   tf-tf_ops.fabric_pre_unlink(se_tpg, lun);
   }
  
 - core_dev_del_lun(se_tpg, lun);
 + core_tpg_remove_lun(se_tpg, lun);
   return 0;
  }
  


--
To unsubscribe from this list: send the line unsubscribe linux-scsi in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 25/32] target: Call remove_lun instead of del_lun in fabric_port_unlink

2013-12-13 Thread Andy Grover 
We want to be freeing the port here, not freeing the lun.

Signed-off-by: Andy Grover agro...@redhat.com
---
 drivers/target/target_core_fabric_configfs.c |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/target/target_core_fabric_configfs.c 
b/drivers/target/target_core_fabric_configfs.c
index 4715836..fe940d4 100644
--- a/drivers/target/target_core_fabric_configfs.c
+++ b/drivers/target/target_core_fabric_configfs.c
@@ -808,7 +808,7 @@ static int target_fabric_port_unlink(
tf-tf_ops.fabric_pre_unlink(se_tpg, lun);
}
 
-   core_dev_del_lun(se_tpg, lun);
+   core_tpg_remove_lun(se_tpg, lun);
return 0;
 }
 
-- 
1.7.1

--
To unsubscribe from this list: send the line unsubscribe linux-scsi in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html