From: Andy Grover <agrover@redhat.com>
To: Christoph Hellwig <hch@infradead.org>
Cc: target-devel@vger.kernel.org, linux-scsi@vger.kernel.org
Subject: Re: [PATCH 3/6] target: Change core_dev_del_lun to take a se_lun instead of unpacked_lun
Date: Fri, 07 Mar 2014 10:20:02 -0800 [thread overview]
Message-ID: <531A0DD2.6020905@redhat.com> (raw)
In-Reply-To: <20140307103534.GB14195@infradead.org>
On 03/07/2014 02:35 AM, Christoph Hellwig wrote:
> On Thu, Mar 06, 2014 at 02:15:28PM -0800, Andy Grover wrote:
>> +void core_dev_del_lun(
>> struct se_portal_group *tpg,
>> + struct se_lun *lun)
>> {
>> + pr_debug("%s_TPG[%u]_LUN[%u] - Deactivating %s Logical Unit from"
>> " device object\n", tpg->se_tpg_tfo->get_fabric_name(),
>> + tpg->se_tpg_tfo->tpg_get_tag(tpg), lun->unpacked_lun,
>> tpg->se_tpg_tfo->get_fabric_name());
>>
>> + core_tpg_post_dellun(tpg, lun);
>
> Seems like core_tpg_post_dellun should be folded into core_dev_del_lun?
It gets more stuff added back to it in a later patch, and post_dellun
gets renamed too.
Regards -- Andy
next prev parent reply other threads:[~2014-03-07 18:20 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-03-06 22:15 [PATCH 0/6] target: Save memory on unused se_dev_entrys and se_luns Andy Grover
2014-03-06 22:15 ` [PATCH 1/6] target: Allocate se_dev_entrys in device list only when used Andy Grover
2014-03-07 10:33 ` Christoph Hellwig
2014-03-06 22:15 ` [PATCH 2/6] target: core_tpg_post_dellun can return void Andy Grover
2014-03-06 22:15 ` [PATCH 3/6] target: Change core_dev_del_lun to take a se_lun instead of unpacked_lun Andy Grover
2014-03-07 10:35 ` Christoph Hellwig
2014-03-07 18:20 ` Andy Grover [this message]
2014-03-06 22:15 ` [PATCH 4/6] target: Rename core_tpg_post_dellun to remove_lun Andy Grover
2014-03-06 22:15 ` [PATCH 5/6] target: Allocate se_luns only when used Andy Grover
2014-03-07 10:41 ` Christoph Hellwig
2014-03-07 18:12 ` Andy Grover
2014-03-07 18:22 ` Christoph Hellwig
2014-03-07 18:31 ` Andy Grover
2014-03-12 19:04 ` Nicholas A. Bellinger
2014-03-06 22:15 ` [PATCH 6/6] target: Remove core_tpg_release_virtual_lun0 function Andy Grover
2014-03-07 17:50 ` [PATCH 7/6] target: Do not allocate device_list and tpg_lun_list arrays separately Andy Grover
2014-03-10 22:45 ` [PATCHv2 0/9] target: Save memory on unused se_dev_entrys and se_luns Andy Grover
2014-03-10 22:45 ` [PATCHv2 1/9] target: Add locking to some accesses to nacl.device_list Andy Grover
2014-03-10 22:45 ` [PATCHv2 2/9] target: Don't unlock/relock tpg_lun_lock in loop in add_node_to_devs Andy Grover
2014-03-10 22:45 ` [PATCHv2 3/9] target: Allocate se_dev_entrys in device list only when used Andy Grover
2014-03-10 22:45 ` [PATCHv2 4/9] target: core_tpg_post_dellun can return void Andy Grover
2014-03-10 22:45 ` [PATCHv2 5/9] target: Change core_dev_del_lun to take a se_lun instead of unpacked_lun Andy Grover
2014-03-10 22:45 ` [PATCHv2 6/9] target: Rename core_tpg_post_dellun to remove_lun Andy Grover
2014-03-10 22:45 ` [PATCHv2 7/9] target: Allocate se_luns only when used Andy Grover
2014-03-10 22:45 ` [PATCHv2 8/9] target: Remove core_tpg_release_virtual_lun0 function Andy Grover
2014-03-10 22:45 ` [PATCHv2 9/9] target: Refactor core_enable_device_list_for_node Andy Grover
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=531A0DD2.6020905@redhat.com \
--to=agrover@redhat.com \
--cc=hch@infradead.org \
--cc=linux-scsi@vger.kernel.org \
--cc=target-devel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).