Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Fri, 25 Sep 2020 16:23:41 -0400
From:   Konstantin Ryabitsev <konstantin@linuxfoundation.org>
To:     Arnaldo Carvalho de Melo <acme@kernel.org>
Cc:     "Jin, Yao" <yao.jin@linux.intel.com>, Jiri Olsa <jolsa@kernel.org>,
        Peter Zijlstra <peterz@infradead.org>,
        Ingo Molnar <mingo@kernel.org>,
        Alexander Shishkin <alexander.shishkin@linux.intel.com>,
        Linux-kernel@vger.kernel.org, Andi Kleen <andi@firstfloor.org>,
        Kan Liang <kan.liang@intel.com>,
        Ian Rogers <irogers@google.com>
Subject: Re: [PATCH v2 1/2] perf vendor events: Update CascadelakeX events to
 v1.08
Message-ID: <20200925202341.k2zznmefv7tcjsnc@chatter.i7.local>
References: <20200922031918.3723-1-yao.jin@linux.intel.com>
 <20200922031918.3723-2-yao.jin@linux.intel.com>
 <20200922194232.GA2505291@kernel.org>
 <4db737f2-0705-5183-6971-9031acfd123b@linux.intel.com>
 <20200925180527.GA2883319@kernel.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
In-Reply-To: <20200925180527.GA2883319@kernel.org>
Precedence: bulk

On Fri, Sep 25, 2020 at 03:05:27PM -0300, Arnaldo Carvalho de Melo wrote:
> Things like b4 help with this and probably have to take into account
> attachments as well, that is why I'm adding Konstantin to the Cc: list
> of this message.
> 
> Konstantin, is this case covered? I.e. patches that get botched and then
> require attachments to be sent to then gets processed?

Hmm... it's complicated. The trouble with handling corner-cases is 
unexpected ways this can affect other mail. For example, what do we do 
when we see a patch in the body, but also a patch as an attachment -- 
should the attachment win, or did the developer mean something entirely 
different ("this is the fixed patch -- I attached the previous version 
for your reference").

I am working on a service that will automatically "explode" pull 
requests into patch series, so this may help work around this particular 
issue. For example, a developer would send a pull-request to the list 
and cc "exploderbot@kernel.org" (or someone else can follow up with a cc 
to that address). When the bot sees the cc, it will automatically 
convert the pull request into patch series and send it to the same 
recipients as on the original pull request.

This should help avoid the problem of terrible mail relays and nasty 
mail clients.

B4 can already do most of that (see "b4 pr --explode"), so adding the 
remaining bits should be easy enough. If this functionality is 
interesting to you, I would be happy to have early beta testers.

-K