linux-block.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Omar Sandoval <osandov@osandov.com>
To: Bart Van Assche <bart.vanassche@wdc.com>
Cc: Omar Sandoval <osandov@fb.com>, linux-block@vger.kernel.org
Subject: Re: [PATCH blktests 00/15] Add SRP initiator driver tests
Date: Tue, 26 Jun 2018 14:02:47 -0700	[thread overview]
Message-ID: <20180626210247.GG1218@vader> (raw)
In-Reply-To: <20180622221946.10987-1-bart.vanassche@wdc.com>

On Fri, Jun 22, 2018 at 03:19:31PM -0700, Bart Van Assche wrote:
> Hello Omar,
> 
> As promised during LSF/MM, I have converted the srp-tests software to the
> blktests framework. This patch series included all blktests patches I postd
> a few days ago. Please consider this patch series for inclusion in the
> official blktests repository. Note: this patch series is also available as
> a github pull request, namely https://github.com/osandov/blktests/pull/24.
> 
> Thanks,
> 
> Bart.
> 
> Bart Van Assche (15):
>   common/rc: Fix _have_tracepoint()
>   Annotate include statements in shell scripts where the source file is
>     a variable
>   check, tests/meta/012: Use array["index"] instead of array[index]
>   Suppress shellcheck complaints about global variables
>   check: Avoid that shellcheck complains that $FULL appears unused
>   Multiple tests: remove unused and undefined variables
>   Avoid passing tests/block/002 arguments to _init_scsi_debug
>   check: Suppress a shellcheck warning about the DMESG_FILTER
>     initialization
>   Makefile: Do not suppress useful shellcheck warnings
>   tests/meta/group: Fix shellcheck complaints
>   make check: Also check tests/*/group files
>   check: Make TMPDIR absolute

I've applied all of these.

>   src/Makefile: Fix pattern rule
>   Add the discontiguous-io test program
>   Add tests for the SRP initiator and target drivers

For these, can you please reroll them with whatever group_requires()
check you need, and with the helper functions moved to tests/srp/rc.
Thanks!

      parent reply	other threads:[~2018-06-26 21:02 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-06-22 22:19 [PATCH blktests 00/15] Add SRP initiator driver tests Bart Van Assche
2018-06-22 22:19 ` [PATCH blktests 01/15] common/rc: Fix _have_tracepoint() Bart Van Assche
2018-06-22 22:19 ` [PATCH blktests 02/15] Annotate include statements in shell scripts where the source file is a variable Bart Van Assche
2018-06-22 22:19 ` [PATCH blktests 03/15] check, tests/meta/012: Use array["index"] instead of array[index] Bart Van Assche
2018-06-22 22:19 ` [PATCH blktests 04/15] Suppress shellcheck complaints about global variables Bart Van Assche
2018-06-22 22:19 ` [PATCH blktests 05/15] check: Avoid that shellcheck complains that $FULL appears unused Bart Van Assche
2018-06-22 22:19 ` [PATCH blktests 06/15] Multiple tests: remove unused and undefined variables Bart Van Assche
2018-06-22 22:19 ` [PATCH blktests 07/15] Avoid passing tests/block/002 arguments to _init_scsi_debug Bart Van Assche
2018-06-22 22:19 ` [PATCH blktests 08/15] check: Suppress a shellcheck warning about the DMESG_FILTER initialization Bart Van Assche
2018-06-22 22:19 ` [PATCH blktests 09/15] Makefile: Do not suppress useful shellcheck warnings Bart Van Assche
2018-06-22 22:19 ` [PATCH blktests 10/15] tests/meta/group: Fix shellcheck complaints Bart Van Assche
2018-06-22 22:19 ` [PATCH blktests 11/15] make check: Also check tests/*/group files Bart Van Assche
2018-06-22 22:19 ` [PATCH blktests 12/15] check: Make TMPDIR absolute Bart Van Assche
2018-06-22 22:19 ` [PATCH blktests 13/15] src/Makefile: Fix pattern rule Bart Van Assche
2018-06-25 22:54   ` Omar Sandoval
2018-06-22 22:19 ` [PATCH blktests 14/15] Add the discontiguous-io test program Bart Van Assche
2018-06-22 22:19 ` [PATCH blktests 15/15] Add tests for the SRP initiator and target drivers Bart Van Assche
2018-06-26  8:57   ` Johannes Thumshirn
2018-06-26 15:25     ` Bart Van Assche
2018-06-26 21:01       ` Omar Sandoval
2018-06-23  3:14 ` [PATCH blktests 00/15] Add SRP initiator driver tests Ming Lei
2018-06-23  3:19   ` Ming Lei
2018-06-25 17:05   ` Bart Van Assche
2018-06-26  1:37     ` Ming Lei
2018-06-26  2:16       ` Bart Van Assche
2018-06-26  7:27         ` Johannes Thumshirn
2018-06-26 15:24           ` Bart Van Assche
2018-06-26 21:00             ` Omar Sandoval
2018-06-27  7:04             ` Johannes Thumshirn
2018-06-26 21:02 ` Omar Sandoval [this message]

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=20180626210247.GG1218@vader \
    --to=osandov@osandov.com \
    --cc=bart.vanassche@wdc.com \
    --cc=linux-block@vger.kernel.org \
    --cc=osandov@fb.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).