From: Andy Grover <agrover@redhat.com>
To: target-devel@vger.kernel.org
Cc: linux-scsi@vger.kernel.org,
"Nicholas A. Bellinger" <nab@linux-iscsi.org>,
Christoph Hellwig <hch@lst.de>
Subject: Re: [PATCHv2 RESEND 0/9] target: Save memory on unused se_dev_entrys and se_luns
Date: Wed, 23 Apr 2014 15:36:27 -0700 [thread overview]
Message-ID: <5358406B.5040906@redhat.com> (raw)
In-Reply-To: <1395704598-31073-1-git-send-email-agrover@redhat.com>
On 03/24/2014 04:43 PM, Andy Grover wrote:
> Hi nab, hch, and all,
>
> This patchset reduces the amount of memory for se_dev_entry and se_lun
> arrays by waiting to allocate array members, and includes some related
> simplification patches too. This is a rework of a few patches I submitted
> in December in a larger series, but keeping it simple by sticking with
> fixed-size arrays for device_list and tpg_lun_list for now.
>
> Testing: created and removed luns and mapped luns, worked ok.
>
> Changes in resend:
> Rebase to latest target-pending/for-next
> Changes in v2:
> Added patches 1-2, which ensure locking around the arrays
> Added patch 9, refactors enable_device_list_for_node
> Patch 7/6 folded into series
Does this set look ok, or any other changes needed? Just haven't heard
anything.
Thanks -- Andy
next prev parent reply other threads:[~2014-04-23 22:36 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-03-24 23:43 [PATCHv2 RESEND 0/9] target: Save memory on unused se_dev_entrys and se_luns Andy Grover
2014-03-24 23:43 ` [PATCHv2 RESEND 1/9] target: Add locking to some accesses to nacl.device_list Andy Grover
2014-03-24 23:43 ` [PATCHv2 RESEND 2/9] target: Don't unlock/relock tpg_lun_lock in loop in add_node_to_devs Andy Grover
2014-03-24 23:43 ` [PATCHv2 RESEND 3/9] target: Allocate se_dev_entrys in device list only when used Andy Grover
2014-03-24 23:43 ` [PATCHv2 RESEND 4/9] target: core_tpg_post_dellun can return void Andy Grover
2014-03-24 23:43 ` [PATCHv2 RESEND 5/9] target: Change core_dev_del_lun to take a se_lun instead of unpacked_lun Andy Grover
2014-03-24 23:43 ` [PATCHv2 RESEND 6/9] target: Rename core_tpg_post_dellun to remove_lun Andy Grover
2014-03-24 23:43 ` [PATCHv2 RESEND 7/9] target: Allocate se_luns only when used Andy Grover
2014-03-24 23:43 ` [PATCHv2 RESEND 8/9] target: Remove core_tpg_release_virtual_lun0 function Andy Grover
2014-03-24 23:43 ` [PATCHv2 RESEND 9/9] target: Refactor core_enable_device_list_for_node Andy Grover
2014-04-23 22:36 ` Andy Grover [this message]
2014-04-25 5:34 ` [PATCHv2 RESEND 0/9] target: Save memory on unused se_dev_entrys and se_luns Christoph Hellwig
2014-05-01 17:53 ` Andy Grover
2014-05-01 21:21 ` Nicholas A. Bellinger
2014-05-01 21:51 ` 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=5358406B.5040906@redhat.com \
--to=agrover@redhat.com \
--cc=hch@lst.de \
--cc=linux-scsi@vger.kernel.org \
--cc=nab@linux-iscsi.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).