Received: by 2002:ad5:474a:0:0:0:0:0 with SMTP id i10csp6571116imu; Mon, 21 Jan 2019 11:15:32 -0800 (PST) X-Google-Smtp-Source: ALg8bN7K7pxmXtEXdVW8IM0M/4gQSL64Z4aAM1LrwuAP6t21NzLG5WvbMHgXxecAN8U4tN3Eg3Ix X-Received: by 2002:a62:c28e:: with SMTP id w14mr30759896pfk.115.1548098132852; Mon, 21 Jan 2019 11:15:32 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1548098132; cv=none; d=google.com; s=arc-20160816; b=Vvs3kLA9ryrOp7UUkjnasE+QOxsVengWkkTWrSOpHDcV9t8d9Wzbp7hdrXwQJOgkDj T7YyUff+1STsIA7uzL3vgaCgijR9vNz2lp+fRPbXmU8O6/PT6mB1f2fiPLR0puNmdhz6 pawCqLjb53Qh1XH/HWAAyJJ+CEjH9smRy6v1pZc7NzmkJ9r2hui4ME9Ut/kjb+AE6BDQ lq0L0rHK+4YxbThkjOo5p2wgBvq5NC9gSmkOVD5NuVaB+38FGDyXq3h2egxFrxCgRn8w FzFaw5oqf8qEGyXlwrMkEAXCgXGUkRb1DEbfUfxdmVGCKAab0Z97zohrsUG7eaYVpEQQ rSmw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-language :content-transfer-encoding:in-reply-to:mime-version:user-agent:date :message-id:references:cc:to:from:subject:dkim-signature; bh=aVzxV0IkABr8yRf2yZfqdYegrJVqmtk73WWhqLWtCGQ=; b=XA4Lb5U3PrCnLs+gYz0WNhZeehDsiB6Ajw+emuwgMTMKOaDS3wEYoAqjOqfP9zez5S NpzRnh/81oPstHn4vlnM3AHNF8a4yooM2WyAnMxVlZyY2Pbd8iWjC8qsiyA6laExf80f WtOOa8iAWJtOJY7V4AB6NHDdp5aV300SFROKwOlOChOHdqn2AIipxdqvGmwnTYEuw1lm 1YG3T2HaOXcZIY8hmFItXD9jAS4ISUPmdPW7s1fDfJHVtqVHRQoEqnBhZHBVdaWPgGBw c61jCcF6a3SiEEC5IzRD2DipJBNk8+wX/bYufy2K603Cgh5hZEaPVqkE0RF72CV1fkkx Phhg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=Hl6VTekQ; 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; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id u7si13013808pgg.357.2019.01.21.11.15.17; Mon, 21 Jan 2019 11:15:32 -0800 (PST) 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; dkim=pass header.i=@gmail.com header.s=20161025 header.b=Hl6VTekQ; 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; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728056AbfAUSqM (ORCPT + 99 others); Mon, 21 Jan 2019 13:46:12 -0500 Received: from mail-wr1-f66.google.com ([209.85.221.66]:44819 "EHLO mail-wr1-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726353AbfAUSqM (ORCPT ); Mon, 21 Jan 2019 13:46:12 -0500 Received: by mail-wr1-f66.google.com with SMTP id z5so24549932wrt.11; Mon, 21 Jan 2019 10:46:10 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=subject:from:to:cc:references:message-id:date:user-agent :mime-version:in-reply-to:content-transfer-encoding:content-language; bh=aVzxV0IkABr8yRf2yZfqdYegrJVqmtk73WWhqLWtCGQ=; b=Hl6VTekQdfeOZTZsQB1JbhW1iMEjd1YDOZx4qSSic6Wh2ubbZsjh8qKNXow2/gjkES E0eNADnN9/m6+nfkT7wm1ACSYuEabYyLjA6EMJ6VRV9HR26vMmxcf96icQaymY+Zj8XJ y/7F/QC+RRks5x86dt209UcDxxlBSKUEA3sRVA1NzIxIHg+8MoRunguiCQkMO0xylJS+ /nHyQ6L6t3hYgawVQwJNhFqLeY2V+6PTmcOK4WpYN5knTqOkwY8fwVxJ2sNK53MpDik/ +4XzK4Fa61p77ueuETqfgW7EAEvXWKNG9COjCcyiLNlUp/KgE9YDWAXHNimnsEtJ4rF6 NDyg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:from:to:cc:references:message-id:date :user-agent:mime-version:in-reply-to:content-transfer-encoding :content-language; bh=aVzxV0IkABr8yRf2yZfqdYegrJVqmtk73WWhqLWtCGQ=; b=aFCO65lONuKEle1MEwDGEzOryWv+4XZA6RsDMR97P4kzwiai/M1+DtHEufZMrpAEkU SBx9Afq4Ty3keDHoJUMPcqSaog7AMG8PcZkLmNZcPEMTn1iDxO98JZjX89rZzfwygVSw Iyt6C5l9N+hixGRVgLE4goYIkfAlCR7eV36BviJCw3iKlsEmEQBQ2rCa3YSw5IXG7TcW LG8FuMHY/qL7LgPR7IQ6xIRe+iTYb+0H119G0G+wIg7Ns3ZiyrDcAAw4boyEuOexalwk QCQuosxxZU7mciISCmeFk7Rbgn+b45UseSWe9Oc6hSYTwBFAxCAam3fWt5KxZ++nt28I h+DQ== X-Gm-Message-State: AJcUukfZLZG9zhlH4gwcehr06IKDb7H1vMyrxAe+QReKf8YEzQvxXPUH pbmFR293ZTEGnHiF5iKQuFvfjkS/ X-Received: by 2002:adf:fbc8:: with SMTP id d8mr28973974wrs.318.1548096369558; Mon, 21 Jan 2019 10:46:09 -0800 (PST) Received: from [172.30.88.24] (sjewanfw1-nat.mentorg.com. [139.181.7.34]) by smtp.gmail.com with ESMTPSA id e27sm109694616wra.67.2019.01.21.10.46.07 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 21 Jan 2019 10:46:09 -0800 (PST) Subject: Re: [PATCH v3 1/2] media: imx: csi: Disable SMFC before disabling IDMA channel From: Steve Longerbeam To: Philipp Zabel , linux-media@vger.kernel.org Cc: Gael PORTAY , Peter Seiderer , stable@vger.kernel.org, Mauro Carvalho Chehab , Greg Kroah-Hartman , "open list:STAGING SUBSYSTEM" , open list References: <20190119010457.2623-1-slongerbeam@gmail.com> <20190119010457.2623-2-slongerbeam@gmail.com> <1548071350.3287.3.camel@pengutronix.de> <7432d18b-12fc-34c6-832f-576fc1b8e2e8@gmail.com> Message-ID: <12937f2b-e25e-6cc5-0727-59a5e6224fd9@gmail.com> Date: Mon, 21 Jan 2019 10:46:05 -0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.2.1 MIME-Version: 1.0 In-Reply-To: <7432d18b-12fc-34c6-832f-576fc1b8e2e8@gmail.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 1/21/19 10:43 AM, Steve Longerbeam wrote: > > > On 1/21/19 3:49 AM, Philipp Zabel wrote: >> Also ipu_smfc_disable is refcounted, so if the other CSI is capturing >> simultaneously, this change has no effect. > > Sigh, you're right. Let me go back to disabling the CSI before the > channel, the CSI enable/disable is not refcounted (it doesn't need to > be since it is single use) so it doesn't have this problem. > > Should we drop this patch or keep it (with a big comment)? By only > changing the disable order to "CSI then channel", the hang is reliably > fixed from my and Gael's testing, but my concern is that by not > disabling the SMFC before the channel, the SMFC could still empty its > FIFO to the channel's internal FIFO and still create a hang. Well, as you said it will have no effect if both CSI's are streaming with the SMFC, in which case the danger would still exist. Perhaps it would be best to just drop this patch. Steve