All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Bogendoerfer <tsbogend@alpha.franken.de>
To: Huacai Chen <chenhc@lemote.com>
Cc: "open list:MIPS" <linux-mips@vger.kernel.org>,
	Fuxin Zhang <zhangfx@lemote.com>,
	Zhangjin Wu <wuzhangjin@gmail.com>,
	Jiaxun Yang <jiaxun.yang@flygoat.com>
Subject: Re: [PATCH V2] MIPS: perf: Add hardware perf events support for new Loongson-3
Date: Thu, 30 Apr 2020 17:10:04 +0200	[thread overview]
Message-ID: <20200430151004.GC17822@alpha.franken.de> (raw)
In-Reply-To: <CAAhV-H75EQ=Zd37qPe=VBwT-Z3Fu=_7W4TSG2v8TLpq03MRCdg@mail.gmail.com>

On Thu, Apr 30, 2020 at 05:42:43PM +0800, Huacai Chen wrote:
> On Thu, Apr 30, 2020 at 4:16 PM Thomas Bogendoerfer
> <tsbogend@alpha.franken.de> wrote:
> > that is fine. Could you check how much size increase you get for
> > a non loongson64 target with your patch ?
> I have build an IP28 kernel (ip28_defconfig+perf):
> 
> size of vmlinux with this perf patch: 10913440
> size of vmlinux without this perf patch:10913296
> 
> readelf -a vmlinux with this perf patch:
>   Type           Offset             VirtAddr           PhysAddr
>                  FileSiz            MemSiz              Flags  Align
>   LOAD           0x0000000000001000 0xa800000020004000 0xa800000020004000
>                  0x0000000000800c0c 0x0000000000830dc0  RWE    0x1000
>   NOTE           0x000000000070d020 0xa800000020710020 0xa800000020710020
>                  0x000000000000003c 0x000000000000003c  R      0x4
> 
> readelf -a vmlinux without this perf patch:
>   Type           Offset             VirtAddr           PhysAddr
>                  FileSiz            MemSiz              Flags  Align
>   LOAD           0x0000000000001000 0xa800000020004000 0xa800000020004000
>                  0x0000000000800c0c 0x0000000000830dc0  RWE    0x1000
>   NOTE           0x000000000070cff0 0xa80000002070fff0 0xa80000002070fff0
>                  0x000000000000003c 0x000000000000003c  R      0x4

this looks the same, are you sure you used the right images ?

Nevertheless it's not much. I've applied your new patch. Thank you
for checking.

Thomas.

-- 
Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
good idea.                                                [ RFC1925, 2.3 ]

      reply	other threads:[~2020-04-30 15:11 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-04-29  7:26 [PATCH V2] MIPS: perf: Add hardware perf events support for new Loongson-3 Huacai Chen
2020-04-29 18:22 ` Thomas Bogendoerfer
2020-04-30  7:30   ` Huacai Chen
2020-04-30  8:13     ` Thomas Bogendoerfer
2020-04-30  9:42       ` Huacai Chen
2020-04-30 15:10         ` Thomas Bogendoerfer [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200430151004.GC17822@alpha.franken.de \
    --to=tsbogend@alpha.franken.de \
    --cc=chenhc@lemote.com \
    --cc=jiaxun.yang@flygoat.com \
    --cc=linux-mips@vger.kernel.org \
    --cc=wuzhangjin@gmail.com \
    --cc=zhangfx@lemote.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.