DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 041DE20819
MIME-Version: 1.0
In-Reply-To: <DM5PR02MB3656089F61C0AD9C4D5E5FBCD4CA0@DM5PR02MB3656.namprd02.prod.outlook.com>
References: <1537957497-7790-1-git-send-email-sgruszka@redhat.com>
 <1538697102-3764-1-git-send-email-pozega.tomislav@gmail.com> <DM5PR02MB3656089F61C0AD9C4D5E5FBCD4CA0@DM5PR02MB3656.namprd02.prod.outlook.com>
From:   Tom Psyborg <pozega.tomislav@gmail.com>
Date:   Mon, 5 Nov 2018 13:10:44 +0100
Message-ID: <CAKR_QVKgGhoDhR+JJVW5Ndh38Wia+LGZYDpZqwu_28i1S1kKrw@mail.gmail.com>
Subject: Re: [PATCH 0/5] rt2800mmio txdone/interrupts/flush rework
To:     Craig Matsuura <cmatsuura@vivint.com>
Cc:     "sgruszka@redhat.com" <sgruszka@redhat.com>,
        "linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Sender: linux-wireless-owner@vger.kernel.org
Precedence: bulk

On 05/11/2018, Craig Matsuura <cmatsuura@vivint.com> wrote:
> Tom,
>
>
> I see my name and email in some urls but could not see what you are
> referring too?
>
>
> Craig
>
>
> Craig Matsuura =E2=80=A2 Technical Director, Embedded Software Architectu=
re
>
> cmatsuura@vivint.com<mailto:cmatsuura@vivint.com> =E2=80=A2 P: 801.229.60=
05
>
>
>
> simply smarter =E2=80=A2 vivint.com
>
> 3401 N Ashton Blvd. Lehi, UT 84043
>
>
>
> [1497369905956_vivint-logo-orange.png]
>
> ________________________________
> From: linux-wireless-owner@vger.kernel.org
> <linux-wireless-owner@vger.kernel.org> on behalf of Tomislav Po=C5=BEega
> <pozega.tomislav@gmail.com>
> Sent: Thursday, October 4, 2018 5:51:42 PM
> To: sgruszka@redhat.com
> Cc: linux-wireless@vger.kernel.org
> Subject: Re: [PATCH 0/5] rt2800mmio txdone/interrupts/flush rework
>
> Hi
>
> As suspected this changeset causes throughput regression.
>
> Below screenshots show iperf test from MS150N (RF5370) device connected t=
o
> RT3070 adapter running AP mode:
>
> This is with standard openwrt build without any rt2x00 changes:
>
> [url=3Dhttps://postimg.cc/BtYQLf6r][img]https://na01.safelinks.protection=
.outlook.com/?url=3Dhttps%3A%2F%2Fi.postimg.cc%2FBtYQLf6r%2Fshot-2018-10-04=
_17-23-56.jpg%5B%2Fimg%5D%5B%2Furl&amp;data=3D01%7C01%7Ccmatsuura%40vivint.=
com%7C4b957a380bc24ebda1f708d62a546496%7C54cc98ca024a470185483741e3b8d59d%7=
C0&amp;sdata=3DFsq5PtjbTx3vuXrNyicoVMVyk59dlXin0SecZTKCcOY%3D&amp;reserved=
=3D0]
>
> And this printscreen show iperf test with your changes:
>
> [url=3Dhttps://postimg.cc/D8Sf1p48][img]https://na01.safelinks.protection=
.outlook.com/?url=3Dhttps%3A%2F%2Fi.postimg.cc%2FD8Sf1p48%2Fshot-2018-10-04=
_17-42-09.jpg%5B%2Fimg%5D%5B%2Furl&amp;data=3D01%7C01%7Ccmatsuura%40vivint.=
com%7C4b957a380bc24ebda1f708d62a546496%7C54cc98ca024a470185483741e3b8d59d%7=
C0&amp;sdata=3Dw9eslvFB%2F13VBHGG7n48C2KH4ceyk1Acb5G3wEoPdEc%3D&amp;reserve=
d=3D0]
>
> Atheros card connected to RT3070 iperf test difference was negligible (1M=
bps
> or less) on bodhi system, but
> it started to throw out reorder messages on my standard ubuntu after your
> changes:
>
> [url=3Dhttps://postimg.cc/SjJbP2SP][img]https://na01.safelinks.protection=
.outlook.com/?url=3Dhttps%3A%2F%2Fi.postimg.cc%2FSjJbP2SP%2FScreenshot.png%=
5B%2Fimg%5D%5B%2Furl&amp;data=3D01%7C01%7Ccmatsuura%40vivint.com%7C4b957a38=
0bc24ebda1f708d62a546496%7C54cc98ca024a470185483741e3b8d59d%7C0&amp;sdata=
=3D7AfQGKReywwEYrVZzYOxpLqbmg8kBdCbNE6Mj0%2BmGRM%3D&amp;reserved=3D0]
>
> My advice: stop sending low-quality patches and do some testing before
> submission.
>

Hi

No idea where did you got that message from, some safelink protection
seem to be added somewhere in the mailing process. You can check my
original mail here:
https://www.mail-archive.com/linux-wireless@vger.kernel.org/msg49652.html
. If interested in links you should open them manually as the URL
formating seems inappropriate.