From: Bart Van Assche <bart.vanassche@sandisk.com>
To: "Nicholas A. Bellinger" <nab@daterainc.com>,
target-devel <target-devel@vger.kernel.org>
Cc: linux-scsi <linux-scsi@vger.kernel.org>,
Christoph Hellwig <hch@lst.de>, Hannes Reinecke <hare@suse.de>,
Mike Christie <mchristi@redhat.com>,
Sagi Grimberg <sagig@mellanox.com>,
Andy Grover <agrover@redhat.com>,
Sebastian Andrzej Siewior <bigeasy@linutronix.de>,
Juergen Gross <jgross@suse.com>,
Andrzej Pietrasiewicz <andrzej.p@samsung.com>,
Chris Boot <bootc@bootc.net>,
Nicholas Bellinger <nab@linux-iscsi.org>,
Vu Pham <vu@mellanox.com>
Subject: Re: [PATCH-v3 06/14] ib_srpt: Convert to target_alloc_session usage
Date: Thu, 4 Feb 2016 14:14:52 -0800 [thread overview]
Message-ID: <56B3CD5C.1050905@sandisk.com> (raw)
In-Reply-To: <1454137558-30396-7-git-send-email-nab@daterainc.com>
On 01/29/2016 11:05 PM, Nicholas A. Bellinger wrote:
> try_again:
> - se_acl = core_tpg_get_initiator_node_acl(&sport->port_tpg_1, p);
> - if (!se_acl) {
> + ch->sess = target_alloc_session(&sport->port_tpg_1, 0, 0,
> + TARGET_PROT_NORMAL, p, ch, NULL);
> + if (IS_ERR(ch->sess)) {
> pr_info("Rejected login because no ACL has been"
> - " configured yet for initiator %s.\n", ch->sess_name);
> + " configured yet for initiator %s.\n", p);
> /*
> * XXX: Hack to retry of ch->i_port_id without leading '0x'
> */
> @@ -2426,14 +2418,11 @@ try_again:
> p += 2;
> goto try_again;
> }
> - rej->reason = cpu_to_be32(
> + rej->reason = cpu_to_be32((PTR_ERR(ch->sess) == -ENOMEM) ?
> + SRP_LOGIN_REJ_INSUFFICIENT_RESOURCES :
> SRP_LOGIN_REJ_CHANNEL_LIMIT_REACHED);
Hello Nic,
The "no ACL has been configured yet" error message will be printed for
initator ACLs that do not start with "0x" even if the second lookup
succeeds. This will confuse users. Otherwise this patch looks fine to me.
Bart.
next prev parent reply other threads:[~2016-02-04 22:14 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-01-30 7:05 [PATCH-v3 00/14] target_alloc_session w/ percpu_ida+ACK_KREF conversion Nicholas A. Bellinger
2016-01-30 7:05 ` [PATCH-v3 01/14] target: Add target_alloc_session() helper function Nicholas A. Bellinger
2016-01-30 7:05 ` [PATCH-v3 02/14] target: Convert demo-mode only drivers to target_alloc_session Nicholas A. Bellinger
2016-01-30 7:05 ` [PATCH-v3 03/14] vhost/scsi: Convert to target_alloc_session usage Nicholas A. Bellinger
2016-01-30 7:05 ` [PATCH-v3 04/14] tcm_qla2xxx: " Nicholas A. Bellinger
2016-01-30 7:05 ` [PATCH-v3 05/14] tcm_fc: " Nicholas A. Bellinger
2016-01-30 7:05 ` [PATCH-v3 06/14] ib_srpt: " Nicholas A. Bellinger
2016-02-04 22:14 ` Bart Van Assche [this message]
2016-01-30 7:05 ` [PATCH-v3 07/14] sbp-target: Conversion to percpu_ida tag pre-allocation Nicholas A. Bellinger
2016-01-30 7:05 ` [PATCH-v3 08/14] sbp-target: Convert to TARGET_SCF_ACK_KREF I/O krefs Nicholas A. Bellinger
2016-01-30 7:05 ` [PATCH-v3 09/14] usb-gadget/tcm: Conversion to percpu_ida tag pre-allocation Nicholas A. Bellinger
2016-01-30 7:05 ` [PATCH-v3 10/14] usb-gadget/tcm: Convert to TARGET_SCF_ACK_KREF I/O krefs Nicholas A. Bellinger
2016-01-30 7:05 ` [PATCH-v3 11/14] xen-scsiback: Convert to percpu_ida tag allocation Nicholas A. Bellinger
2016-01-30 7:05 ` [PATCH-v3 12/14] xen-scsiback: Convert to TARGET_SCF_ACK_KREF I/O krefs Nicholas A. Bellinger
2016-02-02 16:31 ` Juergen Gross
2016-02-03 5:03 ` Nicholas A. Bellinger
2016-02-04 8:22 ` Juergen Gross
2016-01-30 7:05 ` [PATCH-v3 13/14] tcm_fc: Convert to TARGET_SCF_ACK_KREF I/O + TMR krefs Nicholas A. Bellinger
2016-01-30 7:05 ` [PATCH-v3 14/14] ib_srpt: Convert to percpu_ida tag allocation Nicholas A. Bellinger
2016-02-04 22:23 ` Bart Van Assche
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=56B3CD5C.1050905@sandisk.com \
--to=bart.vanassche@sandisk.com \
--cc=agrover@redhat.com \
--cc=andrzej.p@samsung.com \
--cc=bigeasy@linutronix.de \
--cc=bootc@bootc.net \
--cc=hare@suse.de \
--cc=hch@lst.de \
--cc=jgross@suse.com \
--cc=linux-scsi@vger.kernel.org \
--cc=mchristi@redhat.com \
--cc=nab@daterainc.com \
--cc=nab@linux-iscsi.org \
--cc=sagig@mellanox.com \
--cc=target-devel@vger.kernel.org \
--cc=vu@mellanox.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 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.