Re: [RFC 1/1] pmem: Add cond_resched() in bio_for_each_segment loop in pmem_make_request

2020-08-07 Thread Ritesh Harjani
On 8/4/20 3:21 AM, Dave Chinner wrote: On Mon, Aug 03, 2020 at 12:44:04PM +0530, Ritesh Harjani wrote: On 8/3/20 4:31 AM, Dave Chinner wrote: On Wed, Jul 29, 2020 at 02:15:18PM +0530, Ritesh Harjani wrote: For systems which do not have CONFIG_PREEMPT set and if there is a heavy multi-thre

Re: [RFC 1/1] pmem: Add cond_resched() in bio_for_each_segment loop in pmem_make_request

2020-08-03 Thread Dave Chinner
On Mon, Aug 03, 2020 at 12:44:04PM +0530, Ritesh Harjani wrote: > > > On 8/3/20 4:31 AM, Dave Chinner wrote: > > On Wed, Jul 29, 2020 at 02:15:18PM +0530, Ritesh Harjani wrote: > > > For systems which do not have CONFIG_PREEMPT set and > > > if there is a heavy multi-threaded load/store operation

Re: [RFC 1/1] pmem: Add cond_resched() in bio_for_each_segment loop in pmem_make_request

2020-08-03 Thread Ritesh Harjani
On 8/3/20 4:31 AM, Dave Chinner wrote: On Wed, Jul 29, 2020 at 02:15:18PM +0530, Ritesh Harjani wrote: For systems which do not have CONFIG_PREEMPT set and if there is a heavy multi-threaded load/store operation happening on pmem + sometimes along with device latencies, softlockup warnings li

Re: [RFC 1/1] pmem: Add cond_resched() in bio_for_each_segment loop in pmem_make_request

2020-08-02 Thread Dave Chinner
On Wed, Jul 29, 2020 at 02:15:18PM +0530, Ritesh Harjani wrote: > For systems which do not have CONFIG_PREEMPT set and > if there is a heavy multi-threaded load/store operation happening > on pmem + sometimes along with device latencies, softlockup warnings like > this could trigger. This was seen

[RFC 1/1] pmem: Add cond_resched() in bio_for_each_segment loop in pmem_make_request

2020-07-29 Thread Ritesh Harjani
For systems which do not have CONFIG_PREEMPT set and if there is a heavy multi-threaded load/store operation happening on pmem + sometimes along with device latencies, softlockup warnings like this could trigger. This was seen on Power where pagesize is 64K. To avoid softlockup, this patch adds a