Re: scatterlist size of 4096

2010-05-18 Thread Ulrich Windl
On 5 May 2010 at 11:47, Mike Christie wrote: On 05/03/2010 06:51 AM, Jack Z wrote: Hi group, I have been tracing the code related to sending PDUs from iscsi initiator (ver 2.0-871). And through some printk()s i realize that starting from iscsi_sw_tcp_pdu_init(), all the functions

Re: scatterlist size of 4096

2010-05-06 Thread Mike Christie
On 05/06/2010 03:53 PM, Jack Z wrote: Hi Mike, Thanks for your reply. What I was doing was just trying to understand how the the data travels through normal fs or block layer, reaches iscsi and gets sent over the network, when there is a normal read()/write() request. So the function call

Re: scatterlist size of 4096

2010-05-06 Thread Jack Z
Hi Mike, Thank you for the super prompt reply. I actually downloaded that book a while but never read. I'm gonna start reading right away. Thanks a lot for telling me. Two more questions about open-iscsi implementaion though.. 1. As you mentioned in a previous reply, open-iscsi only supports

Re: scatterlist size of 4096

2010-05-05 Thread Mike Christie
On 05/03/2010 06:51 AM, Jack Z wrote: Hi group, I have been tracing the code related to sending PDUs from iscsi initiator (ver 2.0-871). And through some printk()s i realize that starting from iscsi_sw_tcp_pdu_init(), all the functions using scatterlist (struct scatterlist *sg) seem to use

Re: scatterlist size of 4096

2010-05-05 Thread Jack Z
Hi Mike, Thank you for your help again. Following the guidance in your reply, I traced the kernel code a bit more and eventually found out a possible path for open-iscsi to get 4K pages in the scatterlist. Kernel version: 2.6.30 open-iscsi version: 2.0.871 Trace 1: SCSI from a write request to

Re: scatterlist size of 4096

2010-05-05 Thread Jack Z
Hi Mike, Just got two more questions... First one is about the scsi_host_template-use_clustering flag, is there any specific reason why it is disabled in open-iscsi? Second one is the way we use sendpage() in iscsi_tcp.c. I was wondering can we maybe send more than one pages per sendpage()