From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1753118AbcIDHIu (ORCPT <rfc822;w@1wt.eu>);
        Sun, 4 Sep 2016 03:08:50 -0400
Received: from mout.web.de ([212.227.15.4]:62043 "EHLO mout.web.de"
        rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
        id S1752554AbcIDHIr (ORCPT <rfc822;linux-kernel@vger.kernel.org>);
        Sun, 4 Sep 2016 03:08:47 -0400
Subject: Re: sparc: bpf_jit: Move four assignments in bpf_jit_compile()
To: Julian Calaby <julian.calaby@gmail.com>
References: <a6afaf8a-837e-6cc3-fdec-3a37f0e731d5@users.sourceforge.net>
 <CAGRGNgUxt+qgiDHQ7qQ3a7QEJ3xZ4rRTW6RN8fTLNK6E6pgWTw@mail.gmail.com>
 <ec58924c-25d6-e525-fb8a-35bdbf6a8f12@users.sourceforge.net>
 <20160903.233256.116565014024856604.davem@davemloft.net>
 <CAGRGNgX1Y2AeaF38Lw1NaM+NvWkmALrVcqx-DL72ot-kcSCmBA@mail.gmail.com>
Cc: David Miller <davem@davemloft.net>,
        sparclinux <sparclinux@vger.kernel.org>,
        Adam Buchbinder <adam.buchbinder@gmail.com>,
        Alexei Starovoitov <ast@kernel.org>,
        Daniel Borkmann <daniel@iogearbox.net>, Rabin Vincent <rabin@rab.in>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        kernel-janitors <kernel-janitors@vger.kernel.org>,
        Julia Lawall <julia.lawall@lip6.fr>,
        Paolo Bonzini <pbonzini@redhat.com>
From: SF Markus Elfring <elfring@users.sourceforge.net>
Message-ID: <d9e83d28-296e-00bc-a18c-66fca51c3adf@users.sourceforge.net>
Date: Sun, 4 Sep 2016 09:07:33 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101
 Thunderbird/45.3.0
MIME-Version: 1.0
In-Reply-To: <CAGRGNgX1Y2AeaF38Lw1NaM+NvWkmALrVcqx-DL72ot-kcSCmBA@mail.gmail.com>
Content-Type: text/plain; charset=windows-1252
Content-Transfer-Encoding: 7bit
X-Provags-ID: V03:K0:h+vPieOO4V9/No/sf+9dUKZL1o0JHYSBI+Broj9UeBWTxxaEP/x
 xvqcW7UD//yITRR8BNtepcHWx2nlP3zJYEU/voQYAcXfgg4od19bwXH6Kn0/LuDAut6vIp7
 dm9La8S0ZIoQ7LOY6AY8Si1H1UCxNdUTRlPFq254emfczenWo1ze9pg2yJSS8VGZHJt4Ix8
 drsYvda9NPAW3/wpmA4sA==
X-UI-Out-Filterresults: notjunk:1;V01:K0:9urvlSSnSvY=:IZtifefbDNW7CjmYkpkgNH
 7wNokbw3jqHyTL8IiYcKmyvtQsjfXVQATOL5rVAlURH04YBMfAlRnAybxqDPF3wbGWUIuOdrb
 cYrAWDX7VujdWPs+ERb6dldnfyM17RO8TUQeagnN1dTMc+pbF9B1XGS/fmQcoN7f7+zfLwLTy
 GzICIuQ8RuncRITEP5DOMbr2dQ9FaaMb7vV1/agc79KSY65fGghnQWd9Jzut1xwlBB/S9G5Gt
 QlEoINHveSbUfZvnz6kNsStLLEQEssnkZaB8XdVX1S/OZ2TsZEwxmYhMP7UlTMs9AzcYomSQB
 KB/P+oVCXtLpwbrH7xsVsd9WYFsmB0LgxvEmI4/JBCIEDuSwEMqGZEwi1hjyfcF2JL3fh8bub
 OjIpfyk4e4l9A65u4wxSulo9MT+aHEK9+P/ifhLhhjlQMwQeLzFh2U3dtPF+1A5nUcOv2rskw
 1OqNrr93uUktDInrp629fCfO72BR697m+Nj+d4lbSHQD8mL2TTJ6Drf7GYD4cAX/VyjOpwfjG
 c3gqhTwtwXvqpjMu4bYSTlXJe9e4bCBeS6+RHLzIfJqNtD/QGjGtd2kALdwo6X7e4MiPujAAD
 SzCudfZ9IewtjCo172Vggh1uMg4FP+SLbkDuaJc1a4Cno0GGO7j9TkXH8Ityw8fRMlw4VWoXl
 cBsvIn3cp1BAAytufxNzEQBAHYhuzH11U9uLpZAL0Xjah7wQWqKUzaKbqjT3M9egzGN4Yf/y2
 MhjZ9xpTvMNZ5+bQQyIb5eotSKhQHJsv2/cCZEgkQOUm0C7vxnUBmupIvHbzRcGeEcpVEfz8s
 FAImEZx
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

> In Markus' defence, he does occasionally write okay patches,
> (for example the kmalloc(a * b) => kmalloc_array(a, b) changes
> get my small tick of approval)

Thanks for another bit of acceptance for this kind of software change.


> however he also submits a lot of patches which are little more than moving stuff around

This is true to some degree.


> pointlessly.

Do we agree to disagree on the value of some collateral software evolutions
I'm proposing for a while?


> I really wish he'd concentrate on the former rather than the latter.

I am on the way for such a software development adventure.
There are further improvement opportunities to consider besides the main route,
aren't there?


> He's not another Nick Krause.

Are you going to remember me under an other nickname anyhow?

Regards,
Markus