From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (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 62DAB1A4AB4 for ; Thu, 26 Sep 2024 10:54:07 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1727348047; cv=none; b=kbKH/MSK+9HCMOGUuRntdUYzejdu+iQt8DdIZx0lu79zmNjUZjVZrJJlqMzsUrFFJ632aHFmH8p03OsPMHHeBZN0dFZFAeTyjVsIJ6cr+JLlUWrG8f3H5/3K7bvzWFAB8/SFa41F22VBZdt7GsmitDwibA4GpRy/lMylv2epNdU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1727348047; c=relaxed/simple; bh=7EuGqace9PuhPWR9Fa9DKY+X6VuDMBWJqqB5LdpUjCM=; h=Date:From:To:Cc:Subject:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=C2DqU374ZbZJzK7zlwgz9x1rwF3T8P3nNeaJBcYs+XlTycw1N8w8+UdJ2zgJParKU+mMftUjqH7RiHbzi5A8mapEEFchADaezrn0605P6W4NBNJHtrvVwCtg4P/zzMeLt4Lf3NsXzgENlt7BR0L326cTuLK2AS8tQTqXvk4mfZc= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=XZfBp9rT; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="XZfBp9rT" Received: by smtp.kernel.org (Postfix) with ESMTPSA id C25E9C4CEC9; Thu, 26 Sep 2024 10:54:01 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1727348046; bh=7EuGqace9PuhPWR9Fa9DKY+X6VuDMBWJqqB5LdpUjCM=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=XZfBp9rT/bNjCTAO1snIpZwlp5JSPlmIZwXts6mqWzZniK9p0Lu4Unp+gfrDkG+eO CMnd9r2hy9pffcRUyX1sOXH3EEGjqcZfGLhN7RWVk/nxEkifThDnA3Q3wNPOfZm6Ju fHBMx07weEFcAtnyrXHfeT7cz1iGAmIZcPLmtG4dHWF7o/PUiBzrCqCQ7w9uNjhNoV ZDBB72Do+S78bQPwp5FI9+Wslu/iuqPzJz/c2i2u8tLnQzmeTV+RSQ00AUUPzBrkfT GFjkeCewuC9gBQWjoyhZ57YrN009XPhMUtdrNJqcbUXSSnSjxOQk1qvcWv9re5v/YL wa9mb3RuLU/dw== Date: Thu, 26 Sep 2024 12:53:58 +0200 From: Mauro Carvalho Chehab To: Laurent Pinchart Cc: Sakari Ailus , Hans Verkuil , Sebastian Fricke , Linux Media Mailing List , Daniel Almeida , Mauro Carvalho Chehab , Martin Hecht , Tommaso Merciai , Jacopo Mondi , Benjamin Mugnier , Ricardo Ribalda , Michael Tretter , Alain Volmat , Sean Young , Steve Cho , Tomasz Figa , Hidenori Kobayashi , "Hu, Jerry W" , Suresh Vankadara , Devarsh Thakkar , r-donadkar@ti.com, Dave Stevenson , Mehdi Djait , Nicolas Dufresne , Salahaldeen Altous Subject: Re: [ANN] Media Summit September 16th: Final Agenda (v7) Message-ID: <20240926125358.4edf0f9a@foz.lan> In-Reply-To: <20240926102448.GA21788@pendragon.ideasonboard.com> References: <98236d10-4024-4b96-a8ce-8e1dc2a34f1b@xs4all.nl> <20240917091744.qltmddftdy7bpgpg@basti-XPS-13-9310> <20240918092454.21884920@sal.lan> <20240925195843.GK30399@pendragon.ideasonboard.com> <902b2828-5eb8-4741-90af-8e42f1240e86@xs4all.nl> <20240926121914.69b47a50@foz.lan> <20240926102448.GA21788@pendragon.ideasonboard.com> X-Mailer: Claws Mail 4.3.0 (GTK 3.24.43; x86_64-redhat-linux-gnu) Precedence: bulk X-Mailing-List: linux-media@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Em Thu, 26 Sep 2024 13:24:48 +0300 Laurent Pinchart escreveu: > On Thu, Sep 26, 2024 at 12:19:14PM +0200, Mauro Carvalho Chehab wrote: > > Em Thu, 26 Sep 2024 09:30:34 +0000 > > Sakari Ailus escreveu: > > > > Yet, even if the committer did an honest handling of the patch, you may > > still disagree or want some changes at the original patch. On such cases, > > the maintainers may decide to drop the changes and do a normal review > > process. They may otherwise request a patch on the top of the applied > > one to address the pointed issues. > > Let's do a revert in that case, and keep rebases for cases where having > content in the git history causes issues other than bisection problems. Rebasing or not is a subsystem maintainers decision. Reverting pollutes git history upstream, and it should be done in cases were we want to preserve the history upstream. On cases where the preserving the history doesn't matter, a rebase is better. There is also a bad side effect of doing: - patch 1: some fixes with c/c stable + fixes tag - patch 2: revert patch 1 - patch 3: apply patch 1 on a different way Even with just 3 patches, this can get messy when backporting to fixes, as we don't want all three patches backported. We want just patch 3. There are also cases like: - patch 1: some fixes with c/c stable + fixes tag - patch 2: revert patch 1 - patch 3: a patch needed by patch 1 to not break compilation - patch 4: re-apply patch 1 in this case, patch 3 (or a variant of it) may or may not needed to be in fixes. This becomes even more complex if there is a pile of patches with some with c/c stable and some without. I saw already enough badly solved merge conflicts risen on different trees because one change was reverted and then applied back with about the same content. Thanks, Mauro