On 12/05/2017 04:31 AM, Peter Zijlstra wrote:

On Mon, Dec 04, 2017 at 02:58:25PM -0800, Tejun Heo wrote:

quoted

Hello, again.

On Mon, Dec 04, 2017 at 12:22:19PM -0800, Tejun Heo wrote:

quoted

Hello,

On Mon, Dec 04, 2017 at 10:44:49AM +0530, Prateek Sood wrote:

quoted

Any feedback/suggestion for this patch?

Sorry about the delay.  I'm a bit worried because it feels like we're
chasing a squirrel.  I'll think through the recent changes and this
one and get back to you.

Can you please take a look at the following pending commit?

  https://git.kernel.org/pub/scm/linux/kernel/git/tj/wq.git/commit/?h=for-4.15-fixes&id=e8b3f8db7aad99fcc5234fc5b89984ff6620de3d

AFAICS, this should remove the circular dependency you originally
reported.  I'll revert the two cpuset commits for now.

So I liked his patches in that we would be able to go back to
synchronous sched_domain building.

https://git.kernel.org/pub/scm/linux/kernel/git/tj/wq.git/commit/?h=for-4.15-fixes&id=e8b3f8db7aad99fcc5234fc5b89984ff6620de3d

This will fix the original circular locking dependency issue.
I will let you both (Peter & TJ) to decide on which one to
pick.



-- 
Qualcomm India Private Limited, on behalf of Qualcomm Innovation
Center, Inc., is a member of Code Aurora Forum, a Linux Foundation
Collaborative Project