Re: [PATCH] ib/core: not to set page dirty bit if it's already set.

2017-06-01 Thread Doug Ledford
On Thu, 2017-05-18 at 16:33 -0700, Qing Huang wrote: > This change will optimize kernel memory deregistration operations. > __ib_umem_release() used to call set_page_dirty_lock() against every > writable page in its memory region. Its purpose is to keep data > synced between CPU and DMA device

Re: [PATCH] ib/core: not to set page dirty bit if it's already set.

2017-06-01 Thread Doug Ledford
On Thu, 2017-05-18 at 16:33 -0700, Qing Huang wrote: > This change will optimize kernel memory deregistration operations. > __ib_umem_release() used to call set_page_dirty_lock() against every > writable page in its memory region. Its purpose is to keep data > synced between CPU and DMA device

Re: [PATCH] ib/core: not to set page dirty bit if it's already set.

2017-05-23 Thread Qing Huang
On 5/23/2017 12:42 AM, Christoph Hellwig wrote: On Mon, May 22, 2017 at 04:43:57PM -0700, Qing Huang wrote: On 5/19/2017 6:05 AM, Christoph Hellwig wrote: On Thu, May 18, 2017 at 04:33:53PM -0700, Qing Huang wrote: This change will optimize kernel memory deregistration operations.

Re: [PATCH] ib/core: not to set page dirty bit if it's already set.

2017-05-23 Thread Qing Huang
On 5/23/2017 12:42 AM, Christoph Hellwig wrote: On Mon, May 22, 2017 at 04:43:57PM -0700, Qing Huang wrote: On 5/19/2017 6:05 AM, Christoph Hellwig wrote: On Thu, May 18, 2017 at 04:33:53PM -0700, Qing Huang wrote: This change will optimize kernel memory deregistration operations.

Re: [PATCH] ib/core: not to set page dirty bit if it's already set.

2017-05-23 Thread Christoph Hellwig
On Mon, May 22, 2017 at 04:43:57PM -0700, Qing Huang wrote: > > On 5/19/2017 6:05 AM, Christoph Hellwig wrote: > > On Thu, May 18, 2017 at 04:33:53PM -0700, Qing Huang wrote: > > > This change will optimize kernel memory deregistration operations. > > > __ib_umem_release() used to call

Re: [PATCH] ib/core: not to set page dirty bit if it's already set.

2017-05-23 Thread Christoph Hellwig
On Mon, May 22, 2017 at 04:43:57PM -0700, Qing Huang wrote: > > On 5/19/2017 6:05 AM, Christoph Hellwig wrote: > > On Thu, May 18, 2017 at 04:33:53PM -0700, Qing Huang wrote: > > > This change will optimize kernel memory deregistration operations. > > > __ib_umem_release() used to call

Re: [PATCH] ib/core: not to set page dirty bit if it's already set.

2017-05-22 Thread Qing Huang
On 5/19/2017 6:05 AM, Christoph Hellwig wrote: On Thu, May 18, 2017 at 04:33:53PM -0700, Qing Huang wrote: This change will optimize kernel memory deregistration operations. __ib_umem_release() used to call set_page_dirty_lock() against every writable page in its memory region. Its purpose is

Re: [PATCH] ib/core: not to set page dirty bit if it's already set.

2017-05-22 Thread Qing Huang
On 5/19/2017 6:05 AM, Christoph Hellwig wrote: On Thu, May 18, 2017 at 04:33:53PM -0700, Qing Huang wrote: This change will optimize kernel memory deregistration operations. __ib_umem_release() used to call set_page_dirty_lock() against every writable page in its memory region. Its purpose is

Re: [PATCH] ib/core: not to set page dirty bit if it's already set.

2017-05-19 Thread Christoph Hellwig
On Thu, May 18, 2017 at 04:33:53PM -0700, Qing Huang wrote: > This change will optimize kernel memory deregistration operations. > __ib_umem_release() used to call set_page_dirty_lock() against every > writable page in its memory region. Its purpose is to keep data > synced between CPU and DMA

Re: [PATCH] ib/core: not to set page dirty bit if it's already set.

2017-05-19 Thread Christoph Hellwig
On Thu, May 18, 2017 at 04:33:53PM -0700, Qing Huang wrote: > This change will optimize kernel memory deregistration operations. > __ib_umem_release() used to call set_page_dirty_lock() against every > writable page in its memory region. Its purpose is to keep data > synced between CPU and DMA

[PATCH] ib/core: not to set page dirty bit if it's already set.

2017-05-18 Thread Qing Huang
This change will optimize kernel memory deregistration operations. __ib_umem_release() used to call set_page_dirty_lock() against every writable page in its memory region. Its purpose is to keep data synced between CPU and DMA device when swapping happens after mem deregistration ops. Now we

[PATCH] ib/core: not to set page dirty bit if it's already set.

2017-05-18 Thread Qing Huang
This change will optimize kernel memory deregistration operations. __ib_umem_release() used to call set_page_dirty_lock() against every writable page in its memory region. Its purpose is to keep data synced between CPU and DMA device when swapping happens after mem deregistration ops. Now we