Re: [PATCH v2 bpf-next] bpf: increment and use correct thread iterator

2020-12-13 Thread Yonghong Song
On 12/11/20 8:30 AM, Jonathan Lemon wrote: On Wed, Dec 09, 2020 at 11:02:54AM -0800, Yonghong Song wrote: Maybe you can post v3 of the patch with the above information in the commit description so people can better understand what the problem you are trying to solve here? Also, could you a

Re: [PATCH v2 bpf-next] bpf: increment and use correct thread iterator

2020-12-11 Thread Jonathan Lemon
On Wed, Dec 09, 2020 at 11:02:54AM -0800, Yonghong Song wrote: > > > Maybe you can post v3 of the patch with the above information in the > commit description so people can better understand what the problem > you are trying to solve here? > > Also, could you also send to b...@vger.kernel.org?

Re: [PATCH v2 bpf-next] bpf: increment and use correct thread iterator

2020-12-09 Thread Yonghong Song
On 12/4/20 9:14 AM, Jonathan Lemon wrote: On Fri, Dec 04, 2020 at 12:01:53AM -0800, Yonghong Song wrote: On 12/3/20 7:43 PM, Jonathan Lemon wrote: From: Jonathan Lemon Could you explain in the commit log what problem this patch tries to solve? What bad things could happen without this p

Re: [PATCH v2 bpf-next] bpf: increment and use correct thread iterator

2020-12-04 Thread Yonghong Song
On 12/4/20 9:14 AM, Jonathan Lemon wrote: On Fri, Dec 04, 2020 at 12:01:53AM -0800, Yonghong Song wrote: On 12/3/20 7:43 PM, Jonathan Lemon wrote: From: Jonathan Lemon Could you explain in the commit log what problem this patch tries to solve? What bad things could happen without this p

Re: [PATCH v2 bpf-next] bpf: increment and use correct thread iterator

2020-12-04 Thread Jonathan Lemon
On Fri, Dec 04, 2020 at 12:01:53AM -0800, Yonghong Song wrote: > > > On 12/3/20 7:43 PM, Jonathan Lemon wrote: > > From: Jonathan Lemon > > Could you explain in the commit log what problem this patch > tries to solve? What bad things could happen without this patch? Without the patch, on a par

Re: [PATCH v2 bpf-next] bpf: increment and use correct thread iterator

2020-12-04 Thread Yonghong Song
On 12/3/20 7:43 PM, Jonathan Lemon wrote: From: Jonathan Lemon Could you explain in the commit log what problem this patch tries to solve? What bad things could happen without this patch? If unable to obtain the file structure for the current task, proceed to the next task number after t

[PATCH v2 bpf-next] bpf: increment and use correct thread iterator

2020-12-03 Thread Jonathan Lemon
From: Jonathan Lemon If unable to obtain the file structure for the current task, proceed to the next task number after the one returned from task_seq_get_next(), instead of the next task number from the original iterator. Use thread_group_leader() instead of comparing tgid vs pid, which might m