Re: Re: Re: [Pv-drivers] 3.7-rc2 regression : file copied to CIFS-mounted directory corrupted

2012-10-28 Thread Jongman Heo
--- Original Message --- Sender : Jongman Heo Date : 2012-10-24 11:53 (GMT+09:00) Title : Re: Re: [Pv-drivers] 3.7-rc2 regression : file copied to CIFS-mounted directory corrupted Hi, --- Original Message --- Sender : Eric Dumazet Date : 2012-10-24 04:39 (GMT+09:00) Title : Re

Re: Re: Re: [Pv-drivers] 3.7-rc2 regression : file copied to CIFS-mounted directory corrupted

2012-10-28 Thread Jongman Heo
--- Original Message --- Sender : Jongman Heojongman@samsung.com Date : 2012-10-24 11:53 (GMT+09:00) Title : Re: Re: [Pv-drivers] 3.7-rc2 regression : file copied to CIFS-mounted directory corrupted Hi, --- Original Message --- Sender : Eric Dumazet Date : 2012-10-24 04:39

Re: Re: [Pv-drivers] 3.7-rc2 regression : file copied to CIFS-mounted directory corrupted

2012-10-23 Thread Jongman Heo
Hi, --- Original Message --- Sender : Eric Dumazet Date : 2012-10-24 04:39 (GMT+09:00) Title : Re: [Pv-drivers] 3.7-rc2 regression : file copied to CIFS-mounted directory corrupted On Tue, 2012-10-23 at 15:50 +0200, Eric Dumazet wrote: > Only the skb head is handled in the code

Re: [Pv-drivers] 3.7-rc2 regression : file copied to CIFS-mounted directory corrupted

2012-10-23 Thread Eric Dumazet
On Tue, 2012-10-23 at 15:50 +0200, Eric Dumazet wrote: > Only the skb head is handled in the code you copy/pasted. > > You need to generalize that to code in lines ~754 > > > Then, the number of estimated descriptors is bad : > > /* conservatively estimate # of descriptors to use */ > count =

Re: [Pv-drivers] 3.7-rc2 regression : file copied to CIFS-mounted directory corrupted

2012-10-23 Thread Eric Dumazet
On Tue, 2012-10-23 at 03:02 -0700, Shreyas Bhatewara wrote: Please dont top post on netdev or lkml > Well, actually the driver does split large frags into frags of > VMXNET3_MAX_TX_BUF_SIZE bytes each. > > vmxnet3_drv.c > 711 while (len) { > 712 u32 buf_size; > 713 >

Re: [Pv-drivers] 3.7-rc2 regression : file copied to CIFS-mounted directory corrupted

2012-10-23 Thread Shreyas Bhatewara
Well, actually the driver does split large frags into frags of VMXNET3_MAX_TX_BUF_SIZE bytes each. vmxnet3_drv.c 711 while (len) { 712 u32 buf_size; 713 714 if (len < VMXNET3_MAX_TX_BUF_SIZE) { 715 buf_size = len; 716

Re: [Pv-drivers] 3.7-rc2 regression : file copied to CIFS-mounted directory corrupted

2012-10-23 Thread Shreyas Bhatewara
Well, actually the driver does split large frags into frags of VMXNET3_MAX_TX_BUF_SIZE bytes each. vmxnet3_drv.c 711 while (len) { 712 u32 buf_size; 713 714 if (len VMXNET3_MAX_TX_BUF_SIZE) { 715 buf_size = len; 716

Re: [Pv-drivers] 3.7-rc2 regression : file copied to CIFS-mounted directory corrupted

2012-10-23 Thread Eric Dumazet
On Tue, 2012-10-23 at 03:02 -0700, Shreyas Bhatewara wrote: Please dont top post on netdev or lkml Well, actually the driver does split large frags into frags of VMXNET3_MAX_TX_BUF_SIZE bytes each. vmxnet3_drv.c 711 while (len) { 712 u32 buf_size; 713 714

Re: [Pv-drivers] 3.7-rc2 regression : file copied to CIFS-mounted directory corrupted

2012-10-23 Thread Eric Dumazet
On Tue, 2012-10-23 at 15:50 +0200, Eric Dumazet wrote: Only the skb head is handled in the code you copy/pasted. You need to generalize that to code in lines ~754 Then, the number of estimated descriptors is bad : /* conservatively estimate # of descriptors to use */ count =

Re: Re: [Pv-drivers] 3.7-rc2 regression : file copied to CIFS-mounted directory corrupted

2012-10-23 Thread Jongman Heo
Hi, --- Original Message --- Sender : Eric Dumazet Date : 2012-10-24 04:39 (GMT+09:00) Title : Re: [Pv-drivers] 3.7-rc2 regression : file copied to CIFS-mounted directory corrupted On Tue, 2012-10-23 at 15:50 +0200, Eric Dumazet wrote: Only the skb head is handled in the code you copy