[LU-1039] data corruption in check_set

• Type: Bug
• Resolution: Fixed
• Priority: Blocker
• Fix Version/s: Lustre 2.2.0
• Affects Version/s: Lustre 2.2.0, Lustre 2.1.4
• Environment: any lustre
• Severity: 3
• Assignee: Oleg Drokin
• Reporter: Alexey Lyashkov

Description

ost_brw_read set a number bytes as rq_status, that confuses check_set() function.
that easy see if checksumming enabled.
i found it's with testing solution to avoid panic in check_set() with request reorder and failed bulk read.
attached patch solves both issues, but broke a request flags policy, and don't resend a bulk request so that can be short time solution, until check_set will cleared.

Issue Links

• is duplicated by: LU-2260 2.1.3 Client LBUG
• Trackbacks: Changelog 2.2 version 2.2.0 Support for networks: o2iblnd OFED 1.5.4 Server support for kernels: 2.6.32220.4.2.el6 (RHEL6) Client support for unpatched kernels: 2.6.18274.18.1.el5 (RHEL5) 2.6.32220.4.2.el6 (RHEL6) 2.6.32.360....

Comments

[Emoly Liu]

b2_1 port has been successfully cherry-picked as 6ba8b7b5d4fbf8d123adbb6b870abf9995eb39cb.

[Emoly Liu]

b2_1 port at http://review.whamcloud.com/4499

[Build Master (Inactive)]

Integrated in lustre-dev » x86_64,client,el6,inkernel #340
LU-1039 ptlrpc: handle bulk IO errors correctly. (Revision c9590221dc43dd5e7a7ede389f0a7d9cf566e5bf)

Result = SUCCESS
Oleg Drokin : c9590221dc43dd5e7a7ede389f0a7d9cf566e5bf
Files :

• lustre/tests/sanity.sh
• lustre/include/obd_support.h
• lustre/ptlrpc/client.c
• lustre/ptlrpc/events.c
• lustre/ost/ost_handler.c

[Build Master (Inactive)]

Integrated in lustre-dev » x86_64,server,el5,inkernel #340
LU-1039 ptlrpc: handle bulk IO errors correctly. (Revision c9590221dc43dd5e7a7ede389f0a7d9cf566e5bf)

Result = SUCCESS
Oleg Drokin : c9590221dc43dd5e7a7ede389f0a7d9cf566e5bf
Files :

• lustre/ptlrpc/client.c
• lustre/include/obd_support.h
• lustre/ost/ost_handler.c
• lustre/tests/sanity.sh
• lustre/ptlrpc/events.c

[Build Master (Inactive)]

Integrated in lustre-dev » i686,client,el5,inkernel #340
LU-1039 ptlrpc: handle bulk IO errors correctly. (Revision c9590221dc43dd5e7a7ede389f0a7d9cf566e5bf)

Result = SUCCESS
Oleg Drokin : c9590221dc43dd5e7a7ede389f0a7d9cf566e5bf
Files :

• lustre/tests/sanity.sh
• lustre/ptlrpc/client.c
• lustre/include/obd_support.h
• lustre/ptlrpc/events.c
• lustre/ost/ost_handler.c

[Build Master (Inactive)]

Integrated in lustre-dev » x86_64,server,el6,inkernel #340
LU-1039 ptlrpc: handle bulk IO errors correctly. (Revision c9590221dc43dd5e7a7ede389f0a7d9cf566e5bf)

Result = SUCCESS
Oleg Drokin : c9590221dc43dd5e7a7ede389f0a7d9cf566e5bf
Files :

• lustre/tests/sanity.sh
• lustre/ptlrpc/events.c
• lustre/ptlrpc/client.c
• lustre/include/obd_support.h
• lustre/ost/ost_handler.c

[Build Master (Inactive)]

Integrated in lustre-dev » i686,server,el5,inkernel #340
LU-1039 ptlrpc: handle bulk IO errors correctly. (Revision c9590221dc43dd5e7a7ede389f0a7d9cf566e5bf)

Result = SUCCESS
Oleg Drokin : c9590221dc43dd5e7a7ede389f0a7d9cf566e5bf
Files :

• lustre/ptlrpc/events.c
• lustre/tests/sanity.sh
• lustre/include/obd_support.h
• lustre/ptlrpc/client.c
• lustre/ost/ost_handler.c

[Build Master (Inactive)]

Integrated in lustre-dev » i686,client,el6,inkernel #340
LU-1039 ptlrpc: handle bulk IO errors correctly. (Revision c9590221dc43dd5e7a7ede389f0a7d9cf566e5bf)

Result = SUCCESS
Oleg Drokin : c9590221dc43dd5e7a7ede389f0a7d9cf566e5bf
Files :

• lustre/ptlrpc/client.c
• lustre/ptlrpc/events.c
• lustre/ost/ost_handler.c
• lustre/include/obd_support.h
• lustre/tests/sanity.sh

[Build Master (Inactive)]

Integrated in lustre-dev » x86_64,client,el5,inkernel #340
LU-1039 ptlrpc: handle bulk IO errors correctly. (Revision c9590221dc43dd5e7a7ede389f0a7d9cf566e5bf)

Result = SUCCESS
Oleg Drokin : c9590221dc43dd5e7a7ede389f0a7d9cf566e5bf
Files :

• lustre/ost/ost_handler.c
• lustre/include/obd_support.h
• lustre/ptlrpc/client.c
• lustre/tests/sanity.sh
• lustre/ptlrpc/events.c

[Build Master (Inactive)]

Integrated in lustre-master » i686,server,el6,ofa #498
LU-1039 ptlrpc: handle bulk IO errors correctly. (Revision c9590221dc43dd5e7a7ede389f0a7d9cf566e5bf)

Result = SUCCESS
Oleg Drokin : c9590221dc43dd5e7a7ede389f0a7d9cf566e5bf
Files :

• lustre/include/obd_support.h
• lustre/ost/ost_handler.c
• lustre/ptlrpc/client.c
• lustre/tests/sanity.sh
• lustre/ptlrpc/events.c