From: Sergey Matyukevich <geomatsi@gmail.com>
To: linux-snps-arc@lists.infradead.org
Cc: kbuild-all@lists.01.org, Vineet Gupta <vgupta@kernel.org>
Subject: Re: [vgupta-arc:for-next 3/4] arch/arc/kernel/disasm.c:494:23: error: 'struct pt_regs' has no member named 'gp'; did you mean 'fp'?
Date: Tue, 26 Apr 2022 10:31:28 +0300 [thread overview]
Message-ID: <Ymef0FkwCE/hdUtb@curiosity> (raw)
In-Reply-To: <202204260826.d9fdltJm-lkp@intel.com>
Hello Vineet,
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/vgupta/arc.git for-next
> head: e88856183749fd59759cd4c617f9ecec8280d8fa
> commit: b08a66fb680018b621fd8be641c537c625132ab3 [3/4] ARC: disasm: handle ARCv2 case in kprobe get/set functions
> config: arc-randconfig-r012-20220425 (https://download.01.org/0day-ci/archive/20220426/202204260826.d9fdltJm-lkp@intel.com/config)
> compiler: arceb-elf-gcc (GCC) 11.3.0
> reproduce (this is a W=1 build):
> wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> chmod +x ~/bin/make.cross
> # https://git.kernel.org/pub/scm/linux/kernel/git/vgupta/arc.git/commit/?id=b08a66fb680018b621fd8be641c537c625132ab3
> git remote add vgupta-arc https://git.kernel.org/pub/scm/linux/kernel/git/vgupta/arc.git
> git fetch --no-tags vgupta-arc for-next
> git checkout b08a66fb680018b621fd8be641c537c625132ab3
> # save the config file
> mkdir build_dir && cp config build_dir/.config
> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.3.0 make.cross W=1 O=build_dir ARCH=arc SHELL=/bin/bash
>
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
>
> All errors (new ones prefixed by >>):
>
> arch/arc/kernel/disasm.c: In function 'disasm_instr':
> arch/arc/kernel/disasm.c:28:19: warning: variable 'ins_ptr' set but not used [-Wunused-but-set-variable]
> 28 | uint16_t *ins_ptr;
> | ^~~~~~~
> arch/arc/kernel/disasm.c: In function 'set_reg':
> >> arch/arc/kernel/disasm.c:494:23: error: 'struct pt_regs' has no member named 'gp'; did you mean 'fp'?
> 494 | regs->gp = val;
> | ^~
> | fp
I have just sent a fixup for this snafu:
https://lore.kernel.org/linux-snps-arc/20220426072447.125975-1-geomatsi@gmail.com/T/#u
IIUC, you have not yet sent a pull-request for next release. So probably
it makes sense to squash this commit with b08a66fb680018b6 ("ARC: disasm:
handle ARCv2 case in kprobe get/set functions").
Regards,
Sergey
_______________________________________________
linux-snps-arc mailing list
linux-snps-arc@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-snps-arc
WARNING: multiple messages have this Message-ID (diff)
From: Sergey Matyukevich <geomatsi@gmail.com>
To: kbuild-all@lists.01.org
Subject: Re: [vgupta-arc:for-next 3/4] arch/arc/kernel/disasm.c:494:23: error: 'struct pt_regs' has no member named 'gp'; did you mean 'fp'?
Date: Tue, 26 Apr 2022 10:31:28 +0300 [thread overview]
Message-ID: <Ymef0FkwCE/hdUtb@curiosity> (raw)
In-Reply-To: <202204260826.d9fdltJm-lkp@intel.com>
[-- Attachment #1: Type: text/plain, Size: 2241 bytes --]
Hello Vineet,
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/vgupta/arc.git for-next
> head: e88856183749fd59759cd4c617f9ecec8280d8fa
> commit: b08a66fb680018b621fd8be641c537c625132ab3 [3/4] ARC: disasm: handle ARCv2 case in kprobe get/set functions
> config: arc-randconfig-r012-20220425 (https://download.01.org/0day-ci/archive/20220426/202204260826.d9fdltJm-lkp(a)intel.com/config)
> compiler: arceb-elf-gcc (GCC) 11.3.0
> reproduce (this is a W=1 build):
> wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> chmod +x ~/bin/make.cross
> # https://git.kernel.org/pub/scm/linux/kernel/git/vgupta/arc.git/commit/?id=b08a66fb680018b621fd8be641c537c625132ab3
> git remote add vgupta-arc https://git.kernel.org/pub/scm/linux/kernel/git/vgupta/arc.git
> git fetch --no-tags vgupta-arc for-next
> git checkout b08a66fb680018b621fd8be641c537c625132ab3
> # save the config file
> mkdir build_dir && cp config build_dir/.config
> COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.3.0 make.cross W=1 O=build_dir ARCH=arc SHELL=/bin/bash
>
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
>
> All errors (new ones prefixed by >>):
>
> arch/arc/kernel/disasm.c: In function 'disasm_instr':
> arch/arc/kernel/disasm.c:28:19: warning: variable 'ins_ptr' set but not used [-Wunused-but-set-variable]
> 28 | uint16_t *ins_ptr;
> | ^~~~~~~
> arch/arc/kernel/disasm.c: In function 'set_reg':
> >> arch/arc/kernel/disasm.c:494:23: error: 'struct pt_regs' has no member named 'gp'; did you mean 'fp'?
> 494 | regs->gp = val;
> | ^~
> | fp
I have just sent a fixup for this snafu:
https://lore.kernel.org/linux-snps-arc/20220426072447.125975-1-geomatsi(a)gmail.com/T/#u
IIUC, you have not yet sent a pull-request for next release. So probably
it makes sense to squash this commit with b08a66fb680018b6 ("ARC: disasm:
handle ARCv2 case in kprobe get/set functions").
Regards,
Sergey
next prev parent reply other threads:[~2022-04-26 7:31 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-04-26 0:41 [vgupta-arc:for-next 3/4] arch/arc/kernel/disasm.c:494:23: error: 'struct pt_regs' has no member named 'gp'; did you mean 'fp'? kernel test robot
2022-04-26 0:41 ` kernel test robot
2022-04-26 7:31 ` Sergey Matyukevich [this message]
2022-04-26 7:31 ` Sergey Matyukevich
2022-04-27 4:04 ` Vineet Gupta
2022-04-27 4:04 ` Vineet Gupta
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=Ymef0FkwCE/hdUtb@curiosity \
--to=geomatsi@gmail.com \
--cc=kbuild-all@lists.01.org \
--cc=linux-snps-arc@lists.infradead.org \
--cc=vgupta@kernel.org \
/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.