Re: [PATCH v2 1/1] ixgbe: sync the first fragment unconditionally

2019-08-07 Thread Alexander Duyck
On Wed, Aug 7, 2019 at 6:58 PM Jacob Wen wrote: > > > On 8/7/19 4:38 PM, Firo Yang wrote: > > The 08/07/2019 15:56, Jacob Wen wrote: > >> I think the description is not correct. Consider using something like > >> below. > > Thank you for comments. > > > >> In Xen environment, due to memory fragme

Re: [PATCH v2 1/1] ixgbe: sync the first fragment unconditionally

2019-08-07 Thread Jacob Wen
On 8/7/19 4:38 PM, Firo Yang wrote: The 08/07/2019 15:56, Jacob Wen wrote: I think the description is not correct. Consider using something like below. Thank you for comments. In Xen environment, due to memory fragmentation ixgbe may allocate a 'DMA' buffer with pages that are not physicall

Re: [PATCH v2 1/1] ixgbe: sync the first fragment unconditionally

2019-08-07 Thread Firo Yang
The 08/07/2019 15:56, Jacob Wen wrote: > I think the description is not correct. Consider using something like below. Thank you for comments. > > In Xen environment, due to memory fragmentation ixgbe may allocate a 'DMA' > buffer with pages that are not physically contiguous. Actually, I didn't

Re: [PATCH v2 1/1] ixgbe: sync the first fragment unconditionally

2019-08-07 Thread Jacob Wen
I think the description is not correct. Consider using something like below. In Xen environment, due to memory fragmentation ixgbe may allocate a 'DMA' buffer with pages that are not physically contiguous. A NIC doesn't support directly write such buffer. So xen-swiotlb would use the pages,