From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from wp530.webpack.hosteurope.de (wp530.webpack.hosteurope.de [80.237.130.52]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 596E153AF for ; Wed, 15 Feb 2023 13:18:38 +0000 (UTC) Received: from [2a02:8108:8980:2478:8cde:aa2c:f324:937e]; authenticated by wp530.webpack.hosteurope.de running ExIM with esmtpsa (TLS1.3:ECDHE_RSA_AES_128_GCM_SHA256:128) id 1pSHg6-0001fQ-LV; Wed, 15 Feb 2023 14:18:30 +0100 Message-ID: Date: Wed, 15 Feb 2023 14:18:29 +0100 Precedence: bulk X-Mailing-List: regressions@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.7.2 Subject: Re: [PATCH] Revert "venus: firmware: Correct non-pix start and end addresses" To: Mauro Carvalho Chehab Cc: Vikash Garodia , "linux-kernel@vger.kernel.org" , "mka@chromium.org" , Albert Esteve , "stanimir.varbanov@linaro.org" , Enric Balletbo i Serra , Andy Gross , Bjorn Andersson , Konrad Dybcio , Stanimir Varbanov , "Vikash Garodia (QUIC)" , "linux-arm-msm@vger.kernel.org" , "linux-media@vger.kernel.org" , Fritz Koenig , "Dikshita Agarwal (QUIC)" , "Rajeshwar Kurapaty (QUIC)" , Javier Martinez Canillas , Linux regressions mailing list References: <20230207102254.1446461-1-javierm@redhat.com> <70c01751-1dd7-c4bd-a96e-94dea437aa40@redhat.com> <6f97a117-0d9c-e21b-9adf-50f2233ba9e3@leemhuis.info> Content-Language: en-US, de-DE From: "Linux regression tracking (Thorsten Leemhuis)" Reply-To: Linux regressions mailing list In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-bounce-key: webpack.hosteurope.de;regressions@leemhuis.info;1676467118;3bed09a8; X-HE-SMSGID: 1pSHg6-0001fQ-LV On 15.02.23 11:57, Javier Martinez Canillas wrote: > On Wed, Feb 15, 2023 at 11:53 AM Linux regression tracking (Thorsten > Leemhuis) wrote: >> On 11.02.23 15:27, Linux regression tracking (Thorsten Leemhuis) wrote: >>> On 10.02.23 11:07, Javier Martinez Canillas wrote: >>>> On 2/10/23 10:22, Vikash Garodia wrote: >>>> >>>>>> So what should we do about this folks? Since not allowing the driver to probe on >>>>>> at least SC7180 is a quite serious regression, can we revert for now until a proper >>>>>> fix is figured out? >>>>> I am able to repro this issue on sc7180 and discussing with firmware team on the cause >>>>> of reset failure. The original patch was raised for fixing rare SMMU faults during warm >>>>> boot of video hardware. Hence looking to understand the regressing part before we >>>>> proceed to revert. >>>> Great, if you are working on a proper fix then that would be much better indeed. >>> Yeah, that's great, but OTOH: there is almost certainly just one week >>> before 6.2 will be released. Ideally this should be fixed by then. >>> Vikash, do you think that's in the cards? If not: why not revert this >>> now to make sure 6.2 works fine? >> Hmm, no reply. And we meanwhile have Wednesday and 6.2 is almost >> certainly going to be out on Sunday. And the problem was called "a quite >> serious regression" above. So why not quickly fix this with the revert, >> as proposed earlier? >> Vikash? Javier? > > I agree with you, that we should land this revert and then properly > fix the page fault issue in v6.3. > > But it's not my call, the v4l2/media folks have to decide that. In that case: Mauro, what's your opinion here? Thread starts here: https://lore.kernel.org/lkml/20230207102254.1446461-1-javierm@redhat.com/ Regression report: https://lore.kernel.org/lkml/Y9LSMap%2BjRxbtpC8@google.com/ Ciao, Thorsten