From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.9 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,FORGED_MUA_MOZILLA,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,MSGID_FROM_MTA_HEADER, SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 7CE3FC433E1 for ; Sat, 11 Jul 2020 18:19:44 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 4FA7220748 for ; Sat, 11 Jul 2020 18:19:44 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="LJBbY3gh" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 4FA7220748 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=live.ca Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:MIME-Version:In-Reply-To:Date:Message-ID:From: References:To:Subject:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=nScZHtS0aFX1abf0TRZZ9jShtUfA/8hBikPaIdCFC2Q=; b=LJBbY3ghoh0WytGYhtwVMAYFq 7+7/qGX2ohA9Cb7cABbZdNWi2WH14l2Idjr5EyIZ3S/Wtdg7aUyhRfsTtK8RdpyrHu6dL7x3rL4+0 PFOEmz/lnI4IRNTUzDDxlcPtjA3Dnl2Dm0ElMdCew6CFhoOyxtVbmhGRz19T29R3Q4e9NxaOq88ZR qfpaxODRKeLI8StfGAewqwAn+s49XzTMARfTTXK35JOYGx6dyJr3oyQ3B6V80K39tZMlyYWHe+mIk SOAsOt1yqNia+qOgC+mH5H0/KhWMqwibsFZKWQVt2ZHr+iS9bEs2b2RoaNvh5zjgR2Cp5KlrzgMXH 9uTKEaJXw==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1juK4Y-000086-82; Sat, 11 Jul 2020 18:18:02 +0000 Received: from mail-co1nam11olkn2047.outbound.protection.outlook.com ([40.92.18.47] helo=NAM11-CO1-obe.outbound.protection.outlook.com) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1juK4T-00007K-CF for linux-arm-kernel@lists.infradead.org; Sat, 11 Jul 2020 18:17:58 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=HTk73Qa19qOHWxeNdwz7hdCd2KP2N9mpX8mLKt0tAw8OyVruN7Vl120UZxPaNeK7EDrfNFrth/MB2Hl9cELLFXF2Z2DYNUbGo5Rmfubi70SZDCoV/L3iNUBIPWqt1Hqmn0I1NeZPDu5WCzoi7djjtmbToRZHqJwlAsQ7nPr6LjP0tidYfPiTxW4dd9R7EmVPWpIyskyXmy2bkTgzB2ZANzV1Xe1J17DCUeFsCJoOzA/cjKoLCe59GRNI5X/23HWN0OFEtCuMXjvxSTbyUvD/OMx8R4mf9ESX4A8LEIGQMvWakbQrqsDLEf64SptAqjdg4fwECSpV4ZY92+GSHGO6YQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=5VKSI+YIQzSPmFxzGzGhf93QH9PmL6oI18YwE5JqW/U=; b=BUjZzPcR8J0S4PfD5oj6b3cXEqa7OJsIMYxncRJSesJByCvdyATjn8Ex59e4cCXC5babEgdnd9kEkDKpKEa8wVENDH4C2Eh7NOgGHO3KaORsd0anXyKWEHU2Oz52DVIqK9G9WoH30ZYXloAyedjJiJvDjWGUhgiwqPQxYofYZl/YhuJwGQHP+hlkTk78l1fHQe72sL0Ve/KN06kFiT5bA4LblLDI9yIjKw16jLgcTopJEjXhBtgnnypvQYXDotk1RTPMmZsvptketMznaeAtxAYDr2EA5lvC0QBuX4QO91DEOwibBGLZVjPP+oTe7DMs8BtxrnyTZBdWg32p19NeVQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; arc=none Received: from DM6NAM11FT041.eop-nam11.prod.protection.outlook.com (2a01:111:e400:fc4d::4a) by DM6NAM11HT013.eop-nam11.prod.protection.outlook.com (2a01:111:e400:fc4d::340) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3174.21; Sat, 11 Jul 2020 18:17:54 +0000 Received: from BN6PR04MB0660.namprd04.prod.outlook.com (2a01:111:e400:fc4d::43) by DM6NAM11FT041.mail.protection.outlook.com (2a01:111:e400:fc4d::98) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3174.21 via Frontend Transport; Sat, 11 Jul 2020 18:17:54 +0000 X-IncomingTopHeaderMarker: OriginalChecksum:C052B90293D42502E93A5D63605774185197E817384AF58A13E208416A14345A; UpperCasedChecksum:A244C54F690AB9BF32693978B311102DB56ACCC13C813C75F9BE957F96029783; SizeAsReceived:9198; Count:49 Received: from BN6PR04MB0660.namprd04.prod.outlook.com ([fe80::b9c3:9bff:541d:f383]) by BN6PR04MB0660.namprd04.prod.outlook.com ([fe80::b9c3:9bff:541d:f383%9]) with mapi id 15.20.3174.024; Sat, 11 Jul 2020 18:17:54 +0000 Subject: Re: [PATCH 10/11] media: exynos4-is: Prevent duplicate call to media_pipeline_stop To: Tomasz Figa References: <20200426022650.10355-1-xc-racer2@live.ca> <20200707184412.GJ2621465@chromium.org> From: Jonathan Bakker Message-ID: Date: Sat, 11 Jul 2020 11:17:50 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.9.0 In-Reply-To: <20200707184412.GJ2621465@chromium.org> Content-Language: en-US X-ClientProxiedBy: MWHPR19CA0078.namprd19.prod.outlook.com (2603:10b6:320:1f::16) To BN6PR04MB0660.namprd04.prod.outlook.com (2603:10b6:404:d9::21) X-Microsoft-Original-Message-ID: <821026ec-61bf-131f-f4a2-df519eb73feb@live.ca> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from [IPv6:2001:569:fb68:9c00:8067:f823:1e15:7520] (2001:569:fb68:9c00:8067:f823:1e15:7520) by MWHPR19CA0078.namprd19.prod.outlook.com (2603:10b6:320:1f::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3174.20 via Frontend Transport; Sat, 11 Jul 2020 18:17:52 +0000 X-Microsoft-Original-Message-ID: <821026ec-61bf-131f-f4a2-df519eb73feb@live.ca> X-TMN: [G/TLD2hGdTTT3ENvRjgYReZbpkn6Hj0Zo10Dhs1cSCoNFw8+T5syPs4xISXPeTcU] X-MS-PublicTrafficType: Email X-IncomingHeaderCount: 49 X-EOPAttributedMessage: 0 X-MS-Office365-Filtering-Correlation-Id: eb6c3866-b26c-4fa5-1c41-08d825c6baba X-MS-TrafficTypeDiagnostic: DM6NAM11HT013: X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: UAlI3125qArwL4+7qyob7Jb0+c37+Cc5mhVnYIjAkgHdIIYqz8Vio2vFdgLt6Lqgo2MtDfDuxVL+9leXI04Kyv3b6IxrYrmEst4SePdhvHlDVWpsup0n20mrWrpnif8VxbEPFlRMdZ6CUvwiEI3Ye2VXXwQOQaPNwDKmn6p6lAhIPvfORvG45Bi0jg0pfgewXu/RmF2r42isfxv1JWtdvA== X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:0; SRV:; IPV:NLI; SFV:NSPM; H:BN6PR04MB0660.namprd04.prod.outlook.com; PTR:; CAT:NONE; SFTY:; SFS:; DIR:OUT; SFP:1901; X-MS-Exchange-AntiSpam-MessageData: iyjQSTR6oJ0xeKh8+Xli4SPHYnwKLxx17phXAYXfx0WzKtx0pcmxd93SPIXlKvqv+nxoCuOjaBskVKoq4OntZqigTtSF0fhewCiETNkfm0vHILOQUlD2O0+7vSueiI3YiLNECP1TJ7hgyS6UnL6WngFCr6IVotdhLakpPachckDKcIduyqpDZ9jh+CVD/pJCD6xaBniqZRDhkrZ+pu98PQ== X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: eb6c3866-b26c-4fa5-1c41-08d825c6baba X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Jul 2020 18:17:54.3584 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa X-MS-Exchange-CrossTenant-AuthSource: DM6NAM11FT041.eop-nam11.prod.protection.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: Internet X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6NAM11HT013 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200711_141757_507614_994DB205 X-CRM114-Status: GOOD ( 14.47 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org, krzk@kernel.org, kyungmin.park@samsung.com, kgene@kernel.org, s.nawrocki@samsung.com, mchehab@kernel.org, linux-arm-kernel@lists.infradead.org, linux-media@vger.kernel.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Tomasz, On 2020-07-07 11:44 a.m., Tomasz Figa wrote: > Hi Jonathan, > > On Sat, Apr 25, 2020 at 07:26:49PM -0700, Jonathan Bakker wrote: >> media_pipeline_stop can be called from both release and streamoff, >> so make sure they're both protected under the streaming flag and >> not just one of them. > > First of all, thanks for the patch. > > Shouldn't it be that release calls streamoff, so that only streamoff > is supposed to have the call to media_pipeline_stop()? > I can't say that I understand the whole media subsystem enough to know :) Since media_pipeline_start is called in streamon, it makes sense that streamoff should have the media_pipeline_stop call. However, even after removing the call in fimc_capture_release I'm still getting a backtrace such as [ 73.843117] ------------[ cut here ]------------ [ 73.843251] WARNING: CPU: 0 PID: 1575 at drivers/media/mc/mc-entity.c:554 media_pipeline_stop+0x20/0x2c [mc] [ 73.843265] Modules linked in: s5p_fimc v4l2_fwnode exynos4_is_common videobuf2_dma_contig pvrsrvkm_s5pv210_sgx540_120 videobuf2_memops v4l2_mem2mem brcmfmac videobuf2_v4l2 videobuf2_common hci_uart sha256_generic libsha256 btbcm bluetooth cfg80211 brcmutil ecdh_generic ecc ce147 libaes s5ka3dfx videodev atmel_mxt_ts mc pwm_vibra rtc_max8998 [ 73.843471] CPU: 0 PID: 1575 Comm: v4l2-ctl Not tainted 5.7.0-14534-g2b33418b254e-dirty #669 [ 73.843487] Hardware name: Samsung S5PC110/S5PV210-based board [ 73.843562] [] (unwind_backtrace) from [] (show_stack+0x10/0x14) [ 73.843613] [] (show_stack) from [] (__warn+0xbc/0xd4) [ 73.843661] [] (__warn) from [] (warn_slowpath_fmt+0x60/0xb8) [ 73.843734] [] (warn_slowpath_fmt) from [] (media_pipeline_stop+0x20/0x2c [mc]) [ 73.843867] [] (media_pipeline_stop [mc]) from [] (fimc_cap_streamoff+0x38/0x48 [s5p_fimc]) [ 73.844109] [] (fimc_cap_streamoff [s5p_fimc]) from [] (__video_do_ioctl+0x220/0x448 [videodev]) [ 73.844308] [] (__video_do_ioctl [videodev]) from [] (video_usercopy+0x114/0x498 [videodev]) [ 73.844438] [] (video_usercopy [videodev]) from [] (ksys_ioctl+0x20c/0xa10) [ 73.844484] [] (ksys_ioctl) from [] (ret_fast_syscall+0x0/0x54) [ 73.844505] Exception stack(0xe5083fa8 to 0xe5083ff0) [ 73.844546] 3fa0: 0049908d bef8f8c0 00000003 40045613 bef8d5ac 004c1d16 [ 73.844590] 3fc0: 0049908d bef8f8c0 bef8f8c0 00000036 bef8d5ac 00000000 b6d6b320 bef8faf8 [ 73.844620] 3fe0: 004e3ed4 bef8c718 004990bb b6f00d0a [ 73.844642] ---[ end trace e6a4a8b2f20addd4 ]--- The command I'm using for testing is v4l2-ctl --verbose -d 1 --stream-mmap=3 --stream-skip=2 --stream-to=./test.yuv --stream-count=1 Since I noticed that the streaming flag was being checked fimc_capture_release but not in fimc_cap_streamoff, I assumed that it was simply a missed check. Comparing with other drivers, they seem to call media_pipeline_stop in their vb2_ops stop_streaming callback. I'm willing to test various options > Best regards, > Tomasz > Thanks, Jonathan _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel