From: vrbagal1 <vrbagal1@linux.vnet.ibm.com>
To: Li Wang <liwang@redhat.com>
Cc: Jens Axboe <axboe@kernel.dk>,
linux-block@vger.kernel.org, snitzer@redhat.com,
Kent Overstreet <kent.overstreet@gmail.com>,
chuhu.ncepu@gmail.com, sachinp <sachinp@linux.vnet.ibm.com>
Subject: Re: [PATCH 0/2] Fix bio_set copy
Date: Fri, 08 Jun 2018 15:03:30 +0530 [thread overview]
Message-ID: <ac9087308264e32cfbfa364af24c83a8@linux.vnet.ibm.com> (raw)
In-Reply-To: <CAEemH2fcvMh2jS2zF0uvUuCrpL79LoBHFEQ84_bsbYkzsvMRmQ@mail.gmail.com>
On 2018-06-08 11:42, Li Wang wrote:
> Jens Axboe <axboe@kernel.dk> wrote:
>
>> After a recent change, dm ends up doing a straight copy of a
>> bio_set. This breaks for various items in that bio_set,
>> most notably lists. Provide a proper helper for it, and use it.
>
>
I am not seeing the issue, I reported with this patch applied to latest
mainline.
JOB ID : 5d6db9ad320b92b77cd92056fb4514c9afa0feea
JOB LOG :
/root/avocado-korg/korg/results/job-2018-06-08T03.49-5d6db9a/job.log
(1/4) memhotplug.py:memstress.test_hotplug_loop: PASS (254.14 s)
(2/4) memhotplug.py:memstress.test_hotplug_toggle: PASS (246.81 s)
(3/4) memhotplug.py:memstress.test_dlpar_mem_hotplug: PASS (3.95 s)
(4/4) memhotplug.py:memstress.test_hotplug_per_numa_node: PASS (223.96
s)
RESULTS : PASS 4 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 |
CANCEL 0
JOB TIME : 734.52 s
JOB HTML :
/root/avocado-korg/korg/results/job-2018-06-08T03.49-5d6db9a/results.html
Reported-by: Venkat R.B <vrbagal1@linux.vnet.ibm.com>
Tested-by: Venkat R.B <vrbagal1@linux.vnet.ibm.com>
>
> I can confirm that the issue[1] was gone with this patchset applied.
>
> Tested-by: Li Wang <liwang@redhat.com>
>
>
> [1] https://lkml.org/lkml/2018/6/7/36 [1]
>
> --
>
> Regards,
>
> Li Wang
>
>
> Links:
> ------
> [1] https://lkml.org/lkml/2018/6/7/36
next prev parent reply other threads:[~2018-06-08 9:33 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-06-07 20:45 [PATCH 0/2] Fix bio_set copy Jens Axboe
2018-06-07 20:45 ` [PATCH 1/2] block: add bioset_init_from_src() helper Jens Axboe
2018-06-07 20:45 ` [PATCH 2/2] dm: use bioset_init_from_src() to copy bio_set Jens Axboe
2018-06-08 6:12 ` [PATCH 0/2] Fix bio_set copy Li Wang
2018-06-08 9:33 ` vrbagal1 [this message]
2018-06-08 13:13 ` Jens Axboe
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=ac9087308264e32cfbfa364af24c83a8@linux.vnet.ibm.com \
--to=vrbagal1@linux.vnet.ibm.com \
--cc=axboe@kernel.dk \
--cc=chuhu.ncepu@gmail.com \
--cc=kent.overstreet@gmail.com \
--cc=linux-block@vger.kernel.org \
--cc=liwang@redhat.com \
--cc=sachinp@linux.vnet.ibm.com \
--cc=snitzer@redhat.com \
/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).