> I think the last use of this list was removed by
> commit 23d6fefbb3f6 ("scsi: iscsi: Fix in-kernel conn failure
> handling").
>
> Build tested only.
Applied to 6.10/scsi-staging, thanks!
--
Martin K. Petersen Oracle Linux Engineering
--
You received
o_lld_bus const
https://git.kernel.org/mkp/scsi/c/ac0dd0f33adb
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
"open-iscsi" group.
To unsubscribe from this group and stop receiving emails from i
ging, thanks!
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
"open-iscsi" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to open-iscsi+unsubscr...@googlegroups.com.
T
> and hostbyte, driverbyte, statusbyte, and sense data for specific SCSI
> Command. Two new error injection is defined to make abort command or
> reset LUN failed.
Please review patches 7 through 10. Thank you!
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message be
msg_len < sizeof(*nlh) + sizeof(*ev) ||
> skb->len < nlh->nlmsg_len) {
> break;
> }
> ...
> err = iscsi_if_recv_msg(skb, nlh, );
>
> [...]
Applied to 6.6/scsi-queue, thanks!
[1/2] scsi: iscsi: Add length check for nlattr payload
https:
On Tue, 25 Jul 2023 22:15:31 +0800, YueHaibing wrote:
> This is not used anymore, so can be removed.
>
>
Applied to 6.6/scsi-queue, thanks!
[1/1] scsi: iscsi: Remove unused extern declaration iscsi_lookup_iface()
https://git.kernel.org/mkp/scsi/c/a615e93d6cfe
--
Martin K.
Lin,
> The function iscsi_if_set_param and iscsi_if_set_host_param converts
> nlattr payload to type char* and then call C string handling functions
> like sscanf and kstrdup.
Applied to 6.6/scsi-staging, thanks!
--
Martin K. Petersen Oracle Linux Engineering
--
You
Lin,
> The current NETLINK_ISCSI netlink parsing loop checks every nlmsg to
> make sure the length is bigger than the sizeof(struct iscsi_uevent)
> and then calls iscsi_if_recv_msg(...).
Applied to 6.6/scsi-staging, thanks!
--
Martin K. Petersen Oracle Linux Engineering
YueHaibing,
> This is not used anymore, so can be removed.
Applied to 6.6/scsi-staging, thanks!
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
"open-iscsi" group.
To unsubscribe from this g
Wenchao,
> I found an issue that kernel would send ISCSI_KEVENT_UNBIND_SESSION
> for multiple times which should be fixed.
Applied to 6.2/scsi-staging, thanks!
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google
t_cnt if dispatch failed
https://git.kernel.org/mkp/scsi/c/cfee29ffb45b
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
"open-iscsi" group.
To unsubscribe from this group and stop receiving emails
scsi-queue, thanks!
[1/1] scsi:iscsi: rename iscsi_set_param to iscsi_if_set_param
https://git.kernel.org/mkp/scsi/c/0c26a2d7c980
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
"open-iscsi"
Wenchao,
> Following scenario would make scsi_device's iodone_cnt mismatch with
> iorequest_cnt even if there is no request on this device any more.
Applied to 6.2/scsi-staging, thanks!
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message becau
Wenchao,
> There are two iscsi_set_param() functions individually defined in
> libiscsi.c and scsi_transport_iscsi.c which is confused.
Applied to 6.2/scsi-staging, thanks!
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subs
si: fix possible memory leak when device_register failed
https://git.kernel.org/mkp/scsi/c/f014165faa7b
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
"open-iscsi" group.
To unsubscribe from this gr
ks!
[1/1] scsi: iscsi: prefer xmit of DataOut before new cmd
https://git.kernel.org/mkp/scsi/c/65080c51fde4
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
"open-iscsi" group.
To unsubscribe from t
3fd3a52ca672
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
"open-iscsi" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to open-iscsi+unsubscr...@googlegroups.com.
T
keliu,
> Use ida_alloc()/ida_free() instead of deprecated
> ida_simple_get()/ida_simple_remove() .
Applied to 5.20/scsi-staging, thanks!
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
"open-i
issue at run time.
>
>
Applied to 5.19/scsi-queue, thanks!
[1/1] scsi: iscsi: fix harmless double shift bug
https://git.kernel.org/mkp/scsi/c/565138ac5f8a
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google G
ging, thanks!
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
"open-iscsi" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to open-iscsi+unsubscr...@googlegroups.com.
T
Wenchao,
> iscsi_cls_conn is alloced by kzalloc(), the whole iscsi_cls_conn is
> zero filled already including the dd_data. So it is unnecessary to
> call memset again.
Applied to 5.18/scsi-staging, thanks!
--
Martin K. Petersen Oracle Linux Engineering
--
You received thi
scsi: Add iscsi_cls_conn to sysfs after been initialized
https://git.kernel.org/mkp/scsi/c/7dae459f5e56
[3/3] scsi:libiscsi: teradown iscsi_cls_conn gracefully
https://git.kernel.org/mkp/scsi/c/8709c323091b
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message be
Wenchao,
> We found a NULL pointer dereference in iscsi_sw_tcp_conn_get_param(),
> the root reason is we did sysfs addition wrong.
Applied to 5.18/scsi-staging, thanks!
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the
it.kernel.org/mkp/scsi/c/e018f03d6ccb
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
"open-iscsi" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to open-iscsi+unsubs
hwait in iscsi_session_setup() to prevent reinit by mistake,
> also in case we can handle multiple conns in the future.
Applied to 5.16/scsi-staging, thanks!
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
"op
ble in sysfs. Found in code review.
>
>
Applied to 5.15/scsi-fixes, thanks!
[1/1] scsi: iscsi: Adjuest iface sysfs attr detection
https://git.kernel.org/mkp/scsi/c/4e2855082925
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscr
ontext of electronics and computing "busses" still appears to be
widely in use. In any case, whether to use one or the other is up to the
author of the code in question. I only merged fixed for what was obvious
typos.
Thanks!
--
Martin K. Petersen Oracle Linux Engineering
=> interrupted
Applied to 5.14/scsi-queue, thanks!
[1/1] scsi: Fix spelling mistakes in header files
https://git.kernel.org/mkp/scsi/c/40d6b939e4df
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
"open-isc
Zhen,
> Fix some spelling mistakes in comments:
> pathes ==> paths
> Resouce ==> Resource
> retreived ==> retrieved
> keep-alives ==> keep-alive
> recevied ==> received
> busses ==> buses
> interruped ==> interrupted
Applied to 5.14/scsi-staging
: iscsi: fix inappropriate use of put_device
https://git.kernel.org/mkp/scsi/c/6dc1c7ab6f04
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
"open-iscsi" group.
To unsubscribe from this group and stop
Mauro,
> Some identifiers have different names between their prototypes and the
> kernel-doc markup.
Applied to 5.11/scsi-staging, thanks!
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
"open-i
t;[SCSI] iscsi_tcp: use iscsi_conn_get_addr_param
> libiscsi function") introduced getpeername() within the session spinlock.
>
> [...]
Applied to 5.9/scsi-fixes, thanks!
[1/1] scsi: iscsi: iscsi_tcp: Avoid holding spinlock while calling getpeername()
https://git.kernel.org/mkp/scsi/c
g)) >= 1 && !PageSlab(sg_page(sg)" part is to
> make sure the page can be sent to network layer's zero copy path. This
> part is exactly what sendpage_ok() does.
>
> This patch uses use sendpage_ok() in iscsi_tcp_segment_map() to replace
> the original open coded checks.
Lo
https://git.kernel.org/mkp/scsi/c/68e12e5f6135
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
"open-iscsi" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to ope
thread_workqueue(), added WQ_SYSFS and drop
> __WQ_ORDERED_EXPLICIT since __WQ_ORDERED_EXPLICIT workqueue isn't allowed to
> change "cpumask".
Applied to 5.9/scsi-queue, thanks!
[1/1] scsi: iscsi: Register sysfs for workqueue iscsi_destroy
https://git.kernel.org/mkp/scsi/c/919a295
e modified when max_active = 1.
>
> This patch change back max_active to 1 so as to keep the same behaviour as
> before.
Applied to 5.8/scsi-fixes, thanks!
[1/2] scsi: iscsi: Change iSCSI workqueue max_active back to 1
https://git.kernel.org/mkp/scsi/c/1a9826204109
--
Martin K. Pe
://git.kernel.org/mkp/scsi/c/0267ffce562c
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
"open-iscsi" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to open-iscsi
1 to 2, because "cpumask" of
> ordered workqueue isn't allowed to change.
>
> [...]
Applied to 5.8/scsi-queue, thanks!
[1/1] scsi: iscsi: Register sysfs for iscsi workqueue
https://git.kernel.org/mkp/scsi/c/3ce419662dd4
--
Martin K. Petersen Oracle Linux Engineerin
] scsi: iscsi: Fix deadlock on recovery path during GFP_IO reclaim
https://git.kernel.org/mkp/scsi/c/7e7cd796f277
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
"open-iscsi" group.
To unsubscribe from
Gabriel,
> iscsi suffers from a deadlock in case a management command submitted
> via the netlink socket sleeps on an allocation while holding the
> rx_queue_mutex,
This does not apply to 5.8/scsi-queue. Please resubmit.
Thanks!
--
Martin K. Petersen Oracle Linux Engineering
nn(conn, ev->u.stop_conn.flag);
> + iscsi_if_stop_conn(conn, ev->u.stop_conn.flag);
> else
> err = -EINVAL;
> break;
> case ISCSI_UEVENT_SEND_PDU:
> conn = iscsi_conn_lookup(ev->u.send_pd
inglethread_workqueue("iscsi_eh");
> + iscsi_eh_timer_workq = alloc_workqueue("%s",
> + WQ_SYSFS | __WQ_LEGACY | WQ_MEM_RECLAIM | WQ_UNBOUND,
> + 2, "iscsi_eh");
> if (!iscsi_eh_timer_workq) {
> err = -ENOMEM;
&g
tely mangled. Please
use git send-email to submit patches in the future! Thanks!
Applied to 5.7/scsi-queue.
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
"open-iscsi" group.
To unsubscribe from this gr
wubo,
> Fix an error count for active session if the total_cmds is invalid on
> the function iscsi_session_setup(). decrement the number of active
> sessions before the func return.
Applied to 5.7/scsi-queue, thanks!
--
Martin K. Petersen Oracle Linux Engineering
--
You
-iscsi tries to
> reopen every connection, but on different initiators, we'd like to
> know which connections have failed.
Applied to 5.7/scsi-queue, thanks!
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
he operation
> is queued and before it is performed. This change allow other
> operations to run in other sessions in the meantime, removing one of
> the major iSCSI bottlenecks for us.
Applied to 5.7/scsi-queue. Thanks!
--
Martin K. Petersen Oracle Linux Engineering
--
You received thi
Gustavo,
> The current codebase makes use of the zero-length array language
> extension to the C90 standard, but the preferred mechanism to declare
> variable-length types such as these ones is a flexible array
> member[1][2], introduced in C99:
Applied to 5.7/scsi-queue, thanks.
In any case: Applied to 5.7/scsi-queue. Thanks!
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
"open-iscsi" group.
To unsubscribe from this group and stop receiving emails from it, send an email
ute_container_device
> trigger version.
Applied to 5.6/scsi-queue, thanks!
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
"open-iscsi" group.
To unsubscribe from this group and stop receiving emails fro
ely inside the kernel.
> This way, the failover can happen and pending IO can continue even if
> the daemon is dead. Once the daemon comes alive again, it can execute
> recovery procedures if applicable.
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message becau
Gabriel,
> A faulty userspace that calls destroy_session() before destroying the
> connections can trigger the failure. This patch prevents the issue by
> refusing to destroy the session if there are outstanding connections.
Applied to 5.6/scsi-queue, thanks!
--
Martin K.
s just
personal taste.
Applied to 5.5/scsi-fixes. Thanks!
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
"open-iscsi" group.
To unsubscribe from this group and stop receiving emails from it, send an emai
example, a client only send msg without calling recvmsg func, then
> it will result in the watchdog soft lockup. The details are given as
> follows,
> Signed-off-by: Bo Wu
> Reviewed-by: Zhiqiang Liu
> Reviewed-by: Lee Duncan
I haven't seen a Reviewed-by: from Lee on th
Gabriel,
> Although, looks like the MAINTAINERS file doesn't list linux-scsi as
> the target for iscsi patches. Would you take the fix below to address
> that?
Applied to 5.5/scsi-queue, thanks!
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message be
> succeeded before trying to use the socket.
Applied to 5.5/scsi-queue. But please make sure to send patch
submissions to linux-s...@vger.kernel.org.
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
&qu
Gabriel,
> A faulty userspace that calls destroy_session() before destroying the
> connections can trigger the failure. This patch prevents the issue by
> refusing to destroy the session if there are outstanding connections.
Ditto. Please send to linux-scsi@vger.
--
Martin K.
ple, a client only send msg without calling recvmsg func,
> then it will result in the watchdog soft lockup.
> The details are given as follows,
Lee/Chris/Ulrich: Please review!
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to
Christoph,
> Use the the GPLv2 SPDX tag instead of verbose boilerplate text.
Applied to 5.3/scsi-queue except for patch #24.
Patch #13 used /* */ syntax on some of the .c files. I fixed those up.
--
Martin K. Petersen Oracle Linux Engineering
Vasily,
> In "XFS over network block device" scenario XFS can create IO requests
> with slab-based XFS metadata. During processing such requests
> tcp_sendpage() can merge skb fragments with neighbour slab objects.
Applied to 5.1/scsi-queue, thanks!
--
Martin K. Peterse
Vasily,
> James, Martin, noone replied 2 weeks, could you please pick up this
> patch?
No objections from me wrt. to the change. However, I am awaiting a
review from the iSCSI maintainers Lee and Chris.
--
Martin K. Petersen Oracle Linux Engineering
--
You received this m
Bob,
> When a target sends Check Condition, whilst initiator is busy xmiting
> re-queued data, could lead to race between iscsi_complete_task() and
> iscsi_xmit_task() and eventually crashing with the following kernel
> backtrace.
Applied to 5.0/scsi-fixes, thanks!
--
Martin
Fred,
> This commit enhances iscsi initiator modules to capture iscsi debug
> messages using linux kernel tracepoint facility:
Applied to 4.21/scsi-queue, thanks!
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the
Fred,
> This commit addresses NULL pointer dereference in
> iscsi_eh_session_reset. Reference should not be made to
> session->leadconn when session->state is set to ISCSI_STATE_TERMINATE.
Applied to 4.20/scsi-fixes, thanks!
--
Martin K. Petersen Oracle Linux Engin
; To fix this, set BDI_CAP_STABLE_WRITES when data digest is enabled
> in iscsi_tcp .slave_configure callback.
Applied to 4.16/scsi-fixes. Thanks!
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
"open-iscsi" group.
STABLE_WRITES when data digest is enabled
> in iscsi_tcp .slave_configure callback.
Lee, Chris: Please review!
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
"open-iscsi" group.
To unsubscribe fro
> Never directly free @dev after calling device_register(), even
> if it returned an error! Always use put_device() to give up the
> reference initialized.
Lee, Chris: Please review!
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are s
rever on its
> sd_sync_cache() logic, after issuing the SYNCHRONIZE_CACHE cmd to all
> still existent paths.
Applied to 4.16/scsi-queue. Thank you!
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
"open-iscsi"
; sd_sync_cache() logic, after issuing the SYNCHRONIZE_CACHE cmd to all
> still existent paths.
Chris and Lee: Please review. Thanks!
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
"open-iscsi" group.
To
Arvind,
> Trivial fix to spelling mistakes in 'iscsi_get_host_stats'.
Applied to 4.15/scsi-queue.
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
"open-iscsi" group.
To unsubscribe from this g
Kees,
> In preparation for unconditionally passing the struct timer_list
> pointer to all timer callbacks, switch to using the new timer_setup()
> and from_timer() to pass the timer pointer explicitly.
Reviewed-by: Martin K. Petersen <martin.peter...@oracle.com>
--
Mar
Khazhismel,
> Noticed these don't seem to be in 4.14/scsi-queue
Not sure what happened there. I apologize.
They are now in 4.14/scsi-fixes.
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
"open-iscs
Chris,
> Looks good to me, fixes up the code given that the comment there about
> calling iscsi_remove_session wasn't being followed.
Applied these two to 4.14/scsi-queue.
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subs
Bhumika,
> Make these const as they are only stored in the type field of a device
> structure, which is const.
Applied to 4.14/scsi-queue. Thanks!
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
"
tuations.
Applied to 4.12/scsi-queue.
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
"open-iscsi" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to open-iscsi+unsubscr.
>>>>> "Chris" == Chris Leech <cle...@redhat.com> writes:
Chris> There's a rather long standing regression from the commit
Chris> "libiscsi: Reduce locking contention in fast path"
Applied to 4.11/scsi-fixes.
--
Martin K. Petersen Oracle L
terprise supported kernels have been backing out the lock
Chris> split for while now, thanks to the efforts at IBM where a lab
Chris> setup has the most reliable reproducer I've seen on this issue.
Chris> This patch has been tested there successfully.
Reviews, please!
--
Martin K. Pe
ugh the Linux SCSI mailing list and repository for a while,
Lee> now.
Yep. Just wanted to make sure.
Thanks!
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
"open-iscsi" group.
To unsubscribe from
>>>>> "Lee" == Lee Duncan <ldun...@suse.com> writes:
Lee,
Lee> Chris Leech and I are taking over as open-iscsi maintainers.
Do you want me to queue the MAINTAINER update?
Lee> * Removed git repository, since code in tree
Is it your plan to go through t
>>>>> "Lee" == Lee Duncan <ldun...@suse.com> writes:
Lee> It looks like Mike and Chris are good with it.
However, it received no formal reviews or acked-bys...
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you a
is a no-go.
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
open-iscsi group.
To unsubscribe from this group and stop receiving emails from it, send an email
to open-iscsi+unsubscr...@googlegroups.com.
To post
out a proposed patch.
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
open-iscsi group.
To unsubscribe from this group and stop receiving emails from it, send an email
to open-iscsi+unsubscr...@googlegroups.com
SAME problem was that the target returns an iSCSI-specific
error code rather than a check condition with ILLEGAL REQUEST. You were
going to talk to MS.
However, if somebody sends me the relevant INQUIRY strings for the MS
target I'll add it to the WRITE SAME quirk list.
--
Martin K. Petersen
handling in the iscsi layer for it. I
Mike can make a patch if you have not already.
It would be awfully nice to get it turned into an ILLEGAL
REQUEST/0x20/0x0.
--
Martin K. Petersen Oracle Linux Engineering
--
You received this message because you are subscribed to the Google Groups
open
83 matches
Mail list logo