Re: [PATCH] ring-buffer: Allow for rescheduling when removing pages

2018-09-17 Thread Vaibhav Nagarnaik
On Mon, Sep 17, 2018 at 2:36 PM Steven Rostedt wrote: > > On Mon, 17 Sep 2018 14:02:59 -0700 > Vaibhav Nagarnaik wrote: > > > On Mon, Sep 17, 2018 at 2:01 PM Steven Rostedt wrote: > > > > > > On Mon, 17 Sep 2018 13:53:05 -0700 > > > Vaibhav Nagarnaik wrote: > > > > > > > Hi Steven, > > > > > >

Re: [PATCH] ring-buffer: Allow for rescheduling when removing pages

2018-09-17 Thread Vaibhav Nagarnaik
On Mon, Sep 17, 2018 at 2:36 PM Steven Rostedt wrote: > > On Mon, 17 Sep 2018 14:02:59 -0700 > Vaibhav Nagarnaik wrote: > > > On Mon, Sep 17, 2018 at 2:01 PM Steven Rostedt wrote: > > > > > > On Mon, 17 Sep 2018 13:53:05 -0700 > > > Vaibhav Nagarnaik wrote: > > > > > > > Hi Steven, > > > > > >

Re: [PATCH] ring-buffer: Allow for rescheduling when removing pages

2018-09-17 Thread Steven Rostedt
On Mon, 17 Sep 2018 14:02:59 -0700 Vaibhav Nagarnaik wrote: > On Mon, Sep 17, 2018 at 2:01 PM Steven Rostedt wrote: > > > > On Mon, 17 Sep 2018 13:53:05 -0700 > > Vaibhav Nagarnaik wrote: > > > > > Hi Steven, > > > > > > Does the patch look good? Can this be picked up in the next rc? > > >

Re: [PATCH] ring-buffer: Allow for rescheduling when removing pages

2018-09-17 Thread Steven Rostedt
On Mon, 17 Sep 2018 14:02:59 -0700 Vaibhav Nagarnaik wrote: > On Mon, Sep 17, 2018 at 2:01 PM Steven Rostedt wrote: > > > > On Mon, 17 Sep 2018 13:53:05 -0700 > > Vaibhav Nagarnaik wrote: > > > > > Hi Steven, > > > > > > Does the patch look good? Can this be picked up in the next rc? > > >

Re: [PATCH] ring-buffer: Allow for rescheduling when removing pages

2018-09-17 Thread Vaibhav Nagarnaik
On Mon, Sep 17, 2018 at 2:01 PM Steven Rostedt wrote: > > On Mon, 17 Sep 2018 13:53:05 -0700 > Vaibhav Nagarnaik wrote: > > > Hi Steven, > > > > Does the patch look good? Can this be picked up in the next rc? > > > > Yes it's fine. I can pick it up. Does it need to be marked for stable? Thanks.

Re: [PATCH] ring-buffer: Allow for rescheduling when removing pages

2018-09-17 Thread Vaibhav Nagarnaik
On Mon, Sep 17, 2018 at 2:01 PM Steven Rostedt wrote: > > On Mon, 17 Sep 2018 13:53:05 -0700 > Vaibhav Nagarnaik wrote: > > > Hi Steven, > > > > Does the patch look good? Can this be picked up in the next rc? > > > > Yes it's fine. I can pick it up. Does it need to be marked for stable? Thanks.

Re: [PATCH] ring-buffer: Allow for rescheduling when removing pages

2018-09-17 Thread Steven Rostedt
On Mon, 17 Sep 2018 13:53:05 -0700 Vaibhav Nagarnaik wrote: > Hi Steven, > > Does the patch look good? Can this be picked up in the next rc? > Yes it's fine. I can pick it up. Does it need to be marked for stable? Thanks! -- Steve

Re: [PATCH] ring-buffer: Allow for rescheduling when removing pages

2018-09-17 Thread Steven Rostedt
On Mon, 17 Sep 2018 13:53:05 -0700 Vaibhav Nagarnaik wrote: > Hi Steven, > > Does the patch look good? Can this be picked up in the next rc? > Yes it's fine. I can pick it up. Does it need to be marked for stable? Thanks! -- Steve

Re: [PATCH] ring-buffer: Allow for rescheduling when removing pages

2018-09-17 Thread Vaibhav Nagarnaik
Hi Steven, Does the patch look good? Can this be picked up in the next rc? Vaibhav On Fri, Sep 7, 2018 at 3:31 PM Vaibhav Nagarnaik wrote: > > When reducing ring buffer size, pages are removed by scheduling a work > item on each CPU for the corresponding CPU ring buffer. After the pages > are

Re: [PATCH] ring-buffer: Allow for rescheduling when removing pages

2018-09-17 Thread Vaibhav Nagarnaik
Hi Steven, Does the patch look good? Can this be picked up in the next rc? Vaibhav On Fri, Sep 7, 2018 at 3:31 PM Vaibhav Nagarnaik wrote: > > When reducing ring buffer size, pages are removed by scheduling a work > item on each CPU for the corresponding CPU ring buffer. After the pages > are

[PATCH] ring-buffer: Allow for rescheduling when removing pages

2018-09-07 Thread Vaibhav Nagarnaik
When reducing ring buffer size, pages are removed by scheduling a work item on each CPU for the corresponding CPU ring buffer. After the pages are removed from ring buffer linked list, the pages are free()d in a tight loop. The loop does not give up CPU until all pages are removed. In a worst case

[PATCH] ring-buffer: Allow for rescheduling when removing pages

2018-09-07 Thread Vaibhav Nagarnaik
When reducing ring buffer size, pages are removed by scheduling a work item on each CPU for the corresponding CPU ring buffer. After the pages are removed from ring buffer linked list, the pages are free()d in a tight loop. The loop does not give up CPU until all pages are removed. In a worst case

Re: [PATCH] ring-buffer: Allow for rescheduling when removing pages

2018-09-07 Thread Vaibhav Nagarnaik
On Fri, Sep 7, 2018 at 11:30 AM Steven Rostedt wrote: > > On Fri, 7 Sep 2018 11:21:31 -0700 > Vaibhav Nagarnaik wrote: > > > When reducing ring buffer size, pages are removed by scheduling a work > > item on each CPU for the corresponding CPU ring buffer. After the pages > > are removed from

Re: [PATCH] ring-buffer: Allow for rescheduling when removing pages

2018-09-07 Thread Vaibhav Nagarnaik
On Fri, Sep 7, 2018 at 11:30 AM Steven Rostedt wrote: > > On Fri, 7 Sep 2018 11:21:31 -0700 > Vaibhav Nagarnaik wrote: > > > When reducing ring buffer size, pages are removed by scheduling a work > > item on each CPU for the corresponding CPU ring buffer. After the pages > > are removed from

Re: [PATCH] ring-buffer: Allow for rescheduling when removing pages

2018-09-07 Thread Steven Rostedt
On Fri, 7 Sep 2018 11:21:31 -0700 Vaibhav Nagarnaik wrote: > When reducing ring buffer size, pages are removed by scheduling a work > item on each CPU for the corresponding CPU ring buffer. After the pages > are removed from ring buffer linked list, the pages are free()d in a > tight loop. The

Re: [PATCH] ring-buffer: Allow for rescheduling when removing pages

2018-09-07 Thread Steven Rostedt
On Fri, 7 Sep 2018 11:21:31 -0700 Vaibhav Nagarnaik wrote: > When reducing ring buffer size, pages are removed by scheduling a work > item on each CPU for the corresponding CPU ring buffer. After the pages > are removed from ring buffer linked list, the pages are free()d in a > tight loop. The

[PATCH] ring-buffer: Allow for rescheduling when removing pages

2018-09-07 Thread Vaibhav Nagarnaik
When reducing ring buffer size, pages are removed by scheduling a work item on each CPU for the corresponding CPU ring buffer. After the pages are removed from ring buffer linked list, the pages are free()d in a tight loop. The loop does not give up CPU until all pages are removed. In a worst case

[PATCH] ring-buffer: Allow for rescheduling when removing pages

2018-09-07 Thread Vaibhav Nagarnaik
When reducing ring buffer size, pages are removed by scheduling a work item on each CPU for the corresponding CPU ring buffer. After the pages are removed from ring buffer linked list, the pages are free()d in a tight loop. The loop does not give up CPU until all pages are removed. In a worst case