Received: by 2002:a25:ad19:0:0:0:0:0 with SMTP id y25csp1686095ybi; Fri, 12 Jul 2019 22:04:34 -0700 (PDT) X-Google-Smtp-Source: APXvYqx2tGHV+QlytvMFcRgdLl9QCGVd2uUhHejVV2GqWoHE4Nui0Giko4PJ6szigLmbowhU4h2N X-Received: by 2002:a17:90a:2247:: with SMTP id c65mr15790417pje.24.1562994274429; Fri, 12 Jul 2019 22:04:34 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1562994274; cv=none; d=google.com; s=arc-20160816; b=VQ7q8yZUe+fg8aBwRu6C0b7jBB3+c48JdtvE0QNwmCzeI2+wM3vPJOm7G8xffu2956 sC1KGPV5+6traSPng2HkSdyDCTWjahgjB+gHCBxHOFG051AyLg3rIKiq3pwJST2G1A2y Gi+Ityn81mYvr8Uz4C1fMbFbxuX6nVhtR+oxQyWzZYj5sGh9CiQPfkdN69d3cGe+KY5l bRdZg/DftfzYWIv2jm4vyaxzlmkJ82+JloO6ZXGxQ+7iz35bK+IgBY5a/bspNKRs4U3W TlsHCr/a0V4eHqH7V/cHbEncWLonJ6Mbvrpt6nBrZBZl5yA5oJ0nPRPJDv21jpS+MEqI 495w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date; bh=IjGJQqR6yW8aVC/XU4cm6gqRoSXx4RgOaDrwW/MwheA=; b=MGBjCMOLOwfufYAFtA3oTtXEXyK5IHyeJjkasRHWv9a3VXTCKW/GLft6FVAB9VFEFe OHAQ71L3Tnu9ifzEhvf2704cIz/CfqP+8Yh70smdKRjzKppRVmXeduZhxU+DKjqhuFy9 Yqh3dt5cKTgZa5SSxq4L+6FZZo8l4VYFA3oj9mW4Mt/Z2iZCw+JSX/pAxkjz/gNVjk9E E7+Y7Mnxu2+PZLjg53DxUbr2rDcnmrCbGoxOVXWpL5yo7lmH3XhR4LeJ+BDo6UdQH/YE 4IE1sKR4JqvaVwcj0OQWMYxlUqgQ6/uFS7Wg2HgGLDTeRt44Io5TXwasP5PWkeb8TFKO lQwg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id v23si10204706pgn.557.2019.07.12.22.04.07; Fri, 12 Jul 2019 22:04:34 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726453AbfGMFDk (ORCPT + 99 others); Sat, 13 Jul 2019 01:03:40 -0400 Received: from helcar.hmeau.com ([216.24.177.18]:48170 "EHLO deadmen.hmeau.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726301AbfGMFDj (ORCPT ); Sat, 13 Jul 2019 01:03:39 -0400 Received: from gondobar.mordor.me.apana.org.au ([192.168.128.4] helo=gondobar) by deadmen.hmeau.com with esmtps (Exim 4.89 #2 (Debian)) id 1hmAC0-0006BY-Hm; Sat, 13 Jul 2019 13:03:28 +0800 Received: from herbert by gondobar with local (Exim 4.89) (envelope-from ) id 1hmABt-0000yf-KY; Sat, 13 Jul 2019 13:03:21 +0800 Date: Sat, 13 Jul 2019 13:03:21 +0800 From: Herbert Xu To: Daniel Jordan Cc: Steffen Klassert , andrea.parri@amarulasolutions.com, boqun.feng@gmail.com, paulmck@linux.ibm.com, peterz@infradead.org, linux-arch@vger.kernel.org, linux-crypto@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] padata: use smp_mb in padata_reorder to avoid orphaned padata jobs Message-ID: <20190713050321.c5wq7a7jrb6q2pxn@gondor.apana.org.au> References: <20190711221205.29889-1-daniel.m.jordan@oracle.com> <20190712100636.mqdr567p7ozanlyl@gondor.apana.org.au> <20190712101012.GW14601@gauss3.secunet.de> <20190712160737.iniaaxlsnhs6azg5@ca-dmjordan1.us.oracle.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190712160737.iniaaxlsnhs6azg5@ca-dmjordan1.us.oracle.com> User-Agent: NeoMutt/20170113 (1.7.2) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Jul 12, 2019 at 12:07:37PM -0400, Daniel Jordan wrote: > > modprobe (CPU2) kworker/21:1-293 (CPU21) kworker/5:2-276 (CPU5) > -------------------------- ------------------------ ---------------------- > > ... > padata_do_parallel > queue_work_on(21, ...) > > padata_parallel_worker > pcrypt_aead_dec > padata_do_serial > padata_reorder This can't happen because if the job started on CPU2 then it must go back to CPU2 for completion. IOW padata_do_serial should be punting this to a work queue for CPU2 rather than calling padata_reorder on CPU21. Cheers, -- Email: Herbert Xu Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt