All of lore.kernel.org
 help / color / mirror / Atom feed
From: 李秀波 <lixiubo@cmss.chinamobile.com>
To: AndyGrover <agrover@redhat.com>,
	nab@linux-iscsi.org, mchristi@redhat.com
Cc: shli@kernel.org, sheng@yasker.org, linux-scsi@vger.kernel.org,
	target-devel@vger.kernel.org, namei.unix@gmail.com,
	linux-mm@kvack.org
Subject: Re:  [PATCHv2 2/5] target/user: Add global data block pool support
Date: Sat, 18 Mar 2017 06:06:11 +0800	[thread overview]
Message-ID: <13903058-78b8-4737-9eef-849ee7bca307@email.android.com> (raw)
In-Reply-To: <2dd405f8-9f5b-405d-e744-9ee8bac77686@redhat.com>

[-- Attachment #1: Type: text/plain, Size: 1082 bytes --]



AndyGrover <agrover@redhat.com>写到：
>On 03/17/2017 01:04 AM, Xiubo Li wrote:
>> [...]
>>> These days what I have gotten is that the unmap_mapping_range()
>could
>>> be used.
>>> At the same time I have deep into the mm code and fixed the double
>>> usage of
>>> the data blocks and possible page fault call trace bugs mentioned
>above.
>>>
>>> Following is the V3 patch. I have test this using 4 targets & fio
>for
>>> about 2 days, so
>>> far so good.
>>>
>>> I'm still testing this using more complex test case.
>>>
>> I have test it the whole day today:
>> - using 4 targets
>> - setting TCMU_GLOBAL_MAX_BLOCKS = [512 1K 1M 1G 2G]
>> - each target here needs more than 450 blocks when running
>> - fio: -iodepth [1 2 4 8 16] -thread -rw=[read write] -bs=[1K 2K 3K
>5K
>> 7K 16K 64K 1M] -size=20G -numjobs=10 -runtime=1000  ...
>
>Hi Xiubo,
>
>V3 is sounding very good. I look forward to reviewing it after it is
>posted.
>

Yes, I will post it later after more test and checking.

Thanks,

BRs
Xiubo


>Thanks -- Regards -- Andy

[-- Attachment #2: Type: text/html, Size: 1168 bytes --]

  reply	other threads:[~2017-03-17 22:06 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-08  8:45 [PATCHv2 0/5] Dynamic growing data area support lixiubo
2017-03-08  8:45 ` [PATCHv2 1/5] target/user: Add dynamic growing data area feature support lixiubo
2017-03-08  8:45 ` [PATCHv2 2/5] target/user: Add global data block pool support lixiubo
2017-03-08 20:20   ` Andy Grover
2017-03-16  9:39     ` Xiubo Li
2017-03-17  8:04       ` Xiubo Li
2017-03-17  8:04         ` Xiubo Li
2017-03-17 17:11         ` Andy Grover
2017-03-17 22:06           ` 李秀波 [this message]
2017-03-08  8:45 ` [PATCHv2 3/5] target/user: Fix possible overwrite of t_data_sg's last iov[] lixiubo
2017-03-16 18:23   ` Bryant G. Ly
2017-03-08  8:45 ` [PATCHv2 4/5] target/user: Fix wrongly calculating of the base_command_size lixiubo
2017-03-17  5:45   ` Xiubo Li
2017-03-08  8:45 ` [PATCHv2 5/5] target/user: Clean up tcmu_queue_cmd_ring lixiubo
2017-03-16 20:50   ` Bryant G. Ly

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=13903058-78b8-4737-9eef-849ee7bca307@email.android.com \
    --to=lixiubo@cmss.chinamobile.com \
    --cc=agrover@redhat.com \
    --cc=linux-mm@kvack.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=mchristi@redhat.com \
    --cc=nab@linux-iscsi.org \
    --cc=namei.unix@gmail.com \
    --cc=sheng@yasker.org \
    --cc=shli@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.