Re: Removing architectures without upstream gcc support

[Lennox Wu]

Hi all,
We agree to remove the Score arch.
I have discussed the status with Sunplus, and they plan to replace the
arch with ARM.
I will make a request to remove Linux kernel, too.
Best,
Lennox

2018-02-22 23:45 GMT+08:00 Arnd Bergmann :
> While building the cross-toolchains, I noticed that overall, we can build 
> almost
> all linux target architectures with upstream binutils and gcc these days,
> however there are still some exceptions, and I'd like to find out if anyone
> has objections to removing the ones that do not have upstream support.
> This are the four architectures I found:
>
> * score (s+core, sunplus core) was a proprietary RISC architecture
>   made by sunplus. It is unclear if they still ship any products based on
>   this architecture, all they list is either ARM Cortex-A9 or an unspecified
>   RISC core that could be any of arm, mips, nds32, arc, xtensa or
>   something completely different. The two maintainers have both left the
>   company many years ago and have not contributed any patches in
>   at least five years. There was an upstream gcc port, which was marked
>   'obsolete' in 2013 and got removed in gcc-5.0.
>   I conclude that this is dead in Linux and can be removed
>
> * unicore32 was a research project at Peking University with a SoC
>   based on the Intel PXA design. No gcc source code has ever been
>   published, the only toolchain available is a set of binaries that include
>   a gcc-4.4 compiler. The project page at
>   http://mprc.pku.edu.cn/~guanxuetao/linux/ has a TODO list that has
>   not been modified since 2011. The maintainer still Acks patches
>   and has last sent a pull request in 2014 and last sent a patch of
>   his own in 2012 when the project appears to have stalled.
>   I would suggest removing this one.
>
> * Hexagon is Qualcomm's DSP architecture. It is being actively used
>   in all Snapdragon ARM SoCs, but the kernel code appears to be
>   the result of a failed research project to make a standalone Hexagon
>   SoC without an ARM core. There is some information about the
>   project at https://wiki.codeaurora.org/xwiki/bin/Hexagon/ and
>   
> https://unix.stackexchange.com/questions/246243/what-is-was-the-qualcomm-hexagon-comet-board
>   There is a port to gcc-4.5 on the project page, which is evidently
>   abandoned, but there is an active upstream LLVM port that is
>   apparently used to build non-Linux programs.
>   I would consider this one a candidate for removal as well, given that
>   there were never any machines outside of Qualcomm that used this,
>   and they are no longer interested themselves.
>
> * Meta was ImgTec's own architecture and they upstreamed the kernel
>   port just before they acquired MIPS. Apparently Meta was abandoned
>   shortly afterwards and disappeared from imgtec's website in 2014.
>   The maintainer is still fixing bugs in the port, but I could not find
>   any toolchain more recent than
>   
> https://github.com/img-meta/metag-buildroot/tree/metag-core/toolchain/gcc/4.2.4
>   Not sure about this one, I'd be interested in more background
>   from James Hogan, who probably has an opinion and might have
>   newer toolchain sources.
>
> * OpenRISC is a RISC architecture with a free license and an
>   active community. It seems to have lost a bit of steam after RISC-V
>   is rapidly taking over that niche, but there are chips out there and
>   the design isn't going away. Listing it here for completeness only
>   because there is no upstream gcc port yet, but this will hopefully
>   change in the future based on
>   https://lists.librecores.org/pipermail/openrisc/2018-January/000958.html
>   and I had no problems locating the gcc-7.x tree for building my
>   toolchains. The port is actively being maintained.
>
> There are also a couple of architectures that are more or less
> unmaintained but do have working gcc support: FR-V and M32R
> have been orphaned for a while and are not getting updated
> MN10300 is still maintained officially by David Howells but doesn't
> seem any more active than the other two, the last real updates were
> in 2013.
>
>Arnd

Re: Removing architectures without upstream gcc support

[Lennox Wu]

Hi all,
We agree to remove the Score arch.
I have discussed the status with Sunplus, and they plan to replace the
arch with ARM.
I will make a request to remove Linux kernel, too.
Best,
Lennox

2018-02-22 23:45 GMT+08:00 Arnd Bergmann :
> While building the cross-toolchains, I noticed that overall, we can build 
> almost
> all linux target architectures with upstream binutils and gcc these days,
> however there are still some exceptions, and I'd like to find out if anyone
> has objections to removing the ones that do not have upstream support.
> This are the four architectures I found:
>
> * score (s+core, sunplus core) was a proprietary RISC architecture
>   made by sunplus. It is unclear if they still ship any products based on
>   this architecture, all they list is either ARM Cortex-A9 or an unspecified
>   RISC core that could be any of arm, mips, nds32, arc, xtensa or
>   something completely different. The two maintainers have both left the
>   company many years ago and have not contributed any patches in
>   at least five years. There was an upstream gcc port, which was marked
>   'obsolete' in 2013 and got removed in gcc-5.0.
>   I conclude that this is dead in Linux and can be removed
>
> * unicore32 was a research project at Peking University with a SoC
>   based on the Intel PXA design. No gcc source code has ever been
>   published, the only toolchain available is a set of binaries that include
>   a gcc-4.4 compiler. The project page at
>   http://mprc.pku.edu.cn/~guanxuetao/linux/ has a TODO list that has
>   not been modified since 2011. The maintainer still Acks patches
>   and has last sent a pull request in 2014 and last sent a patch of
>   his own in 2012 when the project appears to have stalled.
>   I would suggest removing this one.
>
> * Hexagon is Qualcomm's DSP architecture. It is being actively used
>   in all Snapdragon ARM SoCs, but the kernel code appears to be
>   the result of a failed research project to make a standalone Hexagon
>   SoC without an ARM core. There is some information about the
>   project at https://wiki.codeaurora.org/xwiki/bin/Hexagon/ and
>   
> https://unix.stackexchange.com/questions/246243/what-is-was-the-qualcomm-hexagon-comet-board
>   There is a port to gcc-4.5 on the project page, which is evidently
>   abandoned, but there is an active upstream LLVM port that is
>   apparently used to build non-Linux programs.
>   I would consider this one a candidate for removal as well, given that
>   there were never any machines outside of Qualcomm that used this,
>   and they are no longer interested themselves.
>
> * Meta was ImgTec's own architecture and they upstreamed the kernel
>   port just before they acquired MIPS. Apparently Meta was abandoned
>   shortly afterwards and disappeared from imgtec's website in 2014.
>   The maintainer is still fixing bugs in the port, but I could not find
>   any toolchain more recent than
>   
> https://github.com/img-meta/metag-buildroot/tree/metag-core/toolchain/gcc/4.2.4
>   Not sure about this one, I'd be interested in more background
>   from James Hogan, who probably has an opinion and might have
>   newer toolchain sources.
>
> * OpenRISC is a RISC architecture with a free license and an
>   active community. It seems to have lost a bit of steam after RISC-V
>   is rapidly taking over that niche, but there are chips out there and
>   the design isn't going away. Listing it here for completeness only
>   because there is no upstream gcc port yet, but this will hopefully
>   change in the future based on
>   https://lists.librecores.org/pipermail/openrisc/2018-January/000958.html
>   and I had no problems locating the gcc-7.x tree for building my
>   toolchains. The port is actively being maintained.
>
> There are also a couple of architectures that are more or less
> unmaintained but do have working gcc support: FR-V and M32R
> have been orphaned for a while and are not getting updated
> MN10300 is still maintained officially by David Howells but doesn't
> seem any more active than the other two, the last real updates were
> in 2013.
>
>Arnd

[GIT PULL] Score: Fix the check condition of get_sigframe

[Lennox Wu]

Hi Linus,
Please pull the patch issued by Stas Sergeev for a signal issue.


 The following changes since commit 67944024c1cdd897e49a09b0d6af3ea38d1388ca:

  Merge tag 'sound-4.5-rc7' of
git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound (2016-03-05
12:35:48 -0800)

are available in the git repository at:


  git://github.com/sctscore/linux-off.git sig

for you to fetch changes up to ea5cf3c0c898848f5b402df385d873ae8479fe80:

  Currently get_sigframe() checks only (ka->sa.sa_flags & SA_ONSTACK) &&
(!on_sig_stack(sp)) to determine whether the switch to sigaltstack is
needed. It forgets to checks whether the sigaltstack was previously set.
(2016-03-11 03:34:05 +0800)

--------
Lennox Wu (1):
  Currently get_sigframe() checks only (ka->sa.sa_flags &
SA_ONSTACK) && (!on_sig_stack(sp)) to determine whether the switch
to sigaltstack is needed. It forgets to checks whether the
sigaltstack was previously set.

 arch/score/kernel/signal.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


--
Best,
Lennox 




signature.asc
Description: OpenPGP digital signature

[GIT PULL] Score: Fix the check condition of get_sigframe

[Lennox Wu]

Hi Linus,
Please pull the patch issued by Stas Sergeev for a signal issue.


 The following changes since commit 67944024c1cdd897e49a09b0d6af3ea38d1388ca:

  Merge tag 'sound-4.5-rc7' of
git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound (2016-03-05
12:35:48 -0800)

are available in the git repository at:


  git://github.com/sctscore/linux-off.git sig

for you to fetch changes up to ea5cf3c0c898848f5b402df385d873ae8479fe80:

  Currently get_sigframe() checks only (ka->sa.sa_flags & SA_ONSTACK) &&
(!on_sig_stack(sp)) to determine whether the switch to sigaltstack is
needed. It forgets to checks whether the sigaltstack was previously set.
(2016-03-11 03:34:05 +0800)

--------
Lennox Wu (1):
  Currently get_sigframe() checks only (ka->sa.sa_flags &
SA_ONSTACK) && (!on_sig_stack(sp)) to determine whether the switch
to sigaltstack is needed. It forgets to checks whether the
sigaltstack was previously set.

 arch/score/kernel/signal.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


--
Best,
Lennox 




signature.asc
Description: OpenPGP digital signature

Re: [PATCH 2/4] score: signal: fix sigaltstack check

[Lennox Wu]

Hi Sergeev,
Your are right. I will merge it.
Thanks.

Acked-by: Lennox Wu 

2016-02-01 0:21 GMT+08:00 Stas Sergeev :
>
> Currently get_sigframe() checks only
> (ka->sa.sa_flags & SA_ONSTACK) && (!on_sig_stack(sp))
> to determine whether the switch to sigaltstack is needed.
> It forgets to checks whether the sigaltstack was previously set.
>
> This patch replaces the !on_sig_stack(sp) with the standard check
> sas_ss_flags(sp) == 0
> that takes into account both conditions: it succeeds only if the
> sigaltstack is enabled but currently not active.
>
> CC: Andy Lutomirski 
> CC: linux-kernel@vger.kernel.org
> CC: Chen Liqin 
> CC: Lennox Wu 
> CC: Michael Ellerman 
> CC: Andrew Morton 
> CC: James Hogan 
>
> Signed-off-by: Stas Sergeev 
> ---
>  arch/score/kernel/signal.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/score/kernel/signal.c b/arch/score/kernel/signal.c
> index e381c8c..bd1c7c8 100644
> --- a/arch/score/kernel/signal.c
> +++ b/arch/score/kernel/signal.c
> @@ -127,7 +127,7 @@ static void __user *get_sigframe(struct k_sigaction *ka,
>  sp -= 32;
>
>  /* This is the X/Open sanctioned signal stack switching.  */
> -if ((ka->sa.sa_flags & SA_ONSTACK) && (!on_sig_stack(sp)))
> +if ((ka->sa.sa_flags & SA_ONSTACK) && (sas_ss_flags(sp) == 0))
>  sp = current->sas_ss_sp + current->sas_ss_size;
>
>  return (void __user*)((sp - frame_size) & ~7);
> --
> 2.5.0
>

Re: [PATCH 2/4] score: signal: fix sigaltstack check

[Lennox Wu]

Hi Sergeev,
Your are right. I will merge it.
Thanks.

Acked-by: Lennox Wu <lennox...@gmail.com>

2016-02-01 0:21 GMT+08:00 Stas Sergeev <s...@list.ru>:
>
> Currently get_sigframe() checks only
> (ka->sa.sa_flags & SA_ONSTACK) && (!on_sig_stack(sp))
> to determine whether the switch to sigaltstack is needed.
> It forgets to checks whether the sigaltstack was previously set.
>
> This patch replaces the !on_sig_stack(sp) with the standard check
> sas_ss_flags(sp) == 0
> that takes into account both conditions: it succeeds only if the
> sigaltstack is enabled but currently not active.
>
> CC: Andy Lutomirski <l...@amacapital.net>
> CC: linux-kernel@vger.kernel.org
> CC: Chen Liqin <liqin.li...@gmail.com>
> CC: Lennox Wu <lennox...@gmail.com>
> CC: Michael Ellerman <m...@ellerman.id.au>
> CC: Andrew Morton <a...@linux-foundation.org>
> CC: James Hogan <james.ho...@imgtec.com>
>
> Signed-off-by: Stas Sergeev <s...@users.sourceforge.net>
> ---
>  arch/score/kernel/signal.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/score/kernel/signal.c b/arch/score/kernel/signal.c
> index e381c8c..bd1c7c8 100644
> --- a/arch/score/kernel/signal.c
> +++ b/arch/score/kernel/signal.c
> @@ -127,7 +127,7 @@ static void __user *get_sigframe(struct k_sigaction *ka,
>  sp -= 32;
>
>  /* This is the X/Open sanctioned signal stack switching.  */
> -if ((ka->sa.sa_flags & SA_ONSTACK) && (!on_sig_stack(sp)))
> +if ((ka->sa.sa_flags & SA_ONSTACK) && (sas_ss_flags(sp) == 0))
>  sp = current->sas_ss_sp + current->sas_ss_size;
>
>  return (void __user*)((sp - frame_size) & ~7);
> --
> 2.5.0
>

Re: [PATCH 67/74] score/time: Migrate to new 'set-state' interface

[Lennox Wu]

Acked-by: Lennox Wu 

2015-08-10 17:52 GMT+08:00 Daniel Lezcano :
> From: Viresh Kumar 
>
> Migrate score driver to the new 'set-state' interface provided by
> clockevents core, the earlier 'set-mode' interface is marked obsolete
> now.
>
> This also enables us to implement callbacks for new states of clockevent
> devices, for example: ONESHOT_STOPPED.
>
> We weren't doing anything in ONESHOT/SHUTDOWN/RESUME modes and so
> callbacks for them aren't implemented.
>
> Cc: Chen Liqin 
> Cc: Lennox Wu 
> Cc: Michael Opdenacker 
> Signed-off-by: Viresh Kumar 
> Signed-off-by: Daniel Lezcano 
> ---
>  arch/score/kernel/time.c | 31 ++-
>  1 file changed, 10 insertions(+), 21 deletions(-)
>
> diff --git a/arch/score/kernel/time.c b/arch/score/kernel/time.c
> index 24770cd..679b8d7 100644
> --- a/arch/score/kernel/time.c
> +++ b/arch/score/kernel/time.c
> @@ -55,31 +55,20 @@ static int score_timer_set_next_event(unsigned long delta,
> return 0;
>  }
>
> -static void score_timer_set_mode(enum clock_event_mode mode,
> -   struct clock_event_device *evdev)
> +static int score_timer_set_periodic(struct clock_event_device *evt)
>  {
> -   switch (mode) {
> -   case CLOCK_EVT_MODE_PERIODIC:
> -   outl((TMR_M_PERIODIC | TMR_IE_ENABLE), P_TIMER0_CTRL);
> -   outl(SYSTEM_CLOCK/HZ, P_TIMER0_PRELOAD);
> -   outl(inl(P_TIMER0_CTRL) | TMR_ENABLE, P_TIMER0_CTRL);
> -   break;
> -   case CLOCK_EVT_MODE_ONESHOT:
> -   case CLOCK_EVT_MODE_SHUTDOWN:
> -   case CLOCK_EVT_MODE_RESUME:
> -   case CLOCK_EVT_MODE_UNUSED:
> -   break;
> -   default:
> -   BUG();
> -   }
> +   outl((TMR_M_PERIODIC | TMR_IE_ENABLE), P_TIMER0_CTRL);
> +   outl(SYSTEM_CLOCK / HZ, P_TIMER0_PRELOAD);
> +   outl(inl(P_TIMER0_CTRL) | TMR_ENABLE, P_TIMER0_CTRL);
> +   return 0;
>  }
>
>  static struct clock_event_device score_clockevent = {
> -   .name   = "score_clockevent",
> -   .features   = CLOCK_EVT_FEAT_PERIODIC,
> -   .shift  = 16,
> -   .set_next_event = score_timer_set_next_event,
> -   .set_mode   = score_timer_set_mode,
> +   .name   = "score_clockevent",
> +   .features   = CLOCK_EVT_FEAT_PERIODIC,
> +   .shift  = 16,
> +   .set_next_event = score_timer_set_next_event,
> +   .set_state_periodic = score_timer_set_periodic,
>  };
>
>  void __init time_init(void)
> --
> 1.9.1
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH 67/74] score/time: Migrate to new 'set-state' interface

[Lennox Wu]

Acked-by: Lennox Wu lennox...@gmail.com

2015-08-10 17:52 GMT+08:00 Daniel Lezcano daniel.lezc...@linaro.org:
 From: Viresh Kumar viresh.ku...@linaro.org

 Migrate score driver to the new 'set-state' interface provided by
 clockevents core, the earlier 'set-mode' interface is marked obsolete
 now.

 This also enables us to implement callbacks for new states of clockevent
 devices, for example: ONESHOT_STOPPED.

 We weren't doing anything in ONESHOT/SHUTDOWN/RESUME modes and so
 callbacks for them aren't implemented.

 Cc: Chen Liqin liqin.li...@gmail.com
 Cc: Lennox Wu lennox...@gmail.com
 Cc: Michael Opdenacker michael.opdenac...@free-electrons.com
 Signed-off-by: Viresh Kumar viresh.ku...@linaro.org
 Signed-off-by: Daniel Lezcano daniel.lezc...@linaro.org
 ---
  arch/score/kernel/time.c | 31 ++-
  1 file changed, 10 insertions(+), 21 deletions(-)

 diff --git a/arch/score/kernel/time.c b/arch/score/kernel/time.c
 index 24770cd..679b8d7 100644
 --- a/arch/score/kernel/time.c
 +++ b/arch/score/kernel/time.c
 @@ -55,31 +55,20 @@ static int score_timer_set_next_event(unsigned long delta,
 return 0;
  }

 -static void score_timer_set_mode(enum clock_event_mode mode,
 -   struct clock_event_device *evdev)
 +static int score_timer_set_periodic(struct clock_event_device *evt)
  {
 -   switch (mode) {
 -   case CLOCK_EVT_MODE_PERIODIC:
 -   outl((TMR_M_PERIODIC | TMR_IE_ENABLE), P_TIMER0_CTRL);
 -   outl(SYSTEM_CLOCK/HZ, P_TIMER0_PRELOAD);
 -   outl(inl(P_TIMER0_CTRL) | TMR_ENABLE, P_TIMER0_CTRL);
 -   break;
 -   case CLOCK_EVT_MODE_ONESHOT:
 -   case CLOCK_EVT_MODE_SHUTDOWN:
 -   case CLOCK_EVT_MODE_RESUME:
 -   case CLOCK_EVT_MODE_UNUSED:
 -   break;
 -   default:
 -   BUG();
 -   }
 +   outl((TMR_M_PERIODIC | TMR_IE_ENABLE), P_TIMER0_CTRL);
 +   outl(SYSTEM_CLOCK / HZ, P_TIMER0_PRELOAD);
 +   outl(inl(P_TIMER0_CTRL) | TMR_ENABLE, P_TIMER0_CTRL);
 +   return 0;
  }

  static struct clock_event_device score_clockevent = {
 -   .name   = score_clockevent,
 -   .features   = CLOCK_EVT_FEAT_PERIODIC,
 -   .shift  = 16,
 -   .set_next_event = score_timer_set_next_event,
 -   .set_mode   = score_timer_set_mode,
 +   .name   = score_clockevent,
 +   .features   = CLOCK_EVT_FEAT_PERIODIC,
 +   .shift  = 16,
 +   .set_next_event = score_timer_set_next_event,
 +   .set_state_periodic = score_timer_set_periodic,
  };

  void __init time_init(void)
 --
 1.9.1

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [Git PULL] score: Fix exception handler label

[Lennox Wu]

mm...I also don't know what happened, I will reissue a request this weekend.
Best,
Lennox


2015-06-02 23:16 GMT+08:00 Guenter Roeck :
> On Mon, Jun 01, 2015 at 01:24:05AM +0800, Lennox Wu wrote:
>> The following changes since commit 792498003dadf060aa92abde007e7803f55e145b:
>>
>>   Linux 4.1-rc5 (2015-05-31 23:32:58 +0800)
>>
>> are available in the git repository at:
>>
>>   git://github.com/sctscore/linux-off.git
>> a5c6e73c15d95980a54996350068524b7da55976
>>
>> for you to fetch changes up to a5c6e73c15d95980a54996350068524b7da55976:
>>
>>   score: Fix exception handler label (2015-05-31 23:33:00 +0800)
>>
>> 
>> Guenter Roeck (1):
>>   score: Fix exception handler label
>>
>>  arch/score/lib/string.S | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
> Hi,
>
> gpg says
>
> [-- PGP output follows (current time: Tue 02 Jun 2015 08:02:40 AM PDT) --]
> gpg: Signature made Sun 31 May 2015 10:24:10 AM PDT using RSA key ID 2180C65C
> gpg: Good signature from "Lennox Wu "
> gpg: WARNING: This key is not certified with a trusted signature!
> gpg:  There is no indication that the signature belongs to the owner.
> Primary key fingerprint: 2662 FB4C 0995 7459 79FA  5037 DF7C E3EC 2180 C65C
> [-- End of PGP output --]
>
> Can you get someone to sign your key ?
>
> Also, the history in your tree does not match with the upstream history.
>
> Your tree:
>
> 792498003dad Linux 4.1-rc5
>
> upstream:
>
> ba155e2d21f6 Linux 4.1-rc5
>
> No idea what happened, but it is not entirely surprising that Linus
> did not accept your pull request.
>
> Guenter
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [Git PULL] score: Fix exception handler label

[Lennox Wu]

mm...I also don't know what happened, I will reissue a request this weekend.
Best,
Lennox


2015-06-02 23:16 GMT+08:00 Guenter Roeck li...@roeck-us.net:
 On Mon, Jun 01, 2015 at 01:24:05AM +0800, Lennox Wu wrote:
 The following changes since commit 792498003dadf060aa92abde007e7803f55e145b:

   Linux 4.1-rc5 (2015-05-31 23:32:58 +0800)

 are available in the git repository at:

   git://github.com/sctscore/linux-off.git
 a5c6e73c15d95980a54996350068524b7da55976

 for you to fetch changes up to a5c6e73c15d95980a54996350068524b7da55976:

   score: Fix exception handler label (2015-05-31 23:33:00 +0800)

 
 Guenter Roeck (1):
   score: Fix exception handler label

  arch/score/lib/string.S | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

 Hi,

 gpg says

 [-- PGP output follows (current time: Tue 02 Jun 2015 08:02:40 AM PDT) --]
 gpg: Signature made Sun 31 May 2015 10:24:10 AM PDT using RSA key ID 2180C65C
 gpg: Good signature from Lennox Wu lennox...@gmail.com
 gpg: WARNING: This key is not certified with a trusted signature!
 gpg:  There is no indication that the signature belongs to the owner.
 Primary key fingerprint: 2662 FB4C 0995 7459 79FA  5037 DF7C E3EC 2180 C65C
 [-- End of PGP output --]

 Can you get someone to sign your key ?

 Also, the history in your tree does not match with the upstream history.

 Your tree:

 792498003dad Linux 4.1-rc5

 upstream:

 ba155e2d21f6 Linux 4.1-rc5

 No idea what happened, but it is not entirely surprising that Linus
 did not accept your pull request.

 Guenter
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Git PULL] score: Fix exception handler label

[Lennox Wu]

The following changes since commit 792498003dadf060aa92abde007e7803f55e145b:

  Linux 4.1-rc5 (2015-05-31 23:32:58 +0800)

are available in the git repository at:

  git://github.com/sctscore/linux-off.git
a5c6e73c15d95980a54996350068524b7da55976

for you to fetch changes up to a5c6e73c15d95980a54996350068524b7da55976:

  score: Fix exception handler label (2015-05-31 23:33:00 +0800)


Guenter Roeck (1):
  score: Fix exception handler label

 arch/score/lib/string.S | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)



signature.asc
Description: OpenPGP digital signature

[Git PULL] score: Fix exception handler label

[Lennox Wu]

The following changes since commit 792498003dadf060aa92abde007e7803f55e145b:

  Linux 4.1-rc5 (2015-05-31 23:32:58 +0800)

are available in the git repository at:

  git://github.com/sctscore/linux-off.git
a5c6e73c15d95980a54996350068524b7da55976

for you to fetch changes up to a5c6e73c15d95980a54996350068524b7da55976:

  score: Fix exception handler label (2015-05-31 23:33:00 +0800)


Guenter Roeck (1):
  score: Fix exception handler label

 arch/score/lib/string.S | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)



signature.asc
Description: OpenPGP digital signature

Re: [PATCH resend 4/5] score: Fix exception handler label

[Lennox Wu]

I will issue a pull request in this week.

Best,
Lennox

2015-05-23 12:24 GMT+08:00 Guenter Roeck :
> On 05/04/2015 03:30 PM, Guenter Roeck wrote:
>>
>> The latest version of modinfo fails to compile score architecture
>> targets with the following error.
>>
>> FATAL: The relocation at __ex_table+0x634 references
>> section "__ex_table" which is not executable, IOW
>> the kernel will fault if it ever tries to
>> jump to it.  Something is seriously wrong
>> and should be fixed.
>>
>> The probem is caused by a bad label in an __ex_table entry.
>>
>> Cc: Quentin Casasnovas 
>> Signed-off-by: Guenter Roeck 
>> ---
>> Please consider pushing this patch into mainline, or provide feedback
>> on how to improve it to be acceptable.
>>
> ping ... still not in mainline.
>
> Thanks,
> Guenter
>
>
>>   arch/score/lib/string.S | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/score/lib/string.S b/arch/score/lib/string.S
>> index 00b7d3a2fc60..16efa3ad037f 100644
>> --- a/arch/score/lib/string.S
>> +++ b/arch/score/lib/string.S
>> @@ -175,10 +175,10 @@ ENTRY(__clear_user)
>> br  r3
>>
>> .section .fixup, "ax"
>> +99:
>> br  r3
>> .previous
>> .section __ex_table, "a"
>> .align  2
>> -99:
>> .word   0b, 99b
>> .previous
>>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH resend 4/5] score: Fix exception handler label

[Lennox Wu]

I will issue a pull request in this week.

Best,
Lennox

2015-05-23 12:24 GMT+08:00 Guenter Roeck li...@roeck-us.net:
 On 05/04/2015 03:30 PM, Guenter Roeck wrote:

 The latest version of modinfo fails to compile score architecture
 targets with the following error.

 FATAL: The relocation at __ex_table+0x634 references
 section __ex_table which is not executable, IOW
 the kernel will fault if it ever tries to
 jump to it.  Something is seriously wrong
 and should be fixed.

 The probem is caused by a bad label in an __ex_table entry.

 Cc: Quentin Casasnovas quentin.casasno...@oracle.com
 Signed-off-by: Guenter Roeck li...@roeck-us.net
 ---
 Please consider pushing this patch into mainline, or provide feedback
 on how to improve it to be acceptable.

 ping ... still not in mainline.

 Thanks,
 Guenter


   arch/score/lib/string.S | 2 +-
   1 file changed, 1 insertion(+), 1 deletion(-)

 diff --git a/arch/score/lib/string.S b/arch/score/lib/string.S
 index 00b7d3a2fc60..16efa3ad037f 100644
 --- a/arch/score/lib/string.S
 +++ b/arch/score/lib/string.S
 @@ -175,10 +175,10 @@ ENTRY(__clear_user)
 br  r3

 .section .fixup, ax
 +99:
 br  r3
 .previous
 .section __ex_table, a
 .align  2
 -99:
 .word   0b, 99b
 .previous


--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH resend 4/5] score: Fix exception handler label

[Lennox Wu]

Hi Roeck,
You are right! I will push the patch.
Thank you.

Acked-by: Lennox Wu 

2015-05-05 6:30 GMT+08:00 Guenter Roeck :
> The latest version of modinfo fails to compile score architecture
> targets with the following error.
>
> FATAL: The relocation at __ex_table+0x634 references
> section "__ex_table" which is not executable, IOW
> the kernel will fault if it ever tries to
> jump to it.  Something is seriously wrong
> and should be fixed.
>
> The probem is caused by a bad label in an __ex_table entry.
>
> Cc: Quentin Casasnovas 
> Signed-off-by: Guenter Roeck 
> ---
> Please consider pushing this patch into mainline, or provide feedback
> on how to improve it to be acceptable.
>
>  arch/score/lib/string.S | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/score/lib/string.S b/arch/score/lib/string.S
> index 00b7d3a2fc60..16efa3ad037f 100644
> --- a/arch/score/lib/string.S
> +++ b/arch/score/lib/string.S
> @@ -175,10 +175,10 @@ ENTRY(__clear_user)
> br  r3
>
> .section .fixup, "ax"
> +99:
> br  r3
> .previous
> .section __ex_table, "a"
> .align  2
> -99:
> .word   0b, 99b
> .previous
> --
> 2.1.0
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH resend 4/5] score: Fix exception handler label

[Lennox Wu]

Hi Roeck,
You are right! I will push the patch.
Thank you.

Acked-by: Lennox Wu lennox...@gmail.com

2015-05-05 6:30 GMT+08:00 Guenter Roeck li...@roeck-us.net:
 The latest version of modinfo fails to compile score architecture
 targets with the following error.

 FATAL: The relocation at __ex_table+0x634 references
 section __ex_table which is not executable, IOW
 the kernel will fault if it ever tries to
 jump to it.  Something is seriously wrong
 and should be fixed.

 The probem is caused by a bad label in an __ex_table entry.

 Cc: Quentin Casasnovas quentin.casasno...@oracle.com
 Signed-off-by: Guenter Roeck li...@roeck-us.net
 ---
 Please consider pushing this patch into mainline, or provide feedback
 on how to improve it to be acceptable.

  arch/score/lib/string.S | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

 diff --git a/arch/score/lib/string.S b/arch/score/lib/string.S
 index 00b7d3a2fc60..16efa3ad037f 100644
 --- a/arch/score/lib/string.S
 +++ b/arch/score/lib/string.S
 @@ -175,10 +175,10 @@ ENTRY(__clear_user)
 br  r3

 .section .fixup, ax
 +99:
 br  r3
 .previous
 .section __ex_table, a
 .align  2
 -99:
 .word   0b, 99b
 .previous
 --
 2.1.0

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH v2 08/47] kernel: Move pm_power_off to common code

[Lennox Wu]

Acked-by: Lennox Wu 

2014-10-21 12:12 GMT+08:00 Guenter Roeck :
> pm_power_off is defined for all architectures. Move it to common code.
>
> Have all architectures call do_kernel_power_off instead of pm_power_off.
> Some architectures point pm_power_off to machine_power_off. For those,
> call do_kernel_power_off from machine_power_off instead.
>
> Acked-by: David Vrabel 
> Acked-by: Geert Uytterhoeven 
> Acked-by: Hirokazu Takata 
> Acked-by: Jesper Nilsson 
> Acked-by: Max Filippov 
> Acked-by: Rafael J. Wysocki 
> Acked-by: Richard Weinberger 
> Acked-by: Xuetao Guan 
> Signed-off-by: Guenter Roeck 
> ---
> v2:
> - do_kernel_poweroff -> do_kernel_power_off
> - have_kernel_poweroff -> have_kernel_power_off
>
>  arch/alpha/kernel/process.c|  9 +++--
>  arch/arc/kernel/reset.c|  5 +
>  arch/arm/kernel/process.c  |  5 +
>  arch/arm64/kernel/process.c|  5 +
>  arch/avr32/kernel/process.c|  6 +-
>  arch/blackfin/kernel/process.c |  3 ---
>  arch/blackfin/kernel/reboot.c  |  2 ++
>  arch/c6x/kernel/process.c  |  9 +
>  arch/cris/kernel/process.c |  4 +---
>  arch/frv/kernel/process.c  |  5 ++---
>  arch/hexagon/kernel/reset.c|  5 ++---
>  arch/ia64/kernel/process.c |  5 +
>  arch/m32r/kernel/process.c |  8 
>  arch/m68k/kernel/process.c |  6 +-
>  arch/metag/kernel/process.c|  6 +-
>  arch/microblaze/kernel/process.c   |  3 ---
>  arch/microblaze/kernel/reset.c |  1 +
>  arch/mips/kernel/reset.c   |  6 +-
>  arch/mn10300/kernel/process.c  |  8 ++--
>  arch/openrisc/kernel/process.c |  8 +---
>  arch/parisc/kernel/process.c   |  8 
>  arch/powerpc/kernel/setup-common.c |  6 +++---
>  arch/s390/kernel/setup.c   |  8 ++--
>  arch/score/kernel/process.c|  8 
>  arch/sh/kernel/reboot.c|  6 +-
>  arch/sparc/kernel/process_32.c | 10 ++
>  arch/sparc/kernel/reboot.c |  8 ++--
>  arch/tile/kernel/reboot.c  |  7 +++
>  arch/um/kernel/reboot.c|  2 --
>  arch/unicore32/kernel/process.c|  9 +
>  arch/x86/kernel/reboot.c   | 11 +++
>  arch/x86/xen/enlighten.c   |  3 +--
>  arch/xtensa/kernel/process.c   |  4 
>  drivers/parisc/power.c |  3 +--
>  kernel/power/poweroff_handler.c|  8 
>  kernel/reboot.c|  4 ++--
>  36 files changed, 68 insertions(+), 146 deletions(-)
>
> diff --git a/arch/alpha/kernel/process.c b/arch/alpha/kernel/process.c
> index 1941a07..81c43f8 100644
> --- a/arch/alpha/kernel/process.c
> +++ b/arch/alpha/kernel/process.c
> @@ -24,6 +24,7 @@
>  #include 
>  #include 
>  #include 
> +#include 
>  #include 
>  #include 
>  #include 
> @@ -40,12 +41,6 @@
>  #include "proto.h"
>  #include "pci_impl.h"
>
> -/*
> - * Power off function, if any
> - */
> -void (*pm_power_off)(void) = machine_power_off;
> -EXPORT_SYMBOL(pm_power_off);
> -
>  #ifdef CONFIG_ALPHA_WTINT
>  /*
>   * Sleep the CPU.
> @@ -184,6 +179,8 @@ machine_halt(void)
>  void
>  machine_power_off(void)
>  {
> +   do_kernel_power_off();
> +
> common_shutdown(LINUX_REBOOT_CMD_POWER_OFF, NULL);
>  }
>
> diff --git a/arch/arc/kernel/reset.c b/arch/arc/kernel/reset.c
> index 2768fa1..0758d9d 100644
> --- a/arch/arc/kernel/reset.c
> +++ b/arch/arc/kernel/reset.c
> @@ -26,9 +26,6 @@ void machine_restart(char *__unused)
>
>  void machine_power_off(void)
>  {
> -   /* FIXME ::  power off ??? */
> +   do_kernel_power_off();
> machine_halt();
>  }
> -
> -void (*pm_power_off) (void) = NULL;
> -EXPORT_SYMBOL(pm_power_off);
> diff --git a/arch/arm/kernel/process.c b/arch/arm/kernel/process.c
> index fe972a2..aa3f656 100644
> --- a/arch/arm/kernel/process.c
> +++ b/arch/arm/kernel/process.c
> @@ -117,8 +117,6 @@ void soft_restart(unsigned long addr)
>  /*
>   * Function pointers to optional machine specific functions
>   */
> -void (*pm_power_off)(void);
> -EXPORT_SYMBOL(pm_power_off);
>
>  void (*arm_pm_restart)(enum reboot_mode reboot_mode, const char *cmd);
>
> @@ -205,8 +203,7 @@ void machine_power_off(void)
> local_irq_disable();
> smp_send_stop();
>
> -   if (pm_power_off)
> -   pm_power_off();
> +   do_kernel_power_off();
>  }
>
>  /*
> diff --git a/arch/arm64/kernel/process.c b/arch/arm64/kernel/process.c
> index c3065db..46a483a 100644
> --- a/arch/arm64/kernel/process.c

Re: [PATCH v2 08/47] kernel: Move pm_power_off to common code

[Lennox Wu]

Acked-by: Lennox Wu lennox...@gmail.com

2014-10-21 12:12 GMT+08:00 Guenter Roeck li...@roeck-us.net:
 pm_power_off is defined for all architectures. Move it to common code.

 Have all architectures call do_kernel_power_off instead of pm_power_off.
 Some architectures point pm_power_off to machine_power_off. For those,
 call do_kernel_power_off from machine_power_off instead.

 Acked-by: David Vrabel david.vra...@citrix.com
 Acked-by: Geert Uytterhoeven ge...@linux-m68k.org
 Acked-by: Hirokazu Takata tak...@linux-m32r.org
 Acked-by: Jesper Nilsson jesper.nils...@axis.com
 Acked-by: Max Filippov jcmvb...@gmail.com
 Acked-by: Rafael J. Wysocki r...@rjwysocki.net
 Acked-by: Richard Weinberger rich...@nod.at
 Acked-by: Xuetao Guan g...@mprc.pku.edu.cn
 Signed-off-by: Guenter Roeck li...@roeck-us.net
 ---
 v2:
 - do_kernel_poweroff - do_kernel_power_off
 - have_kernel_poweroff - have_kernel_power_off

  arch/alpha/kernel/process.c|  9 +++--
  arch/arc/kernel/reset.c|  5 +
  arch/arm/kernel/process.c  |  5 +
  arch/arm64/kernel/process.c|  5 +
  arch/avr32/kernel/process.c|  6 +-
  arch/blackfin/kernel/process.c |  3 ---
  arch/blackfin/kernel/reboot.c  |  2 ++
  arch/c6x/kernel/process.c  |  9 +
  arch/cris/kernel/process.c |  4 +---
  arch/frv/kernel/process.c  |  5 ++---
  arch/hexagon/kernel/reset.c|  5 ++---
  arch/ia64/kernel/process.c |  5 +
  arch/m32r/kernel/process.c |  8 
  arch/m68k/kernel/process.c |  6 +-
  arch/metag/kernel/process.c|  6 +-
  arch/microblaze/kernel/process.c   |  3 ---
  arch/microblaze/kernel/reset.c |  1 +
  arch/mips/kernel/reset.c   |  6 +-
  arch/mn10300/kernel/process.c  |  8 ++--
  arch/openrisc/kernel/process.c |  8 +---
  arch/parisc/kernel/process.c   |  8 
  arch/powerpc/kernel/setup-common.c |  6 +++---
  arch/s390/kernel/setup.c   |  8 ++--
  arch/score/kernel/process.c|  8 
  arch/sh/kernel/reboot.c|  6 +-
  arch/sparc/kernel/process_32.c | 10 ++
  arch/sparc/kernel/reboot.c |  8 ++--
  arch/tile/kernel/reboot.c  |  7 +++
  arch/um/kernel/reboot.c|  2 --
  arch/unicore32/kernel/process.c|  9 +
  arch/x86/kernel/reboot.c   | 11 +++
  arch/x86/xen/enlighten.c   |  3 +--
  arch/xtensa/kernel/process.c   |  4 
  drivers/parisc/power.c |  3 +--
  kernel/power/poweroff_handler.c|  8 
  kernel/reboot.c|  4 ++--
  36 files changed, 68 insertions(+), 146 deletions(-)

 diff --git a/arch/alpha/kernel/process.c b/arch/alpha/kernel/process.c
 index 1941a07..81c43f8 100644
 --- a/arch/alpha/kernel/process.c
 +++ b/arch/alpha/kernel/process.c
 @@ -24,6 +24,7 @@
  #include linux/vt.h
  #include linux/mman.h
  #include linux/elfcore.h
 +#include linux/pm.h
  #include linux/reboot.h
  #include linux/tty.h
  #include linux/console.h
 @@ -40,12 +41,6 @@
  #include proto.h
  #include pci_impl.h

 -/*
 - * Power off function, if any
 - */
 -void (*pm_power_off)(void) = machine_power_off;
 -EXPORT_SYMBOL(pm_power_off);
 -
  #ifdef CONFIG_ALPHA_WTINT
  /*
   * Sleep the CPU.
 @@ -184,6 +179,8 @@ machine_halt(void)
  void
  machine_power_off(void)
  {
 +   do_kernel_power_off();
 +
 common_shutdown(LINUX_REBOOT_CMD_POWER_OFF, NULL);
  }

 diff --git a/arch/arc/kernel/reset.c b/arch/arc/kernel/reset.c
 index 2768fa1..0758d9d 100644
 --- a/arch/arc/kernel/reset.c
 +++ b/arch/arc/kernel/reset.c
 @@ -26,9 +26,6 @@ void machine_restart(char *__unused)

  void machine_power_off(void)
  {
 -   /* FIXME ::  power off ??? */
 +   do_kernel_power_off();
 machine_halt();
  }
 -
 -void (*pm_power_off) (void) = NULL;
 -EXPORT_SYMBOL(pm_power_off);
 diff --git a/arch/arm/kernel/process.c b/arch/arm/kernel/process.c
 index fe972a2..aa3f656 100644
 --- a/arch/arm/kernel/process.c
 +++ b/arch/arm/kernel/process.c
 @@ -117,8 +117,6 @@ void soft_restart(unsigned long addr)
  /*
   * Function pointers to optional machine specific functions
   */
 -void (*pm_power_off)(void);
 -EXPORT_SYMBOL(pm_power_off);

  void (*arm_pm_restart)(enum reboot_mode reboot_mode, const char *cmd);

 @@ -205,8 +203,7 @@ void machine_power_off(void)
 local_irq_disable();
 smp_send_stop();

 -   if (pm_power_off)
 -   pm_power_off();
 +   do_kernel_power_off();
  }

  /*
 diff --git a/arch/arm64/kernel/process.c b/arch/arm64/kernel/process.c
 index c3065db..46a483a 100644
 --- a/arch/arm64/kernel/process.c
 +++ b/arch/arm64/kernel/process.c
 @@ -68,8 +68,6 @@ void soft_restart(unsigned long addr)
  /*
   * Function pointers to optional machine specific functions
   */
 -void (*pm_power_off)(void);
 -EXPORT_SYMBOL_GPL(pm_power_off);

  void (*arm_pm_restart)(enum reboot_mode reboot_mode

[GIT PULL] S+core : Patches for building allmodconfig and for removing useless flags (Resend/with a signed tag)

[Lennox Wu]

Please pull these updates for S+core architecture.
Three of the patches are for building allmodconfig, and the others are
for removing useless flags. Thanks.

P.S. Sorry for the previous version without a signature.

The following changes since commit 58586869599f6bb38aeca71a847cd77bfea74808:

  Merge tag 'pm+acpi-3.17-final' of
git://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm
(2014-10-02 18:47:28 -0700)

are available in the git repository at:


  git://github.com/sctscore/linux-off.git tags/for-linus-20141006

for you to fetch changes up to 1742b630e43a6ccca67cf634704e8b76d128947f:

  score: Remove GENERIC_HAS_IOMAP (2014-10-05 04:38:08 +0800)


PULL Request issued on 2014/10/6
Signed by Lennox Wu.


Chen Gang (3):
  arch: score: Export necessary symbols in related files
  arch/score/mm/cache.c: Export 'flush_icache_range'
  arch/score/include/asm/Kbuild: Add generic "serial.h"

Michael Opdenacker (1):
  score: remove deprecated IRQF_DISABLED

Richard Weinberger (1):
  score: Remove GENERIC_HAS_IOMAP

 arch/score/Kconfig | 3 ---
 arch/score/include/asm/Kbuild  | 1 +
 arch/score/kernel/time.c   | 2 +-
 arch/score/lib/checksum_copy.c | 1 +
 arch/score/mm/cache.c  | 2 ++
 5 files changed, 5 insertions(+), 4 deletions(-)




signature.asc
Description: OpenPGP digital signature

[GIT PULL] S+core : Patches for building allmodconfig and for removing useless flags (Resend/with a signed tag)

[Lennox Wu]

Please pull these updates for S+core architecture.
Three of the patches are for building allmodconfig, and the others are
for removing useless flags. Thanks.

P.S. Sorry for the previous version without a signature.

The following changes since commit 58586869599f6bb38aeca71a847cd77bfea74808:

  Merge tag 'pm+acpi-3.17-final' of
git://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm
(2014-10-02 18:47:28 -0700)

are available in the git repository at:


  git://github.com/sctscore/linux-off.git tags/for-linus-20141006

for you to fetch changes up to 1742b630e43a6ccca67cf634704e8b76d128947f:

  score: Remove GENERIC_HAS_IOMAP (2014-10-05 04:38:08 +0800)


PULL Request issued on 2014/10/6
Signed by Lennox Wu.


Chen Gang (3):
  arch: score: Export necessary symbols in related files
  arch/score/mm/cache.c: Export 'flush_icache_range'
  arch/score/include/asm/Kbuild: Add generic serial.h

Michael Opdenacker (1):
  score: remove deprecated IRQF_DISABLED

Richard Weinberger (1):
  score: Remove GENERIC_HAS_IOMAP

 arch/score/Kconfig | 3 ---
 arch/score/include/asm/Kbuild  | 1 +
 arch/score/kernel/time.c   | 2 +-
 arch/score/lib/checksum_copy.c | 1 +
 arch/score/mm/cache.c  | 2 ++
 5 files changed, 5 insertions(+), 4 deletions(-)




signature.asc
Description: OpenPGP digital signature

[GIT PULL] S+core : Patches for building allmodconfig and for removing useless

[Lennox Wu]

Please pull these updates for S+core architecture.
Three of the patches are for building allmodconfig, and the others are
for removing useless flags. Thanks.


The following changes since commit 58586869599f6
bb38aeca71a847cd77bfea74808:

  Merge tag 'pm+acpi-3.17-final' of
git://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm
(2014-10-02 18:47:28 -0700)

are available in the git repository at:


  git://github.com/sctscore/linux-off.git tags/for-linus

for you to fetch changes up to 1742b630e43a6ccca67cf634704e8b76d128947f:

  score: Remove GENERIC_HAS_IOMAP (2014-10-05 04:38:08 +0800)


Chen Gang (3):
  arch: score: Export   'flush_icache_range'
  arch/score/include/asm/Kbuild: Add generic "serial.h"

Michael Opdenacker (1):
  score: remove deprecated IRQF_DISABLED

Richard Weinberger (1):
  score: Remove GENERIC_HAS_IOMAP

 arch/score/Kconfig | 3 ---
 arch/score/include/asm/Kbuild  | 1 +
 arch/score/kernel/time.c   | 2 +-
 arch/score/lib/checksum_copy.c | 1 +
 arch/score/mm/cache.c  | 2 ++
 5 files changed, 5 insertions(+), 4 deletions(-)



signature.asc
Description: OpenPGP digital signature

[GIT PULL] S+core : Patches for building allmodconfig and for removing useless

[Lennox Wu]

Please pull these updates for S+core architecture.
Three of the patches are for building allmodconfig, and the others are
for removing useless flags. Thanks.


The following changes since commit 58586869599f6
bb38aeca71a847cd77bfea74808:

  Merge tag 'pm+acpi-3.17-final' of
git://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm
(2014-10-02 18:47:28 -0700)

are available in the git repository at:


  git://github.com/sctscore/linux-off.git tags/for-linus

for you to fetch changes up to 1742b630e43a6ccca67cf634704e8b76d128947f:

  score: Remove GENERIC_HAS_IOMAP (2014-10-05 04:38:08 +0800)


Chen Gang (3):
  arch: score: Export   'flush_icache_range'
  arch/score/include/asm/Kbuild: Add generic serial.h

Michael Opdenacker (1):
  score: remove deprecated IRQF_DISABLED

Richard Weinberger (1):
  score: Remove GENERIC_HAS_IOMAP

 arch/score/Kconfig | 3 ---
 arch/score/include/asm/Kbuild  | 1 +
 arch/score/kernel/time.c   | 2 +-
 arch/score/lib/checksum_copy.c | 1 +
 arch/score/mm/cache.c  | 2 ++
 5 files changed, 5 insertions(+), 4 deletions(-)



signature.asc
Description: OpenPGP digital signature

Re: [PATCH] [RESEND] score: remove deprecated IRQF_DISABLED

[Lennox Wu]

Hi Michael,
I will merge it and then issue a PULL request this weekend.
Thanks.

2014-10-01 18:04 GMT+08:00 Michael Opdenacker
:
> This patch removes the use of the IRQF_DISABLED flag
> from arch/score/kernel/time.c
>
> It's a NOOP since 2.6.35 and it will be removed one day.
>
> Signed-off-by: Michael Opdenacker 
> Acked-by: Lennox Wu 
> ---
>  arch/score/kernel/time.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/score/kernel/time.c b/arch/score/kernel/time.c
> index f0a43affb201..24770cd9b473 100644
> --- a/arch/score/kernel/time.c
> +++ b/arch/score/kernel/time.c
> @@ -41,7 +41,7 @@ static irqreturn_t timer_interrupt(int irq, void *dev_id)
>
>  static struct irqaction timer_irq = {
> .handler = timer_interrupt,
> -   .flags = IRQF_DISABLED | IRQF_TIMER,
> +   .flags = IRQF_TIMER,
> .name = "timer",
>  };
>
> --
> 1.9.1
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] [RESEND] score: remove deprecated IRQF_DISABLED

[Lennox Wu]

Hi Michael,
I will merge it and then issue a PULL request this weekend.
Thanks.

2014-10-01 18:04 GMT+08:00 Michael Opdenacker
michael.opdenac...@free-electrons.com:
 This patch removes the use of the IRQF_DISABLED flag
 from arch/score/kernel/time.c

 It's a NOOP since 2.6.35 and it will be removed one day.

 Signed-off-by: Michael Opdenacker michael.opdenac...@free-electrons.com
 Acked-by: Lennox Wu lennox...@gmail.com
 ---
  arch/score/kernel/time.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

 diff --git a/arch/score/kernel/time.c b/arch/score/kernel/time.c
 index f0a43affb201..24770cd9b473 100644
 --- a/arch/score/kernel/time.c
 +++ b/arch/score/kernel/time.c
 @@ -41,7 +41,7 @@ static irqreturn_t timer_interrupt(int irq, void *dev_id)

  static struct irqaction timer_irq = {
 .handler = timer_interrupt,
 -   .flags = IRQF_DISABLED | IRQF_TIMER,
 +   .flags = IRQF_TIMER,
 .name = timer,
  };

 --
 1.9.1

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH 16/28] Remove GENERIC_HAS_IOMAP

[Lennox Wu]

The patch was applied in our tree, but we have not issued a pull
request because other patches are still under discussion.
We might issue a pull request first.

Best,
Lennox

2014-09-05 22:00 GMT+08:00 Richard Weinberger :
> Am 05.09.2014 15:57, schrieb Guenter Roeck:
>> On 09/05/2014 02:32 AM, Paul Bolle wrote:
>>> Guenter, Richard,
>>>
>>> On Thu, 2014-06-19 at 03:53 +0800, Lennox Wu wrote:
>>>> 2014-06-18 17:10 GMT+08:00 Paul Bolle :
>>>>>> On Wed, 2014-04-16 at 20:30 +0200, Paul Bolle wrote:
>>>>>> On Thu, 2014-04-17 at 01:29 +0800, Lennox Wu wrote:
>>>>>>> OK, we will merge it into our tree.
>>>>>
>>>>> The last few messages in this thread were sent shortly after v3.15-rc1.
>>>>> We're now at v3.16-rc1 and those three references to GENERIC_HAS_IOMAP
>>>>> are still there.
>>>>>
>>>>> Are there any issues with this patch?
>>>>>
>>>>>
>>>>> Paul Bolle
>>>>>
>>>> NO, there is no issue. We just want to push patches in a batched way.
>>>> The patch will show recently. Thank you.
>>>
>>> And at v3.17-rc3 this patch has still not shown up in mainline. Do you
>>> have suggestions for an alternative route to get this triviality finally
>>> merged?
>>>
>>
>> I merged it with the latest kernel, added Lennox' Ack, and sent it to
>> triv...@kernel.org. Let's see what happens.
>
> Thanks a lot Guenther!
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH 16/28] Remove GENERIC_HAS_IOMAP

[Lennox Wu]

The patch was applied in our tree, but we have not issued a pull
request because other patches are still under discussion.
We might issue a pull request first.

Best,
Lennox

2014-09-05 22:00 GMT+08:00 Richard Weinberger rich...@nod.at:
 Am 05.09.2014 15:57, schrieb Guenter Roeck:
 On 09/05/2014 02:32 AM, Paul Bolle wrote:
 Guenter, Richard,

 On Thu, 2014-06-19 at 03:53 +0800, Lennox Wu wrote:
 2014-06-18 17:10 GMT+08:00 Paul Bolle pebo...@tiscali.nl:
 On Wed, 2014-04-16 at 20:30 +0200, Paul Bolle wrote:
 On Thu, 2014-04-17 at 01:29 +0800, Lennox Wu wrote:
 OK, we will merge it into our tree.

 The last few messages in this thread were sent shortly after v3.15-rc1.
 We're now at v3.16-rc1 and those three references to GENERIC_HAS_IOMAP
 are still there.

 Are there any issues with this patch?


 Paul Bolle

 NO, there is no issue. We just want to push patches in a batched way.
 The patch will show recently. Thank you.

 And at v3.17-rc3 this patch has still not shown up in mainline. Do you
 have suggestions for an alternative route to get this triviality finally
 merged?


 I merged it with the latest kernel, added Lennox' Ack, and sent it to
 triv...@kernel.org. Let's see what happens.

 Thanks a lot Guenther!
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH v2] arch/score/kernel: Add 'score_ksyms.c' to export special symbols

[Lennox Wu]

Acked-by: Lennox Wu

It was merged into Score's git tree, and it will show in the next PULL 
request.


Best,
Lennox

Chen Gang 於 2014/7/9 上午 08:19 寫道:

Some symbols are not implemented in C, or not in "arch/score", but
score need export them, so add 'score_ksyms.c' to export them, just
like other architectures have done.

The related error (with allmodconfig under score):

MODPOST 1365 modules
   ERROR: "__clear_user" [drivers/md/dm-mod.ko] undefined!
   ERROR: "__copy_tofrom_user" [net/x25/x25.ko] undefined!
   ERROR: "__strncpy_from_user" [fs/xfs/xfs.ko] undefined!
   ERROR: "csum_partial" [net/unix/unix.ko] undefined!

   ERROR: "max_low_pfn" [net/wireless/lib80211_crypt_tkip.ko] undefined!
   ERROR: "min_low_pfn" [net/wireless/lib80211_crypt_tkip.ko] undefined!


Signed-off-by: Chen Gang 
---
  arch/score/kernel/Makefile  |  2 +-
  arch/score/kernel/score_ksyms.c | 41 +
  2 files changed, 42 insertions(+), 1 deletion(-)
  create mode 100644 arch/score/kernel/score_ksyms.c

diff --git a/arch/score/kernel/Makefile b/arch/score/kernel/Makefile
index fb1802b..891133b 100644
--- a/arch/score/kernel/Makefile
+++ b/arch/score/kernel/Makefile
@@ -6,6 +6,6 @@ extra-y := head.o vmlinux.lds

  obj-y += entry.o irq.o process.o ptrace.o \
setup.o signal.o sys_score.o time.o traps.o \
-   sys_call_table.o
+   sys_call_table.o score_ksyms.o

  obj-$(CONFIG_MODULES) += module.o
diff --git a/arch/score/kernel/score_ksyms.c b/arch/score/kernel/score_ksyms.c
new file mode 100644
index 000..d093774
--- /dev/null
+++ b/arch/score/kernel/score_ksyms.c
@@ -0,0 +1,41 @@
+/*
+ * arch/score/kernel/score_ksyms.c
+ *
+ * Score Processor version.
+ *
+ * No Copyright (C)
+ *  Chen Gang 
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, see the file COPYING, or write
+ * to the Free Software Foundation, Inc.,
+ * 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
+ */
+
+#include 
+#include 
+#include 
+
+/*
+ * Assembly functions that may be used (directly or indirectly) by modules
+ */
+EXPORT_SYMBOL(__clear_user);
+EXPORT_SYMBOL(__copy_tofrom_user);
+EXPORT_SYMBOL(__strncpy_from_user);
+EXPORT_SYMBOL(csum_partial);
+
+/*
+ * Define but not export in generic mm sub-systems, but score need export
+ */
+EXPORT_SYMBOL(min_low_pfn);
+EXPORT_SYMBOL(max_low_pfn);


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] arch: score: Export necessary symbols in related files

[Lennox Wu]

Yes, I will issue a PULL request to Linus.
Best,
Lennox

Chen Gang 於 2014/8/2 下午 04:18 寫道:

On 08/02/2014 02:20 PM, Lennox Wu wrote:

It was merged into Score's git tree, and it will show in the next PULL
request.



OK, thanks. And next, will it be integrated into upstream main line? (I
guess it will be).


Thanks


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH 18/43] score: Use get_signal() signal_setup_done()

[Lennox Wu]

It was merged into Score's git tree, and it will show in the next PULL 
request.


Best,
Lennox

Lennox Wu 於 2014/7/21 下午 10:47 寫道:

I am picking up some missed patches and merging them into our git
tree. If the patch is missed, we will merge it.

Best,
Lennox

2014-07-21 22:03 GMT+08:00 Richard Weinberger :

From: Richard Weinberger 

Use the more generic functions get_signal() signal_setup_done()
for signal delivery.

Acked-by: Lennox Wu 
Signed-off-by: Richard Weinberger 
---
  arch/score/kernel/signal.c | 43 ++-
  1 file changed, 18 insertions(+), 25 deletions(-)

diff --git a/arch/score/kernel/signal.c b/arch/score/kernel/signal.c
index a00fba3..1651807 100644
--- a/arch/score/kernel/signal.c
+++ b/arch/score/kernel/signal.c
@@ -173,15 +173,15 @@ badframe:
 return 0;
  }

-static int setup_rt_frame(struct k_sigaction *ka, struct pt_regs *regs,
-   int signr, sigset_t *set, siginfo_t *info)
+static int setup_rt_frame(struct ksignal *ksig, struct pt_regs *regs,
+ sigset_t *set)
  {
 struct rt_sigframe __user *frame;
 int err = 0;

-   frame = get_sigframe(ka, regs, sizeof(*frame));
+   frame = get_sigframe(>ka, regs, sizeof(*frame));
 if (!access_ok(VERIFY_WRITE, frame, sizeof(*frame)))
-   goto give_sigsegv;
+   return -EFAULT;

 /*
  * Set up the return code ...
@@ -194,7 +194,7 @@ static int setup_rt_frame(struct k_sigaction *ka, struct 
pt_regs *regs,
 err |= __put_user(0x80008002, frame->rs_code + 1);
 flush_cache_sigtramp((unsigned long) frame->rs_code);

-   err |= copy_siginfo_to_user(>rs_info, info);
+   err |= copy_siginfo_to_user(>rs_info, >info);
 err |= __put_user(0, >rs_uc.uc_flags);
 err |= __put_user(NULL, >rs_uc.uc_link);
 err |= __save_altstack(>rs_uc.uc_stack, regs->regs[0]);
@@ -202,26 +202,23 @@ static int setup_rt_frame(struct k_sigaction *ka, struct 
pt_regs *regs,
 err |= __copy_to_user(>rs_uc.uc_sigmask, set, sizeof(*set));

 if (err)
-   goto give_sigsegv;
+   return -EFAULT;

 regs->regs[0] = (unsigned long) frame;
 regs->regs[3] = (unsigned long) frame->rs_code;
-   regs->regs[4] = signr;
+   regs->regs[4] = ksig->sig;
 regs->regs[5] = (unsigned long) >rs_info;
 regs->regs[6] = (unsigned long) >rs_uc;
-   regs->regs[29] = (unsigned long) ka->sa.sa_handler;
-   regs->cp0_epc = (unsigned long) ka->sa.sa_handler;
+   regs->regs[29] = (unsigned long) ksig->ka.sa.sa_handler;
+   regs->cp0_epc = (unsigned long) ksig->ka.sa.sa_handler;

 return 0;
-
-give_sigsegv:
-   force_sigsegv(signr, current);
-   return -EFAULT;
  }

-static void handle_signal(unsigned long sig, siginfo_t *info,
-   struct k_sigaction *ka, struct pt_regs *regs)
+static void handle_signal(struct ksignal *ksig, struct pt_regs *regs)
  {
+   int ret;
+
 if (regs->is_syscall) {
 switch (regs->regs[4]) {
 case ERESTART_RESTARTBLOCK:
@@ -229,7 +226,7 @@ static void handle_signal(unsigned long sig, siginfo_t 
*info,
 regs->regs[4] = EINTR;
 break;
 case ERESTARTSYS:
-   if (!(ka->sa.sa_flags & SA_RESTART)) {
+   if (!(ksig->ka.sa.sa_flags & SA_RESTART)) {
 regs->regs[4] = EINTR;
 break;
 }
@@ -245,17 +242,14 @@ static void handle_signal(unsigned long sig, siginfo_t 
*info,
 /*
  * Set up the stack frame
  */
-   if (setup_rt_frame(ka, regs, sig, sigmask_to_save(), info) < 0)
-   return;
+   ret = setup_rt_frame(ksig, regs, sigmask_to_save());

-   signal_delivered(sig, info, ka, regs, 0);
+   signal_setup_done(ret, ksig, 0);
  }

  static void do_signal(struct pt_regs *regs)
  {
-   struct k_sigaction ka;
-   siginfo_t info;
-   int signr;
+   struct ksignal ksig;

 /*
  * We want the common case to go fast, which is why we may in certain
@@ -265,10 +259,9 @@ static void do_signal(struct pt_regs *regs)
 if (!user_mode(regs))
 return;

-   signr = get_signal_to_deliver(, , regs, NULL);
-   if (signr > 0) {
+   if (get_signal()) {
 /* Actually deliver the signal.  */
-   handle_signal(signr, , , regs);
+   handle_signal(, regs);
 return;
 }

--
1.8.4.5


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] arch: score: Export necessary symbols in related files

[Lennox Wu]

It was merged into Score's git tree, and it will show in the next PULL 
request.


Best,
Lennox

Chen Gang 於 2014/7/17 下午 05:01 寫道:

On 07/17/2014 10:22 AM, Lennox Wu wrote:

  Acked-by: Lennox Wu



OK, thank you for your work.



2014-07-09 16:48 GMT+08:00 Chen Gang :

'csum_partial_copy_from_user' and 'flush_dcache_page' are also needed by
outside modules, so need export them in the related files.

The related error (with allmodconfig under score):

 MODPOST 1365 modules
   ERROR: "csum_partial_copy_from_user" [net/rxrpc/af-rxrpc.ko] undefined!
   ERROR: "flush_dcache_page" [net/sunrpc/sunrpc.ko] undefined!


Signed-off-by: Chen Gang 
---
  arch/score/lib/checksum_copy.c | 1 +
  arch/score/mm/cache.c  | 1 +
  2 files changed, 2 insertions(+)

diff --git a/arch/score/lib/checksum_copy.c b/arch/score/lib/checksum_copy.c
index 04565dd..9b770b3 100644
--- a/arch/score/lib/checksum_copy.c
+++ b/arch/score/lib/checksum_copy.c
@@ -50,3 +50,4 @@ unsigned int csum_partial_copy_from_user(const char *src, 
char *dst,

 return csum_partial(dst, len, sum);
  }
+EXPORT_SYMBOL(csum_partial_copy_from_user);
diff --git a/arch/score/mm/cache.c b/arch/score/mm/cache.c
index f85ec1a..be91041 100644
--- a/arch/score/mm/cache.c
+++ b/arch/score/mm/cache.c
@@ -72,6 +72,7 @@ void flush_dcache_page(struct page *page)
 addr = (unsigned long) page_address(page);
 flush_data_cache_page(addr);
  }
+EXPORT_SYMBOL(flush_dcache_page);

  /* called by update_mmu_cache. */
  void __update_cache(struct vm_area_struct *vma, unsigned long address,
--
1.9.2.459.g68773ac



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] arch: score: Export necessary symbols in related files

[Lennox Wu]

It was merged into Score's git tree, and it will show in the next PULL 
request.


Best,
Lennox

Chen Gang 於 2014/7/17 下午 05:01 寫道:

On 07/17/2014 10:22 AM, Lennox Wu wrote:

  Acked-by: Lennox Wulennox...@gmail.com



OK, thank you for your work.



2014-07-09 16:48 GMT+08:00 Chen Gang gang.chen.5...@gmail.com:

'csum_partial_copy_from_user' and 'flush_dcache_page' are also needed by
outside modules, so need export them in the related files.

The related error (with allmodconfig under score):

 MODPOST 1365 modules
   ERROR: csum_partial_copy_from_user [net/rxrpc/af-rxrpc.ko] undefined!
   ERROR: flush_dcache_page [net/sunrpc/sunrpc.ko] undefined!


Signed-off-by: Chen Gang gang.chen.5...@gmail.com
---
  arch/score/lib/checksum_copy.c | 1 +
  arch/score/mm/cache.c  | 1 +
  2 files changed, 2 insertions(+)

diff --git a/arch/score/lib/checksum_copy.c b/arch/score/lib/checksum_copy.c
index 04565dd..9b770b3 100644
--- a/arch/score/lib/checksum_copy.c
+++ b/arch/score/lib/checksum_copy.c
@@ -50,3 +50,4 @@ unsigned int csum_partial_copy_from_user(const char *src, 
char *dst,

 return csum_partial(dst, len, sum);
  }
+EXPORT_SYMBOL(csum_partial_copy_from_user);
diff --git a/arch/score/mm/cache.c b/arch/score/mm/cache.c
index f85ec1a..be91041 100644
--- a/arch/score/mm/cache.c
+++ b/arch/score/mm/cache.c
@@ -72,6 +72,7 @@ void flush_dcache_page(struct page *page)
 addr = (unsigned long) page_address(page);
 flush_data_cache_page(addr);
  }
+EXPORT_SYMBOL(flush_dcache_page);

  /* called by update_mmu_cache. */
  void __update_cache(struct vm_area_struct *vma, unsigned long address,
--
1.9.2.459.g68773ac



--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH 18/43] score: Use get_signal() signal_setup_done()

[Lennox Wu]

It was merged into Score's git tree, and it will show in the next PULL 
request.


Best,
Lennox

Lennox Wu 於 2014/7/21 下午 10:47 寫道:

I am picking up some missed patches and merging them into our git
tree. If the patch is missed, we will merge it.

Best,
Lennox

2014-07-21 22:03 GMT+08:00 Richard Weinberger rich...@sigma-star.at:

From: Richard Weinberger rich...@nod.at

Use the more generic functions get_signal() signal_setup_done()
for signal delivery.

Acked-by: Lennox Wu lennox...@gmail.com
Signed-off-by: Richard Weinberger rich...@nod.at
---
  arch/score/kernel/signal.c | 43 ++-
  1 file changed, 18 insertions(+), 25 deletions(-)

diff --git a/arch/score/kernel/signal.c b/arch/score/kernel/signal.c
index a00fba3..1651807 100644
--- a/arch/score/kernel/signal.c
+++ b/arch/score/kernel/signal.c
@@ -173,15 +173,15 @@ badframe:
 return 0;
  }

-static int setup_rt_frame(struct k_sigaction *ka, struct pt_regs *regs,
-   int signr, sigset_t *set, siginfo_t *info)
+static int setup_rt_frame(struct ksignal *ksig, struct pt_regs *regs,
+ sigset_t *set)
  {
 struct rt_sigframe __user *frame;
 int err = 0;

-   frame = get_sigframe(ka, regs, sizeof(*frame));
+   frame = get_sigframe(ksig-ka, regs, sizeof(*frame));
 if (!access_ok(VERIFY_WRITE, frame, sizeof(*frame)))
-   goto give_sigsegv;
+   return -EFAULT;

 /*
  * Set up the return code ...
@@ -194,7 +194,7 @@ static int setup_rt_frame(struct k_sigaction *ka, struct 
pt_regs *regs,
 err |= __put_user(0x80008002, frame-rs_code + 1);
 flush_cache_sigtramp((unsigned long) frame-rs_code);

-   err |= copy_siginfo_to_user(frame-rs_info, info);
+   err |= copy_siginfo_to_user(frame-rs_info, ksig-info);
 err |= __put_user(0, frame-rs_uc.uc_flags);
 err |= __put_user(NULL, frame-rs_uc.uc_link);
 err |= __save_altstack(frame-rs_uc.uc_stack, regs-regs[0]);
@@ -202,26 +202,23 @@ static int setup_rt_frame(struct k_sigaction *ka, struct 
pt_regs *regs,
 err |= __copy_to_user(frame-rs_uc.uc_sigmask, set, sizeof(*set));

 if (err)
-   goto give_sigsegv;
+   return -EFAULT;

 regs-regs[0] = (unsigned long) frame;
 regs-regs[3] = (unsigned long) frame-rs_code;
-   regs-regs[4] = signr;
+   regs-regs[4] = ksig-sig;
 regs-regs[5] = (unsigned long) frame-rs_info;
 regs-regs[6] = (unsigned long) frame-rs_uc;
-   regs-regs[29] = (unsigned long) ka-sa.sa_handler;
-   regs-cp0_epc = (unsigned long) ka-sa.sa_handler;
+   regs-regs[29] = (unsigned long) ksig-ka.sa.sa_handler;
+   regs-cp0_epc = (unsigned long) ksig-ka.sa.sa_handler;

 return 0;
-
-give_sigsegv:
-   force_sigsegv(signr, current);
-   return -EFAULT;
  }

-static void handle_signal(unsigned long sig, siginfo_t *info,
-   struct k_sigaction *ka, struct pt_regs *regs)
+static void handle_signal(struct ksignal *ksig, struct pt_regs *regs)
  {
+   int ret;
+
 if (regs-is_syscall) {
 switch (regs-regs[4]) {
 case ERESTART_RESTARTBLOCK:
@@ -229,7 +226,7 @@ static void handle_signal(unsigned long sig, siginfo_t 
*info,
 regs-regs[4] = EINTR;
 break;
 case ERESTARTSYS:
-   if (!(ka-sa.sa_flags  SA_RESTART)) {
+   if (!(ksig-ka.sa.sa_flags  SA_RESTART)) {
 regs-regs[4] = EINTR;
 break;
 }
@@ -245,17 +242,14 @@ static void handle_signal(unsigned long sig, siginfo_t 
*info,
 /*
  * Set up the stack frame
  */
-   if (setup_rt_frame(ka, regs, sig, sigmask_to_save(), info)  0)
-   return;
+   ret = setup_rt_frame(ksig, regs, sigmask_to_save());

-   signal_delivered(sig, info, ka, regs, 0);
+   signal_setup_done(ret, ksig, 0);
  }

  static void do_signal(struct pt_regs *regs)
  {
-   struct k_sigaction ka;
-   siginfo_t info;
-   int signr;
+   struct ksignal ksig;

 /*
  * We want the common case to go fast, which is why we may in certain
@@ -265,10 +259,9 @@ static void do_signal(struct pt_regs *regs)
 if (!user_mode(regs))
 return;

-   signr = get_signal_to_deliver(info, ka, regs, NULL);
-   if (signr  0) {
+   if (get_signal(ksig)) {
 /* Actually deliver the signal.  */
-   handle_signal(signr, info, ka, regs);
+   handle_signal(ksig, regs);
 return;
 }

--
1.8.4.5


--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org

Re: [PATCH] arch: score: Export necessary symbols in related files

[Lennox Wu]

Yes, I will issue a PULL request to Linus.
Best,
Lennox

Chen Gang 於 2014/8/2 下午 04:18 寫道:

On 08/02/2014 02:20 PM, Lennox Wu wrote:

It was merged into Score's git tree, and it will show in the next PULL
request.



OK, thanks. And next, will it be integrated into upstream main line? (I
guess it will be).


Thanks


--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH v2] arch/score/kernel: Add 'score_ksyms.c' to export special symbols

[Lennox Wu]

Acked-by: Lennox Wulennox...@gmail.com

It was merged into Score's git tree, and it will show in the next PULL 
request.


Best,
Lennox

Chen Gang 於 2014/7/9 上午 08:19 寫道:

Some symbols are not implemented in C, or not in arch/score, but
score need export them, so add 'score_ksyms.c' to export them, just
like other architectures have done.

The related error (with allmodconfig under score):

MODPOST 1365 modules
   ERROR: __clear_user [drivers/md/dm-mod.ko] undefined!
   ERROR: __copy_tofrom_user [net/x25/x25.ko] undefined!
   ERROR: __strncpy_from_user [fs/xfs/xfs.ko] undefined!
   ERROR: csum_partial [net/unix/unix.ko] undefined!

   ERROR: max_low_pfn [net/wireless/lib80211_crypt_tkip.ko] undefined!
   ERROR: min_low_pfn [net/wireless/lib80211_crypt_tkip.ko] undefined!


Signed-off-by: Chen Gang gang.chen.5...@gmail.com
---
  arch/score/kernel/Makefile  |  2 +-
  arch/score/kernel/score_ksyms.c | 41 +
  2 files changed, 42 insertions(+), 1 deletion(-)
  create mode 100644 arch/score/kernel/score_ksyms.c

diff --git a/arch/score/kernel/Makefile b/arch/score/kernel/Makefile
index fb1802b..891133b 100644
--- a/arch/score/kernel/Makefile
+++ b/arch/score/kernel/Makefile
@@ -6,6 +6,6 @@ extra-y := head.o vmlinux.lds

  obj-y += entry.o irq.o process.o ptrace.o \
setup.o signal.o sys_score.o time.o traps.o \
-   sys_call_table.o
+   sys_call_table.o score_ksyms.o

  obj-$(CONFIG_MODULES) += module.o
diff --git a/arch/score/kernel/score_ksyms.c b/arch/score/kernel/score_ksyms.c
new file mode 100644
index 000..d093774
--- /dev/null
+++ b/arch/score/kernel/score_ksyms.c
@@ -0,0 +1,41 @@
+/*
+ * arch/score/kernel/score_ksyms.c
+ *
+ * Score Processor version.
+ *
+ * No Copyright (C)
+ *  Chen Gang gang.chen.5...@gmail.com
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, see the file COPYING, or write
+ * to the Free Software Foundation, Inc.,
+ * 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
+ */
+
+#include linux/export.h
+#include linux/syscalls.h
+#include asm/checksum.h
+
+/*
+ * Assembly functions that may be used (directly or indirectly) by modules
+ */
+EXPORT_SYMBOL(__clear_user);
+EXPORT_SYMBOL(__copy_tofrom_user);
+EXPORT_SYMBOL(__strncpy_from_user);
+EXPORT_SYMBOL(csum_partial);
+
+/*
+ * Define but not export in generic mm sub-systems, but score need export
+ */
+EXPORT_SYMBOL(min_low_pfn);
+EXPORT_SYMBOL(max_low_pfn);


--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH 18/43] score: Use get_signal() signal_setup_done()

[Lennox Wu]

I am picking up some missed patches and merging them into our git
tree. If the patch is missed, we will merge it.

Best,
Lennox

2014-07-21 22:03 GMT+08:00 Richard Weinberger :
> From: Richard Weinberger 
>
> Use the more generic functions get_signal() signal_setup_done()
> for signal delivery.
>
> Acked-by: Lennox Wu 
> Signed-off-by: Richard Weinberger 
> ---
>  arch/score/kernel/signal.c | 43 ++-
>  1 file changed, 18 insertions(+), 25 deletions(-)
>
> diff --git a/arch/score/kernel/signal.c b/arch/score/kernel/signal.c
> index a00fba3..1651807 100644
> --- a/arch/score/kernel/signal.c
> +++ b/arch/score/kernel/signal.c
> @@ -173,15 +173,15 @@ badframe:
> return 0;
>  }
>
> -static int setup_rt_frame(struct k_sigaction *ka, struct pt_regs *regs,
> -   int signr, sigset_t *set, siginfo_t *info)
> +static int setup_rt_frame(struct ksignal *ksig, struct pt_regs *regs,
> + sigset_t *set)
>  {
> struct rt_sigframe __user *frame;
> int err = 0;
>
> -   frame = get_sigframe(ka, regs, sizeof(*frame));
> +   frame = get_sigframe(>ka, regs, sizeof(*frame));
> if (!access_ok(VERIFY_WRITE, frame, sizeof(*frame)))
> -   goto give_sigsegv;
> +   return -EFAULT;
>
> /*
>  * Set up the return code ...
> @@ -194,7 +194,7 @@ static int setup_rt_frame(struct k_sigaction *ka, struct 
> pt_regs *regs,
> err |= __put_user(0x80008002, frame->rs_code + 1);
> flush_cache_sigtramp((unsigned long) frame->rs_code);
>
> -   err |= copy_siginfo_to_user(>rs_info, info);
> +   err |= copy_siginfo_to_user(>rs_info, >info);
> err |= __put_user(0, >rs_uc.uc_flags);
> err |= __put_user(NULL, >rs_uc.uc_link);
> err |= __save_altstack(>rs_uc.uc_stack, regs->regs[0]);
> @@ -202,26 +202,23 @@ static int setup_rt_frame(struct k_sigaction *ka, 
> struct pt_regs *regs,
> err |= __copy_to_user(>rs_uc.uc_sigmask, set, sizeof(*set));
>
> if (err)
> -   goto give_sigsegv;
> +   return -EFAULT;
>
> regs->regs[0] = (unsigned long) frame;
> regs->regs[3] = (unsigned long) frame->rs_code;
> -   regs->regs[4] = signr;
> +   regs->regs[4] = ksig->sig;
> regs->regs[5] = (unsigned long) >rs_info;
> regs->regs[6] = (unsigned long) >rs_uc;
> -   regs->regs[29] = (unsigned long) ka->sa.sa_handler;
> -   regs->cp0_epc = (unsigned long) ka->sa.sa_handler;
> +   regs->regs[29] = (unsigned long) ksig->ka.sa.sa_handler;
> +   regs->cp0_epc = (unsigned long) ksig->ka.sa.sa_handler;
>
> return 0;
> -
> -give_sigsegv:
> -   force_sigsegv(signr, current);
> -   return -EFAULT;
>  }
>
> -static void handle_signal(unsigned long sig, siginfo_t *info,
> -   struct k_sigaction *ka, struct pt_regs *regs)
> +static void handle_signal(struct ksignal *ksig, struct pt_regs *regs)
>  {
> +   int ret;
> +
> if (regs->is_syscall) {
> switch (regs->regs[4]) {
> case ERESTART_RESTARTBLOCK:
> @@ -229,7 +226,7 @@ static void handle_signal(unsigned long sig, siginfo_t 
> *info,
> regs->regs[4] = EINTR;
> break;
> case ERESTARTSYS:
> -   if (!(ka->sa.sa_flags & SA_RESTART)) {
> +   if (!(ksig->ka.sa.sa_flags & SA_RESTART)) {
> regs->regs[4] = EINTR;
> break;
> }
> @@ -245,17 +242,14 @@ static void handle_signal(unsigned long sig, siginfo_t 
> *info,
> /*
>  * Set up the stack frame
>  */
> -   if (setup_rt_frame(ka, regs, sig, sigmask_to_save(), info) < 0)
> -   return;
> +   ret = setup_rt_frame(ksig, regs, sigmask_to_save());
>
> -   signal_delivered(sig, info, ka, regs, 0);
> +   signal_setup_done(ret, ksig, 0);
>  }
>
>  static void do_signal(struct pt_regs *regs)
>  {
> -   struct k_sigaction ka;
> -   siginfo_t info;
> -   int signr;
> +   struct ksignal ksig;
>
> /*
>  * We want the common case to go fast, which is why we may in certain
> @@ -265,10 +259,9 @@ static void do_signal(struct pt_regs *regs)
> if (!user_mode(regs))
> return;
>
> -   signr = get_signal_to_deliver(, , regs, NULL);
> -   if (signr > 0) {
> +   if (get_signal()) {
> /* Actually deliver the signal.  */
> -   handle_signal(signr, , , regs);
> +   handle_signal(, regs);
> return;
> }
>
> --
> 1.8.4.5
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH 18/43] score: Use get_signal() signal_setup_done()

[Lennox Wu]

I am picking up some missed patches and merging them into our git
tree. If the patch is missed, we will merge it.

Best,
Lennox

2014-07-21 22:03 GMT+08:00 Richard Weinberger rich...@sigma-star.at:
 From: Richard Weinberger rich...@nod.at

 Use the more generic functions get_signal() signal_setup_done()
 for signal delivery.

 Acked-by: Lennox Wu lennox...@gmail.com
 Signed-off-by: Richard Weinberger rich...@nod.at
 ---
  arch/score/kernel/signal.c | 43 ++-
  1 file changed, 18 insertions(+), 25 deletions(-)

 diff --git a/arch/score/kernel/signal.c b/arch/score/kernel/signal.c
 index a00fba3..1651807 100644
 --- a/arch/score/kernel/signal.c
 +++ b/arch/score/kernel/signal.c
 @@ -173,15 +173,15 @@ badframe:
 return 0;
  }

 -static int setup_rt_frame(struct k_sigaction *ka, struct pt_regs *regs,
 -   int signr, sigset_t *set, siginfo_t *info)
 +static int setup_rt_frame(struct ksignal *ksig, struct pt_regs *regs,
 + sigset_t *set)
  {
 struct rt_sigframe __user *frame;
 int err = 0;

 -   frame = get_sigframe(ka, regs, sizeof(*frame));
 +   frame = get_sigframe(ksig-ka, regs, sizeof(*frame));
 if (!access_ok(VERIFY_WRITE, frame, sizeof(*frame)))
 -   goto give_sigsegv;
 +   return -EFAULT;

 /*
  * Set up the return code ...
 @@ -194,7 +194,7 @@ static int setup_rt_frame(struct k_sigaction *ka, struct 
 pt_regs *regs,
 err |= __put_user(0x80008002, frame-rs_code + 1);
 flush_cache_sigtramp((unsigned long) frame-rs_code);

 -   err |= copy_siginfo_to_user(frame-rs_info, info);
 +   err |= copy_siginfo_to_user(frame-rs_info, ksig-info);
 err |= __put_user(0, frame-rs_uc.uc_flags);
 err |= __put_user(NULL, frame-rs_uc.uc_link);
 err |= __save_altstack(frame-rs_uc.uc_stack, regs-regs[0]);
 @@ -202,26 +202,23 @@ static int setup_rt_frame(struct k_sigaction *ka, 
 struct pt_regs *regs,
 err |= __copy_to_user(frame-rs_uc.uc_sigmask, set, sizeof(*set));

 if (err)
 -   goto give_sigsegv;
 +   return -EFAULT;

 regs-regs[0] = (unsigned long) frame;
 regs-regs[3] = (unsigned long) frame-rs_code;
 -   regs-regs[4] = signr;
 +   regs-regs[4] = ksig-sig;
 regs-regs[5] = (unsigned long) frame-rs_info;
 regs-regs[6] = (unsigned long) frame-rs_uc;
 -   regs-regs[29] = (unsigned long) ka-sa.sa_handler;
 -   regs-cp0_epc = (unsigned long) ka-sa.sa_handler;
 +   regs-regs[29] = (unsigned long) ksig-ka.sa.sa_handler;
 +   regs-cp0_epc = (unsigned long) ksig-ka.sa.sa_handler;

 return 0;
 -
 -give_sigsegv:
 -   force_sigsegv(signr, current);
 -   return -EFAULT;
  }

 -static void handle_signal(unsigned long sig, siginfo_t *info,
 -   struct k_sigaction *ka, struct pt_regs *regs)
 +static void handle_signal(struct ksignal *ksig, struct pt_regs *regs)
  {
 +   int ret;
 +
 if (regs-is_syscall) {
 switch (regs-regs[4]) {
 case ERESTART_RESTARTBLOCK:
 @@ -229,7 +226,7 @@ static void handle_signal(unsigned long sig, siginfo_t 
 *info,
 regs-regs[4] = EINTR;
 break;
 case ERESTARTSYS:
 -   if (!(ka-sa.sa_flags  SA_RESTART)) {
 +   if (!(ksig-ka.sa.sa_flags  SA_RESTART)) {
 regs-regs[4] = EINTR;
 break;
 }
 @@ -245,17 +242,14 @@ static void handle_signal(unsigned long sig, siginfo_t 
 *info,
 /*
  * Set up the stack frame
  */
 -   if (setup_rt_frame(ka, regs, sig, sigmask_to_save(), info)  0)
 -   return;
 +   ret = setup_rt_frame(ksig, regs, sigmask_to_save());

 -   signal_delivered(sig, info, ka, regs, 0);
 +   signal_setup_done(ret, ksig, 0);
  }

  static void do_signal(struct pt_regs *regs)
  {
 -   struct k_sigaction ka;
 -   siginfo_t info;
 -   int signr;
 +   struct ksignal ksig;

 /*
  * We want the common case to go fast, which is why we may in certain
 @@ -265,10 +259,9 @@ static void do_signal(struct pt_regs *regs)
 if (!user_mode(regs))
 return;

 -   signr = get_signal_to_deliver(info, ka, regs, NULL);
 -   if (signr  0) {
 +   if (get_signal(ksig)) {
 /* Actually deliver the signal.  */
 -   handle_signal(signr, info, ka, regs);
 +   handle_signal(ksig, regs);
 return;
 }

 --
 1.8.4.5

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

[Lennox Wu]

Score can provide dummy functions if HAS_IOMEM  and NO_IOMEM will be
removed, even if we indeed have no IOMEM.

Best,
Lennox

2014-07-18 18:51 GMT+08:00 Richard Weinberger :
> Am 18.07.2014 12:44, schrieb Chen Gang:
>> On 07/18/2014 03:35 PM, Richard Weinberger wrote:
>>> Am 18.07.2014 02:36, schrieb Chen Gang:

 On 07/18/2014 02:09 AM, Richard Weinberger wrote:
> Am 17.07.2014 12:48, schrieb Arnd Bergmann:
>> AFAICT, NO_IOMEM only has a real purpose on UML these days. Could we take
>> a shortcut here and make COMPILE_TEST depend on !UML? Getting random 
>> stuff
>> to build on UML seems pointless to me and we special-case it in a number 
>> of
>> places already.
>
> If UML is the only arch without io memory the dependency on !UML seems
> reasonable to me. :-)
>

 For me, if only uml left, I suggest to implement dummy functions within
 uml instead of let CONFIG_UML appear in generic include directory. And
 then remove all HAS_IOMEM and NO_IOMEM from kernel.
>>>
>>> Erm, this is something completely different.
>>> I thought we're focusing on COMPILE_TEST?
>>>
>>
>> COMPILE_TEST is none-architecture specific, but UML is. So in generic
>> include folder, if we're focusing on choosing whether COMPILE_TEST or
>> UML, for me, I will choose COMPILE_TEST.
>>
>> If we're not only focusing on COMPILE_TEST, for me, if something only
>> depend on one architecture, I'd like to put them under "arch/*/" folder.
>>
>> Especially, after that, we can remove all HAS_IOMEM and NO_IOMEM, nobody
>> has to think of them again. :-)
>
> And then we end up with a solution that on UML a lot of completely useless
> drivers are build which fail in various interesting manners because you'll
> add stubs for all kinds of io memory related functions to arch/um/?
> We had this kind of discussion already. You'll need more than ioremap...
>
> I like Arnd's idea *much* more to make COMPILE_TEST depend on !UML.
>
> Thanks,
> //richard
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH 31/38] score: clean-up uapi Kbuild file

[Lennox Wu]

 Acked-by: Lennox Wu 

2014-07-14 23:08 GMT+08:00 Sam Ravnborg :
> Remove redundant assignments.
>
> Signed-off-by: Sam Ravnborg 
> Cc: Chen Liqin 
> Cc: Lennox Wu 
> ---
>  arch/score/include/uapi/asm/Kbuild | 32 
>  1 file changed, 32 deletions(-)
>
> diff --git a/arch/score/include/uapi/asm/Kbuild 
> b/arch/score/include/uapi/asm/Kbuild
> index 040178c..b15bf6b 100644
> --- a/arch/score/include/uapi/asm/Kbuild
> +++ b/arch/score/include/uapi/asm/Kbuild
> @@ -1,34 +1,2 @@
>  # UAPI Header export list
>  include include/uapi/asm-generic/Kbuild.asm
> -
> -header-y += auxvec.h
> -header-y += bitsperlong.h
> -header-y += byteorder.h
> -header-y += errno.h
> -header-y += fcntl.h
> -header-y += ioctl.h
> -header-y += ioctls.h
> -header-y += ipcbuf.h
> -header-y += kvm_para.h
> -header-y += mman.h
> -header-y += msgbuf.h
> -header-y += param.h
> -header-y += poll.h
> -header-y += posix_types.h
> -header-y += ptrace.h
> -header-y += resource.h
> -header-y += sembuf.h
> -header-y += setup.h
> -header-y += shmbuf.h
> -header-y += sigcontext.h
> -header-y += siginfo.h
> -header-y += signal.h
> -header-y += socket.h
> -header-y += sockios.h
> -header-y += stat.h
> -header-y += statfs.h
> -header-y += swab.h
> -header-y += termbits.h
> -header-y += termios.h
> -header-y += types.h
> -header-y += unistd.h
> --
> 1.9.3
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH 31/38] score: clean-up uapi Kbuild file

[Lennox Wu]

 Acked-by: Lennox Wu lennox...@gamil.com

2014-07-14 23:08 GMT+08:00 Sam Ravnborg s...@ravnborg.org:
 Remove redundant assignments.

 Signed-off-by: Sam Ravnborg s...@ravnborg.org
 Cc: Chen Liqin liqin.li...@gmail.com
 Cc: Lennox Wu lennox...@gmail.com
 ---
  arch/score/include/uapi/asm/Kbuild | 32 
  1 file changed, 32 deletions(-)

 diff --git a/arch/score/include/uapi/asm/Kbuild 
 b/arch/score/include/uapi/asm/Kbuild
 index 040178c..b15bf6b 100644
 --- a/arch/score/include/uapi/asm/Kbuild
 +++ b/arch/score/include/uapi/asm/Kbuild
 @@ -1,34 +1,2 @@
  # UAPI Header export list
  include include/uapi/asm-generic/Kbuild.asm
 -
 -header-y += auxvec.h
 -header-y += bitsperlong.h
 -header-y += byteorder.h
 -header-y += errno.h
 -header-y += fcntl.h
 -header-y += ioctl.h
 -header-y += ioctls.h
 -header-y += ipcbuf.h
 -header-y += kvm_para.h
 -header-y += mman.h
 -header-y += msgbuf.h
 -header-y += param.h
 -header-y += poll.h
 -header-y += posix_types.h
 -header-y += ptrace.h
 -header-y += resource.h
 -header-y += sembuf.h
 -header-y += setup.h
 -header-y += shmbuf.h
 -header-y += sigcontext.h
 -header-y += siginfo.h
 -header-y += signal.h
 -header-y += socket.h
 -header-y += sockios.h
 -header-y += stat.h
 -header-y += statfs.h
 -header-y += swab.h
 -header-y += termbits.h
 -header-y += termios.h
 -header-y += types.h
 -header-y += unistd.h
 --
 1.9.3

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

[Lennox Wu]

Score can provide dummy functions if HAS_IOMEM  and NO_IOMEM will be
removed, even if we indeed have no IOMEM.

Best,
Lennox

2014-07-18 18:51 GMT+08:00 Richard Weinberger rich...@nod.at:
 Am 18.07.2014 12:44, schrieb Chen Gang:
 On 07/18/2014 03:35 PM, Richard Weinberger wrote:
 Am 18.07.2014 02:36, schrieb Chen Gang:

 On 07/18/2014 02:09 AM, Richard Weinberger wrote:
 Am 17.07.2014 12:48, schrieb Arnd Bergmann:
 AFAICT, NO_IOMEM only has a real purpose on UML these days. Could we take
 a shortcut here and make COMPILE_TEST depend on !UML? Getting random 
 stuff
 to build on UML seems pointless to me and we special-case it in a number 
 of
 places already.

 If UML is the only arch without io memory the dependency on !UML seems
 reasonable to me. :-)


 For me, if only uml left, I suggest to implement dummy functions within
 uml instead of let CONFIG_UML appear in generic include directory. And
 then remove all HAS_IOMEM and NO_IOMEM from kernel.

 Erm, this is something completely different.
 I thought we're focusing on COMPILE_TEST?


 COMPILE_TEST is none-architecture specific, but UML is. So in generic
 include folder, if we're focusing on choosing whether COMPILE_TEST or
 UML, for me, I will choose COMPILE_TEST.

 If we're not only focusing on COMPILE_TEST, for me, if something only
 depend on one architecture, I'd like to put them under arch/*/ folder.

 Especially, after that, we can remove all HAS_IOMEM and NO_IOMEM, nobody
 has to think of them again. :-)

 And then we end up with a solution that on UML a lot of completely useless
 drivers are build which fail in various interesting manners because you'll
 add stubs for all kinds of io memory related functions to arch/um/?
 We had this kind of discussion already. You'll need more than ioremap...

 I like Arnd's idea *much* more to make COMPILE_TEST depend on !UML.

 Thanks,
 //richard
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH v3] arch/score/include/uapi/asm/ptrace.h: Remove the macros which not be used currently

[Lennox Wu]

Yes, you might miss something. Some patches were merged once I
acknowledged them,and some were missed.
So I am not sure the rule, if someone will merge these patches once
them are acknowledged, it is redundant that I issue a
PULL request again.



2014-07-17 10:30 GMT+08:00 Guenter Roeck :
> On 07/16/2014 07:14 PM, Lennox Wu wrote:
>>
>> Actually, I don't know the rule for merging patches. Some patches will
>> be merge,and some will be miss. If the patch is missed, we will merge
>> it into our git.
>>
> Unless I am missing something, this patch is in arch/score, so the
> expectation would be for you to pick it up and send it to Linus.
>
> Guenter
>
>
>> Best,
>> Lennox
>>
>> 2014-07-17 9:49 GMT+08:00 David Rientjes :
>>>
>>> On Thu, 26 Jun 2014, David Rientjes wrote:
>>>
>>>> On Thu, 26 Jun 2014, Chen Gang wrote:
>>>>
>>>>> 'COUNTER' and other same kind macros are too common to use, and easy to
>>>>> get conflict with other modules.
>>>>>
>>>>> At present, they are not used, so it is OK to simply remove them. And
>>>>> the
>>>>> related warning (allmodconfig with score):
>>>>>
>>>>>  CC [M]  drivers/md/raid1.o
>>>>>In file included from drivers/md/raid1.c:42:0:
>>>>>drivers/md/bitmap.h:93:0: warning: "COUNTER" redefined
>>>>> #define COUNTER(x) (((bitmap_counter_t) x) & COUNTER_MAX)
>>>>> ^
>>>>>In file included from ./arch/score/include/asm/ptrace.h:4:0,
>>>>> from include/linux/sched.h:31,
>>>>> from include/linux/blkdev.h:4,
>>>>> from drivers/md/raid1.c:36:
>>>>>./arch/score/include/uapi/asm/ptrace.h:13:0: note: this is the
>>>>> location of the previous definition
>>>>> #define COUNTER  38
>>>>>
>>>>>
>>>>> Signed-off-by: Chen Gang 
>>>>
>>>>
>>>> Acked-by: David Rientjes 
>>>>
>>>
>>> Lennox or Liqin, could you please queue this patch for 3.17?
>>
>>
>>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] arch: score: Export necessary symbols in related files

[Lennox Wu]

 Acked-by: Lennox Wu

2014-07-09 16:48 GMT+08:00 Chen Gang :
> 'csum_partial_copy_from_user' and 'flush_dcache_page' are also needed by
> outside modules, so need export them in the related files.
>
> The related error (with allmodconfig under score):
>
> MODPOST 1365 modules
>   ERROR: "csum_partial_copy_from_user" [net/rxrpc/af-rxrpc.ko] undefined!
>   ERROR: "flush_dcache_page" [net/sunrpc/sunrpc.ko] undefined!
>
>
> Signed-off-by: Chen Gang 
> ---
>  arch/score/lib/checksum_copy.c | 1 +
>  arch/score/mm/cache.c  | 1 +
>  2 files changed, 2 insertions(+)
>
> diff --git a/arch/score/lib/checksum_copy.c b/arch/score/lib/checksum_copy.c
> index 04565dd..9b770b3 100644
> --- a/arch/score/lib/checksum_copy.c
> +++ b/arch/score/lib/checksum_copy.c
> @@ -50,3 +50,4 @@ unsigned int csum_partial_copy_from_user(const char *src, 
> char *dst,
>
> return csum_partial(dst, len, sum);
>  }
> +EXPORT_SYMBOL(csum_partial_copy_from_user);
> diff --git a/arch/score/mm/cache.c b/arch/score/mm/cache.c
> index f85ec1a..be91041 100644
> --- a/arch/score/mm/cache.c
> +++ b/arch/score/mm/cache.c
> @@ -72,6 +72,7 @@ void flush_dcache_page(struct page *page)
> addr = (unsigned long) page_address(page);
> flush_data_cache_page(addr);
>  }
> +EXPORT_SYMBOL(flush_dcache_page);
>
>  /* called by update_mmu_cache. */
>  void __update_cache(struct vm_area_struct *vma, unsigned long address,
> --
> 1.9.2.459.g68773ac
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH v3] arch/score/include/uapi/asm/ptrace.h: Remove the macros which not be used currently

[Lennox Wu]

Actually, I don't know the rule for merging patches. Some patches will
be merge,and some will be miss. If the patch is missed, we will merge
it into our git.

Best,
Lennox

2014-07-17 9:49 GMT+08:00 David Rientjes :
> On Thu, 26 Jun 2014, David Rientjes wrote:
>
>> On Thu, 26 Jun 2014, Chen Gang wrote:
>>
>> > 'COUNTER' and other same kind macros are too common to use, and easy to
>> > get conflict with other modules.
>> >
>> > At present, they are not used, so it is OK to simply remove them. And the
>> > related warning (allmodconfig with score):
>> >
>> > CC [M]  drivers/md/raid1.o
>> >   In file included from drivers/md/raid1.c:42:0:
>> >   drivers/md/bitmap.h:93:0: warning: "COUNTER" redefined
>> >#define COUNTER(x) (((bitmap_counter_t) x) & COUNTER_MAX)
>> >^
>> >   In file included from ./arch/score/include/asm/ptrace.h:4:0,
>> >from include/linux/sched.h:31,
>> >from include/linux/blkdev.h:4,
>> >from drivers/md/raid1.c:36:
>> >   ./arch/score/include/uapi/asm/ptrace.h:13:0: note: this is the location 
>> > of the previous definition
>> >#define COUNTER  38
>> >
>> >
>> > Signed-off-by: Chen Gang 
>>
>> Acked-by: David Rientjes 
>>
>
> Lennox or Liqin, could you please queue this patch for 3.17?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH v3] arch/score/include/uapi/asm/ptrace.h: Remove the macros which not be used currently

[Lennox Wu]

Actually, I don't know the rule for merging patches. Some patches will
be merge,and some will be miss. If the patch is missed, we will merge
it into our git.

Best,
Lennox

2014-07-17 9:49 GMT+08:00 David Rientjes rient...@google.com:
 On Thu, 26 Jun 2014, David Rientjes wrote:

 On Thu, 26 Jun 2014, Chen Gang wrote:

  'COUNTER' and other same kind macros are too common to use, and easy to
  get conflict with other modules.
 
  At present, they are not used, so it is OK to simply remove them. And the
  related warning (allmodconfig with score):
 
  CC [M]  drivers/md/raid1.o
In file included from drivers/md/raid1.c:42:0:
drivers/md/bitmap.h:93:0: warning: COUNTER redefined
 #define COUNTER(x) (((bitmap_counter_t) x)  COUNTER_MAX)
 ^
In file included from ./arch/score/include/asm/ptrace.h:4:0,
 from include/linux/sched.h:31,
 from include/linux/blkdev.h:4,
 from drivers/md/raid1.c:36:
./arch/score/include/uapi/asm/ptrace.h:13:0: note: this is the location 
  of the previous definition
 #define COUNTER  38
 
 
  Signed-off-by: Chen Gang gang.chen.5...@gmail.com

 Acked-by: David Rientjes rient...@google.com


 Lennox or Liqin, could you please queue this patch for 3.17?
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] arch: score: Export necessary symbols in related files

[Lennox Wu]

 Acked-by: Lennox Wulennox...@gmail.com

2014-07-09 16:48 GMT+08:00 Chen Gang gang.chen.5...@gmail.com:
 'csum_partial_copy_from_user' and 'flush_dcache_page' are also needed by
 outside modules, so need export them in the related files.

 The related error (with allmodconfig under score):

 MODPOST 1365 modules
   ERROR: csum_partial_copy_from_user [net/rxrpc/af-rxrpc.ko] undefined!
   ERROR: flush_dcache_page [net/sunrpc/sunrpc.ko] undefined!


 Signed-off-by: Chen Gang gang.chen.5...@gmail.com
 ---
  arch/score/lib/checksum_copy.c | 1 +
  arch/score/mm/cache.c  | 1 +
  2 files changed, 2 insertions(+)

 diff --git a/arch/score/lib/checksum_copy.c b/arch/score/lib/checksum_copy.c
 index 04565dd..9b770b3 100644
 --- a/arch/score/lib/checksum_copy.c
 +++ b/arch/score/lib/checksum_copy.c
 @@ -50,3 +50,4 @@ unsigned int csum_partial_copy_from_user(const char *src, 
 char *dst,

 return csum_partial(dst, len, sum);
  }
 +EXPORT_SYMBOL(csum_partial_copy_from_user);
 diff --git a/arch/score/mm/cache.c b/arch/score/mm/cache.c
 index f85ec1a..be91041 100644
 --- a/arch/score/mm/cache.c
 +++ b/arch/score/mm/cache.c
 @@ -72,6 +72,7 @@ void flush_dcache_page(struct page *page)
 addr = (unsigned long) page_address(page);
 flush_data_cache_page(addr);
  }
 +EXPORT_SYMBOL(flush_dcache_page);

  /* called by update_mmu_cache. */
  void __update_cache(struct vm_area_struct *vma, unsigned long address,
 --
 1.9.2.459.g68773ac
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH v3] arch/score/include/uapi/asm/ptrace.h: Remove the macros which not be used currently

[Lennox Wu]

Yes, you might miss something. Some patches were merged once I
acknowledged them,and some were missed.
So I am not sure the rule, if someone will merge these patches once
them are acknowledged, it is redundant that I issue a
PULL request again.



2014-07-17 10:30 GMT+08:00 Guenter Roeck li...@roeck-us.net:
 On 07/16/2014 07:14 PM, Lennox Wu wrote:

 Actually, I don't know the rule for merging patches. Some patches will
 be merge,and some will be miss. If the patch is missed, we will merge
 it into our git.

 Unless I am missing something, this patch is in arch/score, so the
 expectation would be for you to pick it up and send it to Linus.

 Guenter


 Best,
 Lennox

 2014-07-17 9:49 GMT+08:00 David Rientjes rient...@google.com:

 On Thu, 26 Jun 2014, David Rientjes wrote:

 On Thu, 26 Jun 2014, Chen Gang wrote:

 'COUNTER' and other same kind macros are too common to use, and easy to
 get conflict with other modules.

 At present, they are not used, so it is OK to simply remove them. And
 the
 related warning (allmodconfig with score):

  CC [M]  drivers/md/raid1.o
In file included from drivers/md/raid1.c:42:0:
drivers/md/bitmap.h:93:0: warning: COUNTER redefined
 #define COUNTER(x) (((bitmap_counter_t) x)  COUNTER_MAX)
 ^
In file included from ./arch/score/include/asm/ptrace.h:4:0,
 from include/linux/sched.h:31,
 from include/linux/blkdev.h:4,
 from drivers/md/raid1.c:36:
./arch/score/include/uapi/asm/ptrace.h:13:0: note: this is the
 location of the previous definition
 #define COUNTER  38


 Signed-off-by: Chen Gang gang.chen.5...@gmail.com


 Acked-by: David Rientjes rient...@google.com


 Lennox or Liqin, could you please queue this patch for 3.17?




--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH 04/38] score: drop irrelevant assignment from Kbuild file

[Lennox Wu]

Thanks :)

Acked-by: Lennox Wu


2014-07-14 23:08 GMT+08:00 Sam Ravnborg :
> header-y assignemnt in this file has no effect - drop it.
>
> Signed-off-by: Sam Ravnborg 
> Cc: Chen Liqin 
> Cc: Lennox Wu 
> Cc: David Howells 
> ---
>  arch/score/include/asm/Kbuild | 3 ---
>  1 file changed, 3 deletions(-)
>
> diff --git a/arch/score/include/asm/Kbuild b/arch/score/include/asm/Kbuild
> index 2f947ab..f67b3f6 100644
> --- a/arch/score/include/asm/Kbuild
> +++ b/arch/score/include/asm/Kbuild
> @@ -1,7 +1,4 @@
>
> -header-y +=
> -
> -
>  generic-y += barrier.h
>  generic-y += clkdev.h
>  generic-y += cputime.h
> --
> 1.9.3
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH 04/38] score: drop irrelevant assignment from Kbuild file

[Lennox Wu]

Thanks :)

Acked-by: Lennox Wulennox...@gmail.com


2014-07-14 23:08 GMT+08:00 Sam Ravnborg s...@ravnborg.org:
 header-y assignemnt in this file has no effect - drop it.

 Signed-off-by: Sam Ravnborg s...@ravnborg.org
 Cc: Chen Liqin liqin.li...@gmail.com
 Cc: Lennox Wu lennox...@gmail.com
 Cc: David Howells dhowe...@redhat.com
 ---
  arch/score/include/asm/Kbuild | 3 ---
  1 file changed, 3 deletions(-)

 diff --git a/arch/score/include/asm/Kbuild b/arch/score/include/asm/Kbuild
 index 2f947ab..f67b3f6 100644
 --- a/arch/score/include/asm/Kbuild
 +++ b/arch/score/include/asm/Kbuild
 @@ -1,7 +1,4 @@

 -header-y +=
 -
 -
  generic-y += barrier.h
  generic-y += clkdev.h
  generic-y += cputime.h
 --
 1.9.3

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

[Lennox Wu]

As I said before, some configurations don't make sense.
I don't think all of the patches are necessary.

Best,
Lennox

2014-07-13 11:45 GMT+08:00 Marek Vasut :
> On Sunday, July 13, 2014 at 05:07:10 AM, Chen Gang wrote:
>> Several drivers need 'devm_ioremap_resource' which need HAS_IOMEM enabled.
>> So let them depend on HAS_IOMEM.
>>
>> The related error (with allmodconfig under score):
>>
>> MODPOST 1365 modules
>>   ERROR: "devm_ioremap_resource" [drivers/watchdog/tegra_wdt.ko] undefined!
>>   ERROR: "devm_ioremap_resource" [drivers/watchdog/of_xilinx_wdt.ko]
>> undefined! ERROR: "devm_ioremap_resource"
>> [drivers/staging/iio/adc/mxs-lradc.ko] undefined! ERROR:
>> "devm_ioremap_resource" [drivers/pwm/pwm-clps711x.ko] undefined! ERROR:
>> "devm_ioremap_resource" [drivers/input/serio/olpc_apsp.ko] undefined!
>> ERROR: "devm_ioremap_resource" [drivers/input/serio/arc_ps2.ko] undefined!
>
> This stuff should go through different trees, so I'd suggest to split this 
> into
> multiple patches. Thanks for catching this stuff !
>
> Best regards,
> Marek Vasut
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] drivers: Let several drivers depends on HAS_IOMEM for 'devm_ioremap_resource'

[Lennox Wu]

As I said before, some configurations don't make sense.
I don't think all of the patches are necessary.

Best,
Lennox

2014-07-13 11:45 GMT+08:00 Marek Vasut ma...@denx.de:
 On Sunday, July 13, 2014 at 05:07:10 AM, Chen Gang wrote:
 Several drivers need 'devm_ioremap_resource' which need HAS_IOMEM enabled.
 So let them depend on HAS_IOMEM.

 The related error (with allmodconfig under score):

 MODPOST 1365 modules
   ERROR: devm_ioremap_resource [drivers/watchdog/tegra_wdt.ko] undefined!
   ERROR: devm_ioremap_resource [drivers/watchdog/of_xilinx_wdt.ko]
 undefined! ERROR: devm_ioremap_resource
 [drivers/staging/iio/adc/mxs-lradc.ko] undefined! ERROR:
 devm_ioremap_resource [drivers/pwm/pwm-clps711x.ko] undefined! ERROR:
 devm_ioremap_resource [drivers/input/serio/olpc_apsp.ko] undefined!
 ERROR: devm_ioremap_resource [drivers/input/serio/arc_ps2.ko] undefined!

 This stuff should go through different trees, so I'd suggest to split this 
 into
 multiple patches. Thanks for catching this stuff !

 Best regards,
 Marek Vasut
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH v3] arch/score/include/uapi/asm/ptrace.h: Remove the macros which not be used currently

[Lennox Wu]

Acked-by: Lennox Wu 

2014-06-27 7:28 GMT+08:00 Chen Gang :
>
>
> On 06/27/2014 05:56 AM, David Rientjes wrote:
>> On Thu, 26 Jun 2014, Chen Gang wrote:
>>
>>> 'COUNTER' and other same kind macros are too common to use, and easy to
>>> get conflict with other modules.
>>>
>>> At present, they are not used, so it is OK to simply remove them. And the
>>> related warning (allmodconfig with score):
>>>
>>> CC [M]  drivers/md/raid1.o
>>>   In file included from drivers/md/raid1.c:42:0:
>>>   drivers/md/bitmap.h:93:0: warning: "COUNTER" redefined
>>>#define COUNTER(x) (((bitmap_counter_t) x) & COUNTER_MAX)
>>>^
>>>   In file included from ./arch/score/include/asm/ptrace.h:4:0,
>>>from include/linux/sched.h:31,
>>>from include/linux/blkdev.h:4,
>>>from drivers/md/raid1.c:36:
>>>   ./arch/score/include/uapi/asm/ptrace.h:13:0: note: this is the location 
>>> of the previous definition
>>>#define COUNTER  38
>>>
>>>
>>> Signed-off-by: Chen Gang 
>>
>> Acked-by: David Rientjes 
>>
>
> OK, thanks. And also thank all of you for your discussions.
>
> Thanks.
> --
> Chen Gang
>
> Open, share, and attitude like air, water, and life which God blessed
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH v3] arch/score/include/uapi/asm/ptrace.h: Remove the macros which not be used currently

[Lennox Wu]

Acked-by: Lennox Wu lennox...@gmail.com

2014-06-27 7:28 GMT+08:00 Chen Gang gang.chen.5...@gmail.com:


 On 06/27/2014 05:56 AM, David Rientjes wrote:
 On Thu, 26 Jun 2014, Chen Gang wrote:

 'COUNTER' and other same kind macros are too common to use, and easy to
 get conflict with other modules.

 At present, they are not used, so it is OK to simply remove them. And the
 related warning (allmodconfig with score):

 CC [M]  drivers/md/raid1.o
   In file included from drivers/md/raid1.c:42:0:
   drivers/md/bitmap.h:93:0: warning: COUNTER redefined
#define COUNTER(x) (((bitmap_counter_t) x)  COUNTER_MAX)
^
   In file included from ./arch/score/include/asm/ptrace.h:4:0,
from include/linux/sched.h:31,
from include/linux/blkdev.h:4,
from drivers/md/raid1.c:36:
   ./arch/score/include/uapi/asm/ptrace.h:13:0: note: this is the location 
 of the previous definition
#define COUNTER  38


 Signed-off-by: Chen Gang gang.chen.5...@gmail.com

 Acked-by: David Rientjes rient...@google.com


 OK, thanks. And also thank all of you for your discussions.

 Thanks.
 --
 Chen Gang

 Open, share, and attitude like air, water, and life which God blessed
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] arch/score/include/asm/uaccess.h: Use standard clear_user() prototype

[Lennox Wu]

Thanks for the patch.

Acked-by: Lennox Wu

2014-06-22 20:29 GMT+08:00 Chen Gang :
> Use standard clear_user() just like another architectures, also match
> __clear_user() under score.
>
> The related warning (allmodconfig under score):
>
> CC [M]  drivers/md/dm-ioctl.o
>   drivers/md/dm-ioctl.c: In function 'copy_params':
>   drivers/md/dm-ioctl.c:1728:33: warning: passing argument 1 of 'clear_user' 
> from incompatible pointer type
>  if (secure_data && clear_user(user, param_kernel->data_size))
>^
>   In file included from include/linux/uaccess.h:5:0,
>from include/linux/highmem.h:8,
>from include/linux/bio.h:23,
>from include/linux/device-mapper.h:11,
>from drivers/md/dm.h:14,
>from drivers/md/dm-ioctl.c:8:
>   ./arch/score/include/asm/uaccess.h:364:29: note: expected 'char *' but 
> argument is of type 'struct dm_ioctl *'
>static inline unsigned long clear_user(char *src, unsigned long size)
>^
>
> Signed-off-by: Chen Gang 
> ---
>  arch/score/include/asm/uaccess.h | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/arch/score/include/asm/uaccess.h 
> b/arch/score/include/asm/uaccess.h
> index ab66ddd..259a539 100644
> --- a/arch/score/include/asm/uaccess.h
> +++ b/arch/score/include/asm/uaccess.h
> @@ -361,7 +361,8 @@ copy_in_user(void *to, const void *from, unsigned long 
> len)
>   */
>  extern unsigned long __clear_user(void __user *src, unsigned long size);
>
> -static inline unsigned long clear_user(char *src, unsigned long size)
> +static inline unsigned long __must_check clear_user(void __user *src,
> +   unsigned long size)
>  {
> if (access_ok(VERIFY_WRITE, src, size))
> return __clear_user(src, size);
> --
> 1.9.2.459.g68773ac
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH v2] rch/score/include/uapi/asm/ptrace.h: Add prefix 'SCORE_' for related macros

[Lennox Wu]

These marcos are listed in advance.
Indeed, they have not been used, and the functions use them have not
be released.
I would like to keep the status even if the configure should not be used.
However, if guys think it is dirty, maybe we can remove or comment them first.

Best,
Lennox


2014-06-25 9:25 GMT+08:00 Chen Gang :
>
>
> On 06/25/2014 07:40 AM, David Rientjes wrote:
>> On Tue, 24 Jun 2014, Guenter Roeck wrote:
>>
>>> That is another question. But for the time being it might be sufficient to
>>> surround the defines with #if !defined(__KERNEL__) without introducing new
>>> (and unused) kernel defines.
>>>
>>
>> On the other hand, if no userspace is referencing these, then you could
>> just rename them and move them out of the uapi header to
>> include/asm/ptrace.h and be done with it.
>>
>
> OK, thanks. And sorry for the subject lost 'a', firstly.  And it is
> related with UAPI,  so it is really important enough to let related
> maintainer to notice about it.
>
> I guess, only the related maintainer know whether can remove these
> macros out of "include/uapi/asm/ptrace.h" (although in current upstream
> kernel source, they are really not used).
>
> It is abnormal in uapi headers for same values with different macros
> switched by "__KERNEL__", but if we have to remain them still in uapi,
> we have to use "__KERNEL__ && __linux__" to carm down kernel and user.
>
>
> Thanks.
> --
> Chen Gang
>
> Open, share, and attitude like air, water, and life which God blessed
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH v2] rch/score/include/uapi/asm/ptrace.h: Add prefix 'SCORE_' for related macros

[Lennox Wu]

These marcos are listed in advance.
Indeed, they have not been used, and the functions use them have not
be released.
I would like to keep the status even if the configure should not be used.
However, if guys think it is dirty, maybe we can remove or comment them first.

Best,
Lennox


2014-06-25 9:25 GMT+08:00 Chen Gang gang.chen.5...@gmail.com:


 On 06/25/2014 07:40 AM, David Rientjes wrote:
 On Tue, 24 Jun 2014, Guenter Roeck wrote:

 That is another question. But for the time being it might be sufficient to
 surround the defines with #if !defined(__KERNEL__) without introducing new
 (and unused) kernel defines.


 On the other hand, if no userspace is referencing these, then you could
 just rename them and move them out of the uapi header to
 include/asm/ptrace.h and be done with it.


 OK, thanks. And sorry for the subject lost 'a', firstly.  And it is
 related with UAPI,  so it is really important enough to let related
 maintainer to notice about it.

 I guess, only the related maintainer know whether can remove these
 macros out of include/uapi/asm/ptrace.h (although in current upstream
 kernel source, they are really not used).

 It is abnormal in uapi headers for same values with different macros
 switched by __KERNEL__, but if we have to remain them still in uapi,
 we have to use __KERNEL__  __linux__ to carm down kernel and user.


 Thanks.
 --
 Chen Gang

 Open, share, and attitude like air, water, and life which God blessed
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] arch/score/include/asm/uaccess.h: Use standard clear_user() prototype

[Lennox Wu]

Thanks for the patch.

Acked-by: Lennox Wulennox...@gmail.com

2014-06-22 20:29 GMT+08:00 Chen Gang gang.chen.5...@gmail.com:
 Use standard clear_user() just like another architectures, also match
 __clear_user() under score.

 The related warning (allmodconfig under score):

 CC [M]  drivers/md/dm-ioctl.o
   drivers/md/dm-ioctl.c: In function 'copy_params':
   drivers/md/dm-ioctl.c:1728:33: warning: passing argument 1 of 'clear_user' 
 from incompatible pointer type
  if (secure_data  clear_user(user, param_kernel-data_size))
^
   In file included from include/linux/uaccess.h:5:0,
from include/linux/highmem.h:8,
from include/linux/bio.h:23,
from include/linux/device-mapper.h:11,
from drivers/md/dm.h:14,
from drivers/md/dm-ioctl.c:8:
   ./arch/score/include/asm/uaccess.h:364:29: note: expected 'char *' but 
 argument is of type 'struct dm_ioctl *'
static inline unsigned long clear_user(char *src, unsigned long size)
^

 Signed-off-by: Chen Gang gang.chen.5...@gmail.com
 ---
  arch/score/include/asm/uaccess.h | 3 ++-
  1 file changed, 2 insertions(+), 1 deletion(-)

 diff --git a/arch/score/include/asm/uaccess.h 
 b/arch/score/include/asm/uaccess.h
 index ab66ddd..259a539 100644
 --- a/arch/score/include/asm/uaccess.h
 +++ b/arch/score/include/asm/uaccess.h
 @@ -361,7 +361,8 @@ copy_in_user(void *to, const void *from, unsigned long 
 len)
   */
  extern unsigned long __clear_user(void __user *src, unsigned long size);

 -static inline unsigned long clear_user(char *src, unsigned long size)
 +static inline unsigned long __must_check clear_user(void __user *src,
 +   unsigned long size)
  {
 if (access_ok(VERIFY_WRITE, src, size))
 return __clear_user(src, size);
 --
 1.9.2.459.g68773ac
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] arch/score/include/uapi/asm/ptrace.h: Add prefix 'SCORE_' for related macros

[Lennox Wu]

Sorry for the typo
.
To discover combinations which doesn't make sense might waste Chen's time.


2014-06-23 11:07 GMT+08:00 Chen Gang :
> On 06/23/2014 12:28 AM, Guenter Roeck wrote:
>> On 06/22/2014 01:09 AM, Lennox Wu wrote:
>>> Hi Chen,
>>> Do you find other modules except device modules cause issues of
>>> compiling?
>>> The applications of Score are limited. Acutally, some errors never occur
>>> in the limited applications, for example, the conbination consists of
>>> Score and the timer of SuperH. I'm not sure we should handle this kind
>>> of error.
>>> If the rule is that we should make sure "all" modules can be compiled
>>> with Score, we will do it.
>>> However, we still thanks for your check.
>>>
>>
>> Other architectures don't care about buildability of allmodconfig or
>> allyesconfig
>> (arm is a good [or maybe bad] example), so it is for sure not mandatory.
>> On the other side, fixing such problems - especially when you have someone
>> doing it for you - may prevent problems down the road, so while not
>> mandatory
>> I consider it a useful goal to achieve.
>>
>> If there is a driver which doesn't make sense to build at all, such as
>> the SuperH
>> timer driver mentioned above, it can always be excluded from building
>> with Kconfig.
>> If that causes opposition for some reason, you can still ignore that it
>> fails
>> to build.
>>
>
> Yeah, what Guenter said is fine to me.
>
> Although allmodconfig is not mandatory, it is still a useful goal to
> achieve.
>
>
> Thanks.
> --
> Chen Gang
>
> Open, share, and attitude like air, water, and life which God blessed
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] arch/score/include/uapi/asm/ptrace.h: Add prefix 'SCORE_' for related macros

[Lennox Wu]

Yes, it's the reason we still thank Chen's checking.
Arbitrary configurations except the default configure are not
guaranteed to pass compiling.
To discover combinations which doesn't make sense might weast Chen's time.

Best,
Lennox

2014-06-23 0:28 GMT+08:00 Guenter Roeck :
> On 06/22/2014 01:09 AM, Lennox Wu wrote:
>>
>> Hi Chen,
>> Do you find other modules except device modules cause issues of compiling?
>> The applications of Score are limited. Acutally, some errors never occur
>> in the limited applications, for example, the conbination consists of
>> Score and the timer of SuperH. I'm not sure we should handle this kind
>> of error.
>> If the rule is that we should make sure "all" modules can be compiled
>> with Score, we will do it.
>> However, we still thanks for your check.
>>
>
> Other architectures don't care about buildability of allmodconfig or
> allyesconfig
> (arm is a good [or maybe bad] example), so it is for sure not mandatory.
> On the other side, fixing such problems - especially when you have someone
> doing it for you - may prevent problems down the road, so while not
> mandatory
> I consider it a useful goal to achieve.
>
> If there is a driver which doesn't make sense to build at all, such as the
> SuperH
> timer driver mentioned above, it can always be excluded from building with
> Kconfig.
> If that causes opposition for some reason, you can still ignore that it
> fails
> to build.
>
> Guenter
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] arch/score/include/uapi/asm/ptrace.h: Add prefix 'SCORE_' for related macros

[Lennox Wu]

Hi Chen,
Do you find other modules except device modules cause issues of compiling?
The applications of Score are limited. Acutally, some errors never occur
in the limited applications, for example, the conbination consists of
Score and the timer of SuperH. I'm not sure we should handle this kind
of error.
If the rule is that we should make sure "all" modules can be compiled
with Score, we will do it.
However, we still thanks for your check.

Best,
Lennox

2014-06-22 12:48 GMT+08:00 Chen Gang :
> 'COUNTER' and other same kind macros are too common to use, and easy to
> get conflict with other modules. So add prefix for them.
>
> The related warning (allmodconfig with score):
>
> CC [M]  drivers/md/raid1.o
>   In file included from drivers/md/raid1.c:42:0:
>   drivers/md/bitmap.h:93:0: warning: "COUNTER" redefined
>#define COUNTER(x) (((bitmap_counter_t) x) & COUNTER_MAX)
>^
>   In file included from ./arch/score/include/asm/ptrace.h:4:0,
>from include/linux/sched.h:31,
>from include/linux/blkdev.h:4,
>from drivers/md/raid1.c:36:
>   ./arch/score/include/uapi/asm/ptrace.h:13:0: note: this is the location of 
> the previous definition
>#define COUNTER  38
>
>
> Signed-off-by: Chen Gang 
> ---
>  arch/score/include/uapi/asm/ptrace.h | 20 ++--
>  1 file changed, 10 insertions(+), 10 deletions(-)
>
> diff --git a/arch/score/include/uapi/asm/ptrace.h 
> b/arch/score/include/uapi/asm/ptrace.h
> index f59771a..7a0ead9 100644
> --- a/arch/score/include/uapi/asm/ptrace.h
> +++ b/arch/score/include/uapi/asm/ptrace.h
> @@ -4,16 +4,16 @@
>  #define PTRACE_GETREGS 12
>  #define PTRACE_SETREGS 13
>
> -#define PC 32
> -#define CONDITION  33
> -#define ECR34
> -#define EMA35
> -#define CEH36
> -#define CEL37
> -#define COUNTER38
> -#define LDCR   39
> -#define STCR   40
> -#define PSR41
> +#define SCORE_PC   32
> +#define SCORE_CONDITION33
> +#define SCORE_ECR  34
> +#define SCORE_EMA  35
> +#define SCORE_CEH  36
> +#define SOCRE_CEL  37
> +#define SCORE_COUNTER  38
> +#define SCORE_LDCR 39
> +#define SCORE_STCR 40
> +#define SCORE_PSR  41
>
>  #define SINGLESTEP16_INSN  0x7006
>  #define SINGLESTEP32_INSN  0x840C8000
> --
> 1.9.2.459.g68773ac
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] arch/score/include/uapi/asm/ptrace.h: Add prefix 'SCORE_' for related macros

[Lennox Wu]

Hi Chen,
Do you find other modules except device modules cause issues of compiling?
The applications of Score are limited. Acutally, some errors never occur
in the limited applications, for example, the conbination consists of
Score and the timer of SuperH. I'm not sure we should handle this kind
of error.
If the rule is that we should make sure all modules can be compiled
with Score, we will do it.
However, we still thanks for your check.

Best,
Lennox

2014-06-22 12:48 GMT+08:00 Chen Gang gang.chen.5...@gmail.com:
 'COUNTER' and other same kind macros are too common to use, and easy to
 get conflict with other modules. So add prefix for them.

 The related warning (allmodconfig with score):

 CC [M]  drivers/md/raid1.o
   In file included from drivers/md/raid1.c:42:0:
   drivers/md/bitmap.h:93:0: warning: COUNTER redefined
#define COUNTER(x) (((bitmap_counter_t) x)  COUNTER_MAX)
^
   In file included from ./arch/score/include/asm/ptrace.h:4:0,
from include/linux/sched.h:31,
from include/linux/blkdev.h:4,
from drivers/md/raid1.c:36:
   ./arch/score/include/uapi/asm/ptrace.h:13:0: note: this is the location of 
 the previous definition
#define COUNTER  38


 Signed-off-by: Chen Gang gang.chen.5...@gmail.com
 ---
  arch/score/include/uapi/asm/ptrace.h | 20 ++--
  1 file changed, 10 insertions(+), 10 deletions(-)

 diff --git a/arch/score/include/uapi/asm/ptrace.h 
 b/arch/score/include/uapi/asm/ptrace.h
 index f59771a..7a0ead9 100644
 --- a/arch/score/include/uapi/asm/ptrace.h
 +++ b/arch/score/include/uapi/asm/ptrace.h
 @@ -4,16 +4,16 @@
  #define PTRACE_GETREGS 12
  #define PTRACE_SETREGS 13

 -#define PC 32
 -#define CONDITION  33
 -#define ECR34
 -#define EMA35
 -#define CEH36
 -#define CEL37
 -#define COUNTER38
 -#define LDCR   39
 -#define STCR   40
 -#define PSR41
 +#define SCORE_PC   32
 +#define SCORE_CONDITION33
 +#define SCORE_ECR  34
 +#define SCORE_EMA  35
 +#define SCORE_CEH  36
 +#define SOCRE_CEL  37
 +#define SCORE_COUNTER  38
 +#define SCORE_LDCR 39
 +#define SCORE_STCR 40
 +#define SCORE_PSR  41

  #define SINGLESTEP16_INSN  0x7006
  #define SINGLESTEP32_INSN  0x840C8000
 --
 1.9.2.459.g68773ac
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] arch/score/include/uapi/asm/ptrace.h: Add prefix 'SCORE_' for related macros

[Lennox Wu]

Yes, it's the reason we still thank Chen's checking.
Arbitrary configurations except the default configure are not
guaranteed to pass compiling.
To discover combinations which doesn't make sense might weast Chen's time.

Best,
Lennox

2014-06-23 0:28 GMT+08:00 Guenter Roeck li...@roeck-us.net:
 On 06/22/2014 01:09 AM, Lennox Wu wrote:

 Hi Chen,
 Do you find other modules except device modules cause issues of compiling?
 The applications of Score are limited. Acutally, some errors never occur
 in the limited applications, for example, the conbination consists of
 Score and the timer of SuperH. I'm not sure we should handle this kind
 of error.
 If the rule is that we should make sure all modules can be compiled
 with Score, we will do it.
 However, we still thanks for your check.


 Other architectures don't care about buildability of allmodconfig or
 allyesconfig
 (arm is a good [or maybe bad] example), so it is for sure not mandatory.
 On the other side, fixing such problems - especially when you have someone
 doing it for you - may prevent problems down the road, so while not
 mandatory
 I consider it a useful goal to achieve.

 If there is a driver which doesn't make sense to build at all, such as the
 SuperH
 timer driver mentioned above, it can always be excluded from building with
 Kconfig.
 If that causes opposition for some reason, you can still ignore that it
 fails
 to build.

 Guenter

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] arch/score/include/uapi/asm/ptrace.h: Add prefix 'SCORE_' for related macros

[Lennox Wu]

Sorry for the typo
.
To discover combinations which doesn't make sense might waste Chen's time.


2014-06-23 11:07 GMT+08:00 Chen Gang gang.chen.5...@gmail.com:
 On 06/23/2014 12:28 AM, Guenter Roeck wrote:
 On 06/22/2014 01:09 AM, Lennox Wu wrote:
 Hi Chen,
 Do you find other modules except device modules cause issues of
 compiling?
 The applications of Score are limited. Acutally, some errors never occur
 in the limited applications, for example, the conbination consists of
 Score and the timer of SuperH. I'm not sure we should handle this kind
 of error.
 If the rule is that we should make sure all modules can be compiled
 with Score, we will do it.
 However, we still thanks for your check.


 Other architectures don't care about buildability of allmodconfig or
 allyesconfig
 (arm is a good [or maybe bad] example), so it is for sure not mandatory.
 On the other side, fixing such problems - especially when you have someone
 doing it for you - may prevent problems down the road, so while not
 mandatory
 I consider it a useful goal to achieve.

 If there is a driver which doesn't make sense to build at all, such as
 the SuperH
 timer driver mentioned above, it can always be excluded from building
 with Kconfig.
 If that causes opposition for some reason, you can still ignore that it
 fails
 to build.


 Yeah, what Guenter said is fine to me.

 Although allmodconfig is not mandatory, it is still a useful goal to
 achieve.


 Thanks.
 --
 Chen Gang

 Open, share, and attitude like air, water, and life which God blessed
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] arch/score/include/asm/io.h: Add generic io map functions when MMU enabled

[Lennox Wu]

Hi Chen,
Although "SuperH Timer Support" are never used in our platforms, we
can still consider if these functions are necessary.
Could you send the config to us?

Best,
Lennox

2014-06-18 13:06 GMT+08:00 Chen Gang :
> score did not implement some io map functions when MMU enabled, so need
> add generic io map functions, just like some other architectures have
> done (e.g. sparc64, s390, and mn10300), or will cause compiling issue.
>
> The related error (allmodconfig under score):
>
> CC  drivers/clocksource/sh_cmt.o
>   drivers/clocksource/sh_cmt.c: In function 'sh_cmt_map_memory':
>   drivers/clocksource/sh_cmt.c:932:2: error: implicit declaration of function 
> 'ioremap_nocache' [-Werror=implicit-function-declaration]
> cmt->mapbase = ioremap_nocache(mem->start, resource_size(mem));
> ^
>   drivers/clocksource/sh_cmt.c:932:15: warning: assignment makes pointer from 
> integer without a cast
> cmt->mapbase = ioremap_nocache(mem->start, resource_size(mem));
>  ^
>   drivers/clocksource/sh_cmt.c: In function 'sh_cmt_map_memory_legacy':
>   drivers/clocksource/sh_cmt.c:953:18: warning: assignment makes pointer from 
> integer without a cast
> cmt->mapbase_ch = ioremap_nocache(res->start, resource_size(res));
> ^
>   drivers/clocksource/sh_cmt.c:963:15: warning: assignment makes pointer from 
> integer without a cast
> cmt->mapbase = ioremap_nocache(res2 ? res2->start :
>  ^
>   drivers/clocksource/sh_cmt.c:968:3: error: implicit declaration of function 
> 'iounmap' [-Werror=implicit-function-declaration]
>  iounmap(cmt->mapbase_ch);
>  ^
>   cc1: some warnings being treated as errors
>   make[2]: *** [drivers/clocksource/sh_cmt.o] Error 1
>   make[1]: *** [drivers/clocksource] Error 2
>   make: *** [drivers] Error 2
>
> Signed-off-by: Chen Gang 
> ---
>  arch/score/include/asm/io.h | 16 
>  1 file changed, 16 insertions(+)
>
> diff --git a/arch/score/include/asm/io.h b/arch/score/include/asm/io.h
> index 574c8827..d67aeea 100644
> --- a/arch/score/include/asm/io.h
> +++ b/arch/score/include/asm/io.h
> @@ -3,6 +3,22 @@
>
>  #include 
>
> +#ifdef CONFIG_MMU
> +static inline void __iomem *ioremap(phys_addr_t offset, unsigned long size)
> +{
> +   return (void __iomem *) (unsigned long)offset;
> +}
> +
> +#define __ioremap(offset, size, flags) ioremap(offset, size)
> +
> +#define ioremap_nocache ioremap
> +#define ioremap_wc ioremap_nocache
> +
> +static inline void iounmap(void __iomem *addr)
> +{
> +}
> +#endif
> +
>  #define virt_to_busvirt_to_phys
>  #define bus_to_virtphys_to_virt
>  #endif /* _ASM_SCORE_IO_H */
> --
> 1.9.2.459.g68773ac
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH 16/28] Remove GENERIC_HAS_IOMAP

[Lennox Wu]

NO, there is no issue. We just want to push patches in a batched way.
The patch will show recently. Thank you.

Best,
Lennox

2014-06-18 17:10 GMT+08:00 Paul Bolle :
> Lennox,
>
> On Wed, 2014-04-16 at 20:30 +0200, Paul Bolle wrote:
>> On Thu, 2014-04-17 at 01:29 +0800, Lennox Wu wrote:
>> > OK, we will merge it into our tree.
>
> The last few messages in this thread were sent shortly after v3.15-rc1.
> We're now at v3.16-rc1 and those three references to GENERIC_HAS_IOMAP
> are still there.
>
> Are there any issues with this patch?
>
>
> Paul Bolle
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH 16/28] Remove GENERIC_HAS_IOMAP

[Lennox Wu]

NO, there is no issue. We just want to push patches in a batched way.
The patch will show recently. Thank you.

Best,
Lennox

2014-06-18 17:10 GMT+08:00 Paul Bolle pebo...@tiscali.nl:
 Lennox,

 On Wed, 2014-04-16 at 20:30 +0200, Paul Bolle wrote:
 On Thu, 2014-04-17 at 01:29 +0800, Lennox Wu wrote:
  OK, we will merge it into our tree.

 The last few messages in this thread were sent shortly after v3.15-rc1.
 We're now at v3.16-rc1 and those three references to GENERIC_HAS_IOMAP
 are still there.

 Are there any issues with this patch?


 Paul Bolle

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] arch/score/include/asm/io.h: Add generic io map functions when MMU enabled

[Lennox Wu]

Hi Chen,
Although SuperH Timer Support are never used in our platforms, we
can still consider if these functions are necessary.
Could you send the config to us?

Best,
Lennox

2014-06-18 13:06 GMT+08:00 Chen Gang gang.chen.5...@gmail.com:
 score did not implement some io map functions when MMU enabled, so need
 add generic io map functions, just like some other architectures have
 done (e.g. sparc64, s390, and mn10300), or will cause compiling issue.

 The related error (allmodconfig under score):

 CC  drivers/clocksource/sh_cmt.o
   drivers/clocksource/sh_cmt.c: In function 'sh_cmt_map_memory':
   drivers/clocksource/sh_cmt.c:932:2: error: implicit declaration of function 
 'ioremap_nocache' [-Werror=implicit-function-declaration]
 cmt-mapbase = ioremap_nocache(mem-start, resource_size(mem));
 ^
   drivers/clocksource/sh_cmt.c:932:15: warning: assignment makes pointer from 
 integer without a cast
 cmt-mapbase = ioremap_nocache(mem-start, resource_size(mem));
  ^
   drivers/clocksource/sh_cmt.c: In function 'sh_cmt_map_memory_legacy':
   drivers/clocksource/sh_cmt.c:953:18: warning: assignment makes pointer from 
 integer without a cast
 cmt-mapbase_ch = ioremap_nocache(res-start, resource_size(res));
 ^
   drivers/clocksource/sh_cmt.c:963:15: warning: assignment makes pointer from 
 integer without a cast
 cmt-mapbase = ioremap_nocache(res2 ? res2-start :
  ^
   drivers/clocksource/sh_cmt.c:968:3: error: implicit declaration of function 
 'iounmap' [-Werror=implicit-function-declaration]
  iounmap(cmt-mapbase_ch);
  ^
   cc1: some warnings being treated as errors
   make[2]: *** [drivers/clocksource/sh_cmt.o] Error 1
   make[1]: *** [drivers/clocksource] Error 2
   make: *** [drivers] Error 2

 Signed-off-by: Chen Gang gang.chen.5...@gmail.com
 ---
  arch/score/include/asm/io.h | 16 
  1 file changed, 16 insertions(+)

 diff --git a/arch/score/include/asm/io.h b/arch/score/include/asm/io.h
 index 574c8827..d67aeea 100644
 --- a/arch/score/include/asm/io.h
 +++ b/arch/score/include/asm/io.h
 @@ -3,6 +3,22 @@

  #include asm-generic/io.h

 +#ifdef CONFIG_MMU
 +static inline void __iomem *ioremap(phys_addr_t offset, unsigned long size)
 +{
 +   return (void __iomem *) (unsigned long)offset;
 +}
 +
 +#define __ioremap(offset, size, flags) ioremap(offset, size)
 +
 +#define ioremap_nocache ioremap
 +#define ioremap_wc ioremap_nocache
 +
 +static inline void iounmap(void __iomem *addr)
 +{
 +}
 +#endif
 +
  #define virt_to_busvirt_to_phys
  #define bus_to_virtphys_to_virt
  #endif /* _ASM_SCORE_IO_H */
 --
 1.9.2.459.g68773ac
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH 16/28] Remove GENERIC_HAS_IOMAP

[Lennox Wu]

Hi Pual,
OK, we will merge it into our tree.

Best,
Lennox

2014-04-15 17:19 GMT+08:00 Paul Bolle :
> On Mon, 2014-02-10 at 10:24 +0800, Lennox Wu wrote:
>> Thanks.
>>
>> Acked-by: Lennox Wu 
>
> I don't think this oneliner is queued somewhere. Lennox, you're listed
> as one of the score maintainers: could you perhaps queue this patch?
>
>
> Paul Bolle
>
>> 2014-02-10 2:47 GMT+08:00 Richard Weinberger :
>> > The symbol is an orphan, get rid of it.
>> >
>> > Signed-off-by: Richard Weinberger 
>> > ---
>> >  arch/score/Kconfig | 3 ---
>> >  1 file changed, 3 deletions(-)
>> >
>> > diff --git a/arch/score/Kconfig b/arch/score/Kconfig
>> > index c75d06a..c7a7b65 100644
>> > --- a/arch/score/Kconfig
>> > +++ b/arch/score/Kconfig
>> > @@ -23,19 +23,16 @@ config ARCH_SCORE7
>> > bool "SCORE7 processor"
>> > select SYS_SUPPORTS_32BIT_KERNEL
>> > select CPU_SCORE7
>> > -   select GENERIC_HAS_IOMAP
>> >
>> >  config MACH_SPCT6600
>> > bool "SPCT6600 series based machines"
>> > select SYS_SUPPORTS_32BIT_KERNEL
>> > select CPU_SCORE7
>> > -   select GENERIC_HAS_IOMAP
>> >
>> >  config SCORE_SIM
>> > bool "Score simulator"
>> > select SYS_SUPPORTS_32BIT_KERNEL
>> > select CPU_SCORE7
>> > -   select GENERIC_HAS_IOMAP
>> >  endchoice
>> >
>> >  endmenu
>> > --
>> > 1.8.4.2
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH 16/28] Remove GENERIC_HAS_IOMAP

[Lennox Wu]

Hi Pual,
OK, we will merge it into our tree.

Best,
Lennox

2014-04-15 17:19 GMT+08:00 Paul Bolle pebo...@tiscali.nl:
 On Mon, 2014-02-10 at 10:24 +0800, Lennox Wu wrote:
 Thanks.

 Acked-by: Lennox Wu lennox...@gmail.com

 I don't think this oneliner is queued somewhere. Lennox, you're listed
 as one of the score maintainers: could you perhaps queue this patch?


 Paul Bolle

 2014-02-10 2:47 GMT+08:00 Richard Weinberger rich...@nod.at:
  The symbol is an orphan, get rid of it.
 
  Signed-off-by: Richard Weinberger rich...@nod.at
  ---
   arch/score/Kconfig | 3 ---
   1 file changed, 3 deletions(-)
 
  diff --git a/arch/score/Kconfig b/arch/score/Kconfig
  index c75d06a..c7a7b65 100644
  --- a/arch/score/Kconfig
  +++ b/arch/score/Kconfig
  @@ -23,19 +23,16 @@ config ARCH_SCORE7
  bool SCORE7 processor
  select SYS_SUPPORTS_32BIT_KERNEL
  select CPU_SCORE7
  -   select GENERIC_HAS_IOMAP
 
   config MACH_SPCT6600
  bool SPCT6600 series based machines
  select SYS_SUPPORTS_32BIT_KERNEL
  select CPU_SCORE7
  -   select GENERIC_HAS_IOMAP
 
   config SCORE_SIM
  bool Score simulator
  select SYS_SUPPORTS_32BIT_KERNEL
  select CPU_SCORE7
  -   select GENERIC_HAS_IOMAP
   endchoice
 
   endmenu
  --
  1.8.4.2

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH 24/31] arch,score: Convert smp_mb__*

[Lennox Wu]

It's fine to S+core
Thanks :)

Acked-by: Lennox Wu

2014-03-19 14:47 GMT+08:00 Peter Zijlstra :
> score fully relies on asm-generic/barrier.h, so it can use its default
> implementation.
>
> Signed-off-by: Peter Zijlstra 
> ---
>  arch/score/include/asm/bitops.h |7 +--
>  1 file changed, 1 insertion(+), 6 deletions(-)
>
> --- a/arch/score/include/asm/bitops.h
> +++ b/arch/score/include/asm/bitops.h
> @@ -2,12 +2,7 @@
>  #define _ASM_SCORE_BITOPS_H
>
>  #include  /* swab32 */
> -
> -/*
> - * clear_bit() doesn't provide any barrier for the compiler.
> - */
> -#define smp_mb__before_clear_bit() barrier()
> -#define smp_mb__after_clear_bit()  barrier()
> +#include 
>
>  #include 
>  #include 
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH 24/31] arch,score: Convert smp_mb__*

[Lennox Wu]

It's fine to S+core
Thanks :)

Acked-by: Lennox Wulennox...@gmail.com

2014-03-19 14:47 GMT+08:00 Peter Zijlstra pet...@infradead.org:
 score fully relies on asm-generic/barrier.h, so it can use its default
 implementation.

 Signed-off-by: Peter Zijlstra pet...@infradead.org
 ---
  arch/score/include/asm/bitops.h |7 +--
  1 file changed, 1 insertion(+), 6 deletions(-)

 --- a/arch/score/include/asm/bitops.h
 +++ b/arch/score/include/asm/bitops.h
 @@ -2,12 +2,7 @@
  #define _ASM_SCORE_BITOPS_H

  #include asm/byteorder.h /* swab32 */
 -
 -/*
 - * clear_bit() doesn't provide any barrier for the compiler.
 - */
 -#define smp_mb__before_clear_bit() barrier()
 -#define smp_mb__after_clear_bit()  barrier()
 +#include asm/barrier.h

  #include asm-generic/bitops.h
  #include asm-generic/bitops/__fls.h


 --
 To unsubscribe from this list: send the line unsubscribe linux-kernel in
 the body of a message to majord...@vger.kernel.org
 More majordomo info at  http://vger.kernel.org/majordomo-info.html
 Please read the FAQ at  http://www.tux.org/lkml/
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] [RESEND] score: remove unused CPU_SCORE7 Kconfig parameter

[Lennox Wu]

Actually, people I don't want to brother are guys intergrate all
patches on the maillist.
There are too many patches whaic are waiting they to process.
I will check the development progress of the another Score and
consider if we should remove the config.
Thanks for your suggestion.

Best,
Lennox

2014-03-09 5:02 GMT+08:00 Paul Bolle :
> On Sun, 2014-03-09 at 04:27 +0800, Lennox Wu wrote:
>> 2014-03-09 4:00 GMT+08:00 Paul Bolle :
>> > In https://lkml.org/lkml/2011/11/12/76 I already reported that this
>> > Kconfig symbol was unused. Why can't it be dropped and re-added when
>> > code is added to the tree that actually uses it?
>>
>> Because I don't think droping it is meaningful, since it will be used.
>
> But, again, it isn't used now and hasn't actually been used in years.
>
>> Moreover, the symbol never affect other architectures or subsystems.
>> I feel that is not good to brother people to process the unimportant patch.
>
> Sure, this is just a small cleanup. But the scripts people use to see
> whether more serious Kconfig related issues have entered the tree will
> likely notice CPU_SCORE7. (And those people will have to investigate why
> that symbol is unused - which might take quite a bit of work - to
> determine what needs to be done about it.) So whenever someone, not
> following this thread, runs a script like that the score maintainers
> might receive a similar patch again.
>
> And I don't believe that handling a single cleanup patch is particularly
> burdensome for the score maintainers.
>
>
> Paul Bolle
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] [RESEND] score: remove unused CPU_SCORE7 Kconfig parameter

[Lennox Wu]

Because I don't think droping it is meaningful, since it will be used.
Moreover, the symbol never affect other architectures or subsystems.
I feel that is not good to brother people to process the unimportant patch.

Best,
Lennox


2014-03-09 4:00 GMT+08:00 Paul Bolle :
> Lennox,
>
> On Sat, 2014-03-08 at 14:34 +0800, Lennox Wu wrote:
>> According to my record, I had told you we have another
>> processor(2013/11/4), hence, we need the parameter.
>> We just wirte it done first. Do you remember it?
>
> In https://lkml.org/lkml/2011/11/12/76 I already reported that this
> Kconfig symbol was unused. Why can't it be dropped and re-added when
> code is added to the tree that actually uses it?
>
>
> Paul Bolle
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] [RESEND] score: remove unused CPU_SCORE7 Kconfig parameter

[Lennox Wu]

Because I don't think droping it is meaningful, since it will be used.
Moreover, the symbol never affect other architectures or subsystems.
I feel that is not good to brother people to process the unimportant patch.

Best,
Lennox


2014-03-09 4:00 GMT+08:00 Paul Bolle pebo...@tiscali.nl:
 Lennox,

 On Sat, 2014-03-08 at 14:34 +0800, Lennox Wu wrote:
 According to my record, I had told you we have another
 processor(2013/11/4), hence, we need the parameter.
 We just wirte it done first. Do you remember it?

 In https://lkml.org/lkml/2011/11/12/76 I already reported that this
 Kconfig symbol was unused. Why can't it be dropped and re-added when
 code is added to the tree that actually uses it?


 Paul Bolle

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] [RESEND] score: remove unused CPU_SCORE7 Kconfig parameter

[Lennox Wu]

Actually, people I don't want to brother are guys intergrate all
patches on the maillist.
There are too many patches whaic are waiting they to process.
I will check the development progress of the another Score and
consider if we should remove the config.
Thanks for your suggestion.

Best,
Lennox

2014-03-09 5:02 GMT+08:00 Paul Bolle pebo...@tiscali.nl:
 On Sun, 2014-03-09 at 04:27 +0800, Lennox Wu wrote:
 2014-03-09 4:00 GMT+08:00 Paul Bolle pebo...@tiscali.nl:
  In https://lkml.org/lkml/2011/11/12/76 I already reported that this
  Kconfig symbol was unused. Why can't it be dropped and re-added when
  code is added to the tree that actually uses it?

 Because I don't think droping it is meaningful, since it will be used.

 But, again, it isn't used now and hasn't actually been used in years.

 Moreover, the symbol never affect other architectures or subsystems.
 I feel that is not good to brother people to process the unimportant patch.

 Sure, this is just a small cleanup. But the scripts people use to see
 whether more serious Kconfig related issues have entered the tree will
 likely notice CPU_SCORE7. (And those people will have to investigate why
 that symbol is unused - which might take quite a bit of work - to
 determine what needs to be done about it.) So whenever someone, not
 following this thread, runs a script like that the score maintainers
 might receive a similar patch again.

 And I don't believe that handling a single cleanup patch is particularly
 burdensome for the score maintainers.


 Paul Bolle

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] [RESEND] score: remove unused CPU_SCORE7 Kconfig parameter

[Lennox Wu]

HI Michael,
According to my record, I had told you we have another
processor(2013/11/4), hence, we need the parameter.
We just wirte it done first. Do you remember it?

Best,
Lennox


2014-03-08 13:26 GMT+08:00 Michael Opdenacker
:
> This removes the CPU_SCORE7 Kconfig parameter,
> which is no longer used anywhere in the source code
> and Makefiles.
>
> Signed-off-by: Michael Opdenacker 
> ---
>  arch/score/Kconfig | 6 --
>  1 file changed, 6 deletions(-)
>
> diff --git a/arch/score/Kconfig b/arch/score/Kconfig
> index c75d06aa27c3..4ac8cae5727c 100644
> --- a/arch/score/Kconfig
> +++ b/arch/score/Kconfig
> @@ -22,27 +22,21 @@ choice
>  config ARCH_SCORE7
> bool "SCORE7 processor"
> select SYS_SUPPORTS_32BIT_KERNEL
> -   select CPU_SCORE7
> select GENERIC_HAS_IOMAP
>
>  config MACH_SPCT6600
> bool "SPCT6600 series based machines"
> select SYS_SUPPORTS_32BIT_KERNEL
> -   select CPU_SCORE7
> select GENERIC_HAS_IOMAP
>
>  config SCORE_SIM
> bool "Score simulator"
> select SYS_SUPPORTS_32BIT_KERNEL
> -   select CPU_SCORE7
> select GENERIC_HAS_IOMAP
>  endchoice
>
>  endmenu
>
> -config CPU_SCORE7
> -   bool
> -
>  config NO_DMA
> bool
> default y
> --
> 1.8.3.2
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] [RESEND] score: remove unused CPU_SCORE7 Kconfig parameter

[Lennox Wu]

HI Michael,
According to my record, I had told you we have another
processor(2013/11/4), hence, we need the parameter.
We just wirte it done first. Do you remember it?

Best,
Lennox


2014-03-08 13:26 GMT+08:00 Michael Opdenacker
michael.opdenac...@free-electrons.com:
 This removes the CPU_SCORE7 Kconfig parameter,
 which is no longer used anywhere in the source code
 and Makefiles.

 Signed-off-by: Michael Opdenacker michael.opdenac...@free-electrons.com
 ---
  arch/score/Kconfig | 6 --
  1 file changed, 6 deletions(-)

 diff --git a/arch/score/Kconfig b/arch/score/Kconfig
 index c75d06aa27c3..4ac8cae5727c 100644
 --- a/arch/score/Kconfig
 +++ b/arch/score/Kconfig
 @@ -22,27 +22,21 @@ choice
  config ARCH_SCORE7
 bool SCORE7 processor
 select SYS_SUPPORTS_32BIT_KERNEL
 -   select CPU_SCORE7
 select GENERIC_HAS_IOMAP

  config MACH_SPCT6600
 bool SPCT6600 series based machines
 select SYS_SUPPORTS_32BIT_KERNEL
 -   select CPU_SCORE7
 select GENERIC_HAS_IOMAP

  config SCORE_SIM
 bool Score simulator
 select SYS_SUPPORTS_32BIT_KERNEL
 -   select CPU_SCORE7
 select GENERIC_HAS_IOMAP
  endchoice

  endmenu

 -config CPU_SCORE7
 -   bool
 -
  config NO_DMA
 bool
 default y
 --
 1.8.3.2

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] [RESEND] score: remove deprecated IRQF_DISABLED

[Lennox Wu]

Thanks for cleaning the code.

Acked-by: Lennox Wu

2014-03-05 5:17 GMT+08:00 Michael Opdenacker
:
> This patch removes the use of the IRQF_DISABLED flag
> from arch/score/kernel/time.c
>
> It's a NOOP since 2.6.35 and it will be removed one day.
>
> Signed-off-by: Michael Opdenacker 
> ---
>  arch/score/kernel/time.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/score/kernel/time.c b/arch/score/kernel/time.c
> index f0a43affb201..24770cd9b473 100644
> --- a/arch/score/kernel/time.c
> +++ b/arch/score/kernel/time.c
> @@ -41,7 +41,7 @@ static irqreturn_t timer_interrupt(int irq, void *dev_id)
>
>  static struct irqaction timer_irq = {
> .handler = timer_interrupt,
> -   .flags = IRQF_DISABLED | IRQF_TIMER,
> +   .flags = IRQF_TIMER,
> .name = "timer",
>  };
>
> --
> 1.8.3.2
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] [RESEND] score: remove deprecated IRQF_DISABLED

[Lennox Wu]

Thanks for cleaning the code.

Acked-by: Lennox Wulennox...@gmail.com

2014-03-05 5:17 GMT+08:00 Michael Opdenacker
michael.opdenac...@free-electrons.com:
 This patch removes the use of the IRQF_DISABLED flag
 from arch/score/kernel/time.c

 It's a NOOP since 2.6.35 and it will be removed one day.

 Signed-off-by: Michael Opdenacker michael.opdenac...@free-electrons.com
 ---
  arch/score/kernel/time.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

 diff --git a/arch/score/kernel/time.c b/arch/score/kernel/time.c
 index f0a43affb201..24770cd9b473 100644
 --- a/arch/score/kernel/time.c
 +++ b/arch/score/kernel/time.c
 @@ -41,7 +41,7 @@ static irqreturn_t timer_interrupt(int irq, void *dev_id)

  static struct irqaction timer_irq = {
 .handler = timer_interrupt,
 -   .flags = IRQF_DISABLED | IRQF_TIMER,
 +   .flags = IRQF_TIMER,
 .name = timer,
  };

 --
 1.8.3.2

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] asm-generic: add sched_setattr/sched_getattr syscalls

[Lennox Wu]

 Acked-by: Lennox Wu

2014-02-03 23:33 GMT+08:00 James Hogan :
> Add the sched_setattr and sched_getattr syscalls to the generic syscall
> list, which is used by the following architectures: arc, arm64, c6x,
> hexagon, metag, openrisc, score, tile, unicore32.
>
> Signed-off-by: James Hogan 
> Cc: Arnd Bergmann 
> Cc: linux-a...@vger.kernel.org
> Cc: Vineet Gupta 
> Cc: Catalin Marinas 
> Cc: Will Deacon 
> Cc: linux-arm-ker...@lists.infradead.org
> Cc: Mark Salter 
> Cc: Aurelien Jacquiot 
> Cc: linux-c6x-...@linux-c6x.org
> Cc: Richard Kuo 
> Cc: linux-hexa...@vger.kernel.org
> Cc: linux-me...@vger.kernel.org
> Cc: Jonas Bonn 
> Cc: li...@lists.openrisc.net
> Cc: Chen Liqin 
> Cc: Lennox Wu 
> Cc: Chris Metcalf 
> Cc: Guan Xuetao 
> ---
>  include/uapi/asm-generic/unistd.h | 6 +->  1 file changed, 5 
> insertions(+), 1 deletion(-)
>
> diff --git a/include/uapi/asm-generic/unistd.h 
> b/include/uapi/asm-generic/unistd.h
> index a20a9b4d3871..dde8041f40d2 100644
> --- a/include/uapi/asm-generic/unistd.h
> +++ b/include/uapi/asm-generic/unistd.h
> @@ -692,9 +692,13 @@ __SC_COMP(__NR_process_vm_writev, sys_process_vm_writev, 
> \
>  __SYSCALL(__NR_kcmp, sys_kcmp)
>  #define __NR_finit_module 273
>  __SYSCALL(__NR_finit_module, sys_finit_module)
> +#define __NR_sched_setattr 274
> +__SYSCALL(__NR_sched_setattr, sys_sched_setattr)
> +#define __NR_sched_getattr 275
> +__SYSCALL(__NR_sched_getattr, sys_sched_getattr)
>
>  #undef __NR_syscalls
> -#define __NR_syscalls 274
> +#define __NR_syscalls 276
>
>  /*
>   * All syscalls below here should go away really,
> --
> 1.8.1.2
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] asm-generic: add sched_setattr/sched_getattr syscalls

[Lennox Wu]

 Acked-by: Lennox Wulennox...@gmail.com

2014-02-03 23:33 GMT+08:00 James Hogan james.ho...@imgtec.com:
 Add the sched_setattr and sched_getattr syscalls to the generic syscall
 list, which is used by the following architectures: arc, arm64, c6x,
 hexagon, metag, openrisc, score, tile, unicore32.

 Signed-off-by: James Hogan james.ho...@imgtec.com
 Cc: Arnd Bergmann a...@arndb.de
 Cc: linux-a...@vger.kernel.org
 Cc: Vineet Gupta vgu...@synopsys.com
 Cc: Catalin Marinas catalin.mari...@arm.com
 Cc: Will Deacon will.dea...@arm.com
 Cc: linux-arm-ker...@lists.infradead.org
 Cc: Mark Salter msal...@redhat.com
 Cc: Aurelien Jacquiot a-jacqu...@ti.com
 Cc: linux-c6x-...@linux-c6x.org
 Cc: Richard Kuo r...@codeaurora.org
 Cc: linux-hexa...@vger.kernel.org
 Cc: linux-me...@vger.kernel.org
 Cc: Jonas Bonn jo...@southpole.se
 Cc: li...@lists.openrisc.net
 Cc: Chen Liqin liqin.li...@gmail.com
 Cc: Lennox Wu lennox...@gmail.com
 Cc: Chris Metcalf cmetc...@tilera.com
 Cc: Guan Xuetao g...@mprc.pku.edu.cn
 ---
  include/uapi/asm-generic/unistd.h | 6 +-  1 file changed, 5 
 insertions(+), 1 deletion(-)

 diff --git a/include/uapi/asm-generic/unistd.h 
 b/include/uapi/asm-generic/unistd.h
 index a20a9b4d3871..dde8041f40d2 100644
 --- a/include/uapi/asm-generic/unistd.h
 +++ b/include/uapi/asm-generic/unistd.h
 @@ -692,9 +692,13 @@ __SC_COMP(__NR_process_vm_writev, sys_process_vm_writev, 
 \
  __SYSCALL(__NR_kcmp, sys_kcmp)
  #define __NR_finit_module 273
  __SYSCALL(__NR_finit_module, sys_finit_module)
 +#define __NR_sched_setattr 274
 +__SYSCALL(__NR_sched_setattr, sys_sched_setattr)
 +#define __NR_sched_getattr 275
 +__SYSCALL(__NR_sched_getattr, sys_sched_getattr)

  #undef __NR_syscalls
 -#define __NR_syscalls 274
 +#define __NR_syscalls 276

  /*
   * All syscalls below here should go away really,
 --
 1.8.1.2


--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH 16/28] Remove GENERIC_HAS_IOMAP

[Lennox Wu]

Thanks.

Acked-by: Lennox Wu 

2014-02-10 2:47 GMT+08:00 Richard Weinberger :
> The symbol is an orphan, get rid of it.
>
> Signed-off-by: Richard Weinberger 
> ---
>  arch/score/Kconfig | 3 ---
>  1 file changed, 3 deletions(-)
>
> diff --git a/arch/score/Kconfig b/arch/score/Kconfig
> index c75d06a..c7a7b65 100644
> --- a/arch/score/Kconfig
> +++ b/arch/score/Kconfig
> @@ -23,19 +23,16 @@ config ARCH_SCORE7
> bool "SCORE7 processor"
> select SYS_SUPPORTS_32BIT_KERNEL
> select CPU_SCORE7
> -   select GENERIC_HAS_IOMAP
>
>  config MACH_SPCT6600
> bool "SPCT6600 series based machines"
> select SYS_SUPPORTS_32BIT_KERNEL
> select CPU_SCORE7
> -   select GENERIC_HAS_IOMAP
>
>  config SCORE_SIM
> bool "Score simulator"
> select SYS_SUPPORTS_32BIT_KERNEL
> select CPU_SCORE7
> -   select GENERIC_HAS_IOMAP
>  endchoice
>
>  endmenu
> --
> 1.8.4.2
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH 16/28] Remove GENERIC_HAS_IOMAP

[Lennox Wu]

Thanks.

Acked-by: Lennox Wu lennox...@gmail.com

2014-02-10 2:47 GMT+08:00 Richard Weinberger rich...@nod.at:
 The symbol is an orphan, get rid of it.

 Signed-off-by: Richard Weinberger rich...@nod.at
 ---
  arch/score/Kconfig | 3 ---
  1 file changed, 3 deletions(-)

 diff --git a/arch/score/Kconfig b/arch/score/Kconfig
 index c75d06a..c7a7b65 100644
 --- a/arch/score/Kconfig
 +++ b/arch/score/Kconfig
 @@ -23,19 +23,16 @@ config ARCH_SCORE7
 bool SCORE7 processor
 select SYS_SUPPORTS_32BIT_KERNEL
 select CPU_SCORE7
 -   select GENERIC_HAS_IOMAP

  config MACH_SPCT6600
 bool SPCT6600 series based machines
 select SYS_SUPPORTS_32BIT_KERNEL
 select CPU_SCORE7
 -   select GENERIC_HAS_IOMAP

  config SCORE_SIM
 bool Score simulator
 select SYS_SUPPORTS_32BIT_KERNEL
 select CPU_SCORE7
 -   select GENERIC_HAS_IOMAP
  endchoice

  endmenu
 --
 1.8.4.2

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] Score: Remove "select HAVE_GENERIC_HARDIRQS" again

[Lennox Wu]

Acked-by: Lennox Wu 

2014/1/22 Geert Uytterhoeven :
> Commit 5fbbf8a1a93452b26e7791cf32cefce62b0a480b ("Score: The commit is for
> compiling successfully.") re-introduced "select HAVE_GENERIC_HARDIRQS" in
> v3.12-rc4, which had just been removed in v3.12-rc1 by commit
> 0244ad004a54e39308d495fee0a2e637f8b5c317 ("Remove GENERIC_HARDIRQ config
> option").
>
> Signed-off-by: Geert Uytterhoeven 
> ---
>  arch/score/Kconfig |1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/arch/score/Kconfig b/arch/score/Kconfig
> index 305f7ee1f382..c75d06aa27c3 100644
> --- a/arch/score/Kconfig
> +++ b/arch/score/Kconfig
> @@ -2,7 +2,6 @@ menu "Machine selection"
>
>  config SCORE
> def_bool y
> -   select HAVE_GENERIC_HARDIRQS
> select GENERIC_IRQ_SHOW
> select GENERIC_IOMAP
> select GENERIC_ATOMIC64
> --
> 1.7.9.5
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] Score: Remove select HAVE_GENERIC_HARDIRQS again

[Lennox Wu]

Acked-by: Lennox Wu lennox...@gmail.com

2014/1/22 Geert Uytterhoeven ge...@linux-m68k.org:
 Commit 5fbbf8a1a93452b26e7791cf32cefce62b0a480b (Score: The commit is for
 compiling successfully.) re-introduced select HAVE_GENERIC_HARDIRQS in
 v3.12-rc4, which had just been removed in v3.12-rc1 by commit
 0244ad004a54e39308d495fee0a2e637f8b5c317 (Remove GENERIC_HARDIRQ config
 option).

 Signed-off-by: Geert Uytterhoeven ge...@linux-m68k.org
 ---
  arch/score/Kconfig |1 -
  1 file changed, 1 deletion(-)

 diff --git a/arch/score/Kconfig b/arch/score/Kconfig
 index 305f7ee1f382..c75d06aa27c3 100644
 --- a/arch/score/Kconfig
 +++ b/arch/score/Kconfig
 @@ -2,7 +2,6 @@ menu Machine selection

  config SCORE
 def_bool y
 -   select HAVE_GENERIC_HARDIRQS
 select GENERIC_IRQ_SHOW
 select GENERIC_IOMAP
 select GENERIC_ATOMIC64
 --
 1.7.9.5

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH 05/17] score: Use Kbuild logic to include

[Lennox Wu]

  Acked-by: Lennox Wu 

於 2013/11/13 上午 03:42, Geert Uytterhoeven 提到:
> Signed-off-by: Geert Uytterhoeven 
> Cc: Chen Liqin 
> Cc: Lennox Wu 
> ---
>  arch/score/include/asm/Kbuild |1 +
>  arch/score/include/asm/sections.h |6 --
>  2 files changed, 1 insertion(+), 6 deletions(-)
>  delete mode 100644 arch/score/include/asm/sections.h
>
> diff --git a/arch/score/include/asm/Kbuild b/arch/score/include/asm/Kbuild
> index f3414ade77a3..32cd967c19a4 100644
> --- a/arch/score/include/asm/Kbuild
> +++ b/arch/score/include/asm/Kbuild
> @@ -2,6 +2,7 @@
>  header-y +=
>  
>  generic-y += clkdev.h
> +generic-y += sections.h
>  generic-y += trace_clock.h
>  generic-y += xor.h
>  generic-y += preempt.h
> diff --git a/arch/score/include/asm/sections.h 
> b/arch/score/include/asm/sections.h
> deleted file mode 100644
> index 9441d23af005..
> --- a/arch/score/include/asm/sections.h
> +++ /dev/null
> @@ -1,6 +0,0 @@
> -#ifndef _ASM_SCORE_SECTIONS_H
> -#define _ASM_SCORE_SECTIONS_H
> -
> -#include 
> -
> -#endif /* _ASM_SCORE_SECTIONS_H */




signature.asc
Description: OpenPGP digital signature

Re: [PATCH 05/17] score: Use Kbuild logic to include asm-generic/sections.h

[Lennox Wu]

  Acked-by: Lennox Wu lennox...@gmail.com

於 2013/11/13 上午 03:42, Geert Uytterhoeven 提到:
 Signed-off-by: Geert Uytterhoeven ge...@linux-m68k.org
 Cc: Chen Liqin liqin.li...@gmail.com
 Cc: Lennox Wu lennox...@gmail.com
 ---
  arch/score/include/asm/Kbuild |1 +
  arch/score/include/asm/sections.h |6 --
  2 files changed, 1 insertion(+), 6 deletions(-)
  delete mode 100644 arch/score/include/asm/sections.h

 diff --git a/arch/score/include/asm/Kbuild b/arch/score/include/asm/Kbuild
 index f3414ade77a3..32cd967c19a4 100644
 --- a/arch/score/include/asm/Kbuild
 +++ b/arch/score/include/asm/Kbuild
 @@ -2,6 +2,7 @@
  header-y +=
  
  generic-y += clkdev.h
 +generic-y += sections.h
  generic-y += trace_clock.h
  generic-y += xor.h
  generic-y += preempt.h
 diff --git a/arch/score/include/asm/sections.h 
 b/arch/score/include/asm/sections.h
 deleted file mode 100644
 index 9441d23af005..
 --- a/arch/score/include/asm/sections.h
 +++ /dev/null
 @@ -1,6 +0,0 @@
 -#ifndef _ASM_SCORE_SECTIONS_H
 -#define _ASM_SCORE_SECTIONS_H
 -
 -#include asm-generic/sections.h
 -
 -#endif /* _ASM_SCORE_SECTIONS_H */




signature.asc
Description: OpenPGP digital signature

Re: [PATCH] score: remove unused CPU_SCORE7 Kconfig parameter

[Lennox Wu]

Hi Michael,
We have another processor, the parameter will be used to differentiate
the different processors.
Anyway, thank you. :)

Best,
Lennox


2013/11/4 Michael Opdenacker :
> This removes the CPU_SCORE7 Kconfig parameter,
> which was no longer used anywhere in the source code
> and Makefiles.
>
> Signed-off-by: Michael Opdenacker 
> ---
>  arch/score/Kconfig | 6 --
>  1 file changed, 6 deletions(-)
>
> diff --git a/arch/score/Kconfig b/arch/score/Kconfig
> index 305f7ee1f382..2bc03d04f3af 100644
> --- a/arch/score/Kconfig
> +++ b/arch/score/Kconfig
> @@ -23,27 +23,21 @@ choice
>  config ARCH_SCORE7
> bool "SCORE7 processor"
> select SYS_SUPPORTS_32BIT_KERNEL
> -   select CPU_SCORE7
> select GENERIC_HAS_IOMAP
>
>  config MACH_SPCT6600
> bool "SPCT6600 series based machines"
> select SYS_SUPPORTS_32BIT_KERNEL
> -   select CPU_SCORE7
> select GENERIC_HAS_IOMAP
>
>  config SCORE_SIM
> bool "Score simulator"
> select SYS_SUPPORTS_32BIT_KERNEL
> -   select CPU_SCORE7
> select GENERIC_HAS_IOMAP
>  endchoice
>
>  endmenu
>
> -config CPU_SCORE7
> -   bool
> -
>  config NO_DMA
> bool
> default y
> --
> 1.8.1.2
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] score: remove unused CPU_SCORE7 Kconfig parameter

[Lennox Wu]

Hi Michael,
We have another processor, the parameter will be used to differentiate
the different processors.
Anyway, thank you. :)

Best,
Lennox


2013/11/4 Michael Opdenacker michael.opdenac...@free-electrons.com:
 This removes the CPU_SCORE7 Kconfig parameter,
 which was no longer used anywhere in the source code
 and Makefiles.

 Signed-off-by: Michael Opdenacker michael.opdenac...@free-electrons.com
 ---
  arch/score/Kconfig | 6 --
  1 file changed, 6 deletions(-)

 diff --git a/arch/score/Kconfig b/arch/score/Kconfig
 index 305f7ee1f382..2bc03d04f3af 100644
 --- a/arch/score/Kconfig
 +++ b/arch/score/Kconfig
 @@ -23,27 +23,21 @@ choice
  config ARCH_SCORE7
 bool SCORE7 processor
 select SYS_SUPPORTS_32BIT_KERNEL
 -   select CPU_SCORE7
 select GENERIC_HAS_IOMAP

  config MACH_SPCT6600
 bool SPCT6600 series based machines
 select SYS_SUPPORTS_32BIT_KERNEL
 -   select CPU_SCORE7
 select GENERIC_HAS_IOMAP

  config SCORE_SIM
 bool Score simulator
 select SYS_SUPPORTS_32BIT_KERNEL
 -   select CPU_SCORE7
 select GENERIC_HAS_IOMAP
  endchoice

  endmenu

 -config CPU_SCORE7
 -   bool
 -
  config NO_DMA
 bool
 default y
 --
 1.8.1.2

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH 21/29] score: Use get_signal() signal_setup_done()

[Lennox Wu]

It's fine for Score. Pass the compiling and other tests.
Thank you.
Acked-by: Lennox Wu 

於 2013/10/8 下午 07:34, Richard Weinberger 提到:
> Use the more generic functions get_signal() signal_setup_done()
> for signal delivery.
>
> Signed-off-by: Richard Weinberger 
> ---
>  arch/score/kernel/signal.c | 43 ++-
>  1 file changed, 18 insertions(+), 25 deletions(-)
>
> diff --git a/arch/score/kernel/signal.c b/arch/score/kernel/signal.c
> index a00fba3..1651807 100644
> --- a/arch/score/kernel/signal.c
> +++ b/arch/score/kernel/signal.c
> @@ -173,15 +173,15 @@ badframe:
>   return 0;
>  }
>  
> -static int setup_rt_frame(struct k_sigaction *ka, struct pt_regs *regs,
> - int signr, sigset_t *set, siginfo_t *info)
> +static int setup_rt_frame(struct ksignal *ksig, struct pt_regs *regs,
> +   sigset_t *set)
>  {
>   struct rt_sigframe __user *frame;
>   int err = 0;
>  
> - frame = get_sigframe(ka, regs, sizeof(*frame));
> + frame = get_sigframe(>ka, regs, sizeof(*frame));
>   if (!access_ok(VERIFY_WRITE, frame, sizeof(*frame)))
> - goto give_sigsegv;
> + return -EFAULT;
>  
>   /*
>* Set up the return code ...
> @@ -194,7 +194,7 @@ static int setup_rt_frame(struct k_sigaction *ka, struct 
> pt_regs *regs,
>   err |= __put_user(0x80008002, frame->rs_code + 1);
>   flush_cache_sigtramp((unsigned long) frame->rs_code);
>  
> - err |= copy_siginfo_to_user(>rs_info, info);
> + err |= copy_siginfo_to_user(>rs_info, >info);
>   err |= __put_user(0, >rs_uc.uc_flags);
>   err |= __put_user(NULL, >rs_uc.uc_link);
>   err |= __save_altstack(>rs_uc.uc_stack, regs->regs[0]);
> @@ -202,26 +202,23 @@ static int setup_rt_frame(struct k_sigaction *ka, 
> struct pt_regs *regs,
>   err |= __copy_to_user(>rs_uc.uc_sigmask, set, sizeof(*set));
>  
>   if (err)
> - goto give_sigsegv;
> + return -EFAULT;
>  
>   regs->regs[0] = (unsigned long) frame;
>   regs->regs[3] = (unsigned long) frame->rs_code;
> - regs->regs[4] = signr;
> + regs->regs[4] = ksig->sig;
>   regs->regs[5] = (unsigned long) >rs_info;
>   regs->regs[6] = (unsigned long) >rs_uc;
> - regs->regs[29] = (unsigned long) ka->sa.sa_handler;
> - regs->cp0_epc = (unsigned long) ka->sa.sa_handler;
> + regs->regs[29] = (unsigned long) ksig->ka.sa.sa_handler;
> + regs->cp0_epc = (unsigned long) ksig->ka.sa.sa_handler;
>  
>   return 0;
> -
> -give_sigsegv:
> - force_sigsegv(signr, current);
> - return -EFAULT;
>  }
>  
> -static void handle_signal(unsigned long sig, siginfo_t *info,
> - struct k_sigaction *ka, struct pt_regs *regs)
> +static void handle_signal(struct ksignal *ksig, struct pt_regs *regs)
>  {
> + int ret;
> +
>   if (regs->is_syscall) {
>   switch (regs->regs[4]) {
>   case ERESTART_RESTARTBLOCK:
> @@ -229,7 +226,7 @@ static void handle_signal(unsigned long sig, siginfo_t 
> *info,
>   regs->regs[4] = EINTR;
>   break;
>   case ERESTARTSYS:
> - if (!(ka->sa.sa_flags & SA_RESTART)) {
> + if (!(ksig->ka.sa.sa_flags & SA_RESTART)) {
>   regs->regs[4] = EINTR;
>   break;
>   }
> @@ -245,17 +242,14 @@ static void handle_signal(unsigned long sig, siginfo_t 
> *info,
>   /*
>* Set up the stack frame
>*/
> - if (setup_rt_frame(ka, regs, sig, sigmask_to_save(), info) < 0)
> - return;
> + ret = setup_rt_frame(ksig, regs, sigmask_to_save());
>  
> - signal_delivered(sig, info, ka, regs, 0);
> + signal_setup_done(ret, ksig, 0);
>  }
>  
>  static void do_signal(struct pt_regs *regs)
>  {
> - struct k_sigaction ka;
> - siginfo_t info;
> - int signr;
> + struct ksignal ksig;
>  
>   /*
>* We want the common case to go fast, which is why we may in certain
> @@ -265,10 +259,9 @@ static void do_signal(struct pt_regs *regs)
>   if (!user_mode(regs))
>   return;
>  
> - signr = get_signal_to_deliver(, , regs, NULL);
> - if (signr > 0) {
> + if (get_signal()) {
>   /* Actually deliver the signal.  */
> - handle_signal(signr, , , regs);
> + handle_signal(, regs);
>   return;
>   }
>  




signature.asc
Description: OpenPGP digital signature

Re: [PATCH 21/29] score: Use get_signal() signal_setup_done()

[Lennox Wu]

It's fine for Score. Pass the compiling and other tests.
Thank you.
Acked-by: Lennox Wu lennox...@gmail.com

於 2013/10/8 下午 07:34, Richard Weinberger 提到:
 Use the more generic functions get_signal() signal_setup_done()
 for signal delivery.

 Signed-off-by: Richard Weinberger rich...@nod.at
 ---
  arch/score/kernel/signal.c | 43 ++-
  1 file changed, 18 insertions(+), 25 deletions(-)

 diff --git a/arch/score/kernel/signal.c b/arch/score/kernel/signal.c
 index a00fba3..1651807 100644
 --- a/arch/score/kernel/signal.c
 +++ b/arch/score/kernel/signal.c
 @@ -173,15 +173,15 @@ badframe:
   return 0;
  }
  
 -static int setup_rt_frame(struct k_sigaction *ka, struct pt_regs *regs,
 - int signr, sigset_t *set, siginfo_t *info)
 +static int setup_rt_frame(struct ksignal *ksig, struct pt_regs *regs,
 +   sigset_t *set)
  {
   struct rt_sigframe __user *frame;
   int err = 0;
  
 - frame = get_sigframe(ka, regs, sizeof(*frame));
 + frame = get_sigframe(ksig-ka, regs, sizeof(*frame));
   if (!access_ok(VERIFY_WRITE, frame, sizeof(*frame)))
 - goto give_sigsegv;
 + return -EFAULT;
  
   /*
* Set up the return code ...
 @@ -194,7 +194,7 @@ static int setup_rt_frame(struct k_sigaction *ka, struct 
 pt_regs *regs,
   err |= __put_user(0x80008002, frame-rs_code + 1);
   flush_cache_sigtramp((unsigned long) frame-rs_code);
  
 - err |= copy_siginfo_to_user(frame-rs_info, info);
 + err |= copy_siginfo_to_user(frame-rs_info, ksig-info);
   err |= __put_user(0, frame-rs_uc.uc_flags);
   err |= __put_user(NULL, frame-rs_uc.uc_link);
   err |= __save_altstack(frame-rs_uc.uc_stack, regs-regs[0]);
 @@ -202,26 +202,23 @@ static int setup_rt_frame(struct k_sigaction *ka, 
 struct pt_regs *regs,
   err |= __copy_to_user(frame-rs_uc.uc_sigmask, set, sizeof(*set));
  
   if (err)
 - goto give_sigsegv;
 + return -EFAULT;
  
   regs-regs[0] = (unsigned long) frame;
   regs-regs[3] = (unsigned long) frame-rs_code;
 - regs-regs[4] = signr;
 + regs-regs[4] = ksig-sig;
   regs-regs[5] = (unsigned long) frame-rs_info;
   regs-regs[6] = (unsigned long) frame-rs_uc;
 - regs-regs[29] = (unsigned long) ka-sa.sa_handler;
 - regs-cp0_epc = (unsigned long) ka-sa.sa_handler;
 + regs-regs[29] = (unsigned long) ksig-ka.sa.sa_handler;
 + regs-cp0_epc = (unsigned long) ksig-ka.sa.sa_handler;
  
   return 0;
 -
 -give_sigsegv:
 - force_sigsegv(signr, current);
 - return -EFAULT;
  }
  
 -static void handle_signal(unsigned long sig, siginfo_t *info,
 - struct k_sigaction *ka, struct pt_regs *regs)
 +static void handle_signal(struct ksignal *ksig, struct pt_regs *regs)
  {
 + int ret;
 +
   if (regs-is_syscall) {
   switch (regs-regs[4]) {
   case ERESTART_RESTARTBLOCK:
 @@ -229,7 +226,7 @@ static void handle_signal(unsigned long sig, siginfo_t 
 *info,
   regs-regs[4] = EINTR;
   break;
   case ERESTARTSYS:
 - if (!(ka-sa.sa_flags  SA_RESTART)) {
 + if (!(ksig-ka.sa.sa_flags  SA_RESTART)) {
   regs-regs[4] = EINTR;
   break;
   }
 @@ -245,17 +242,14 @@ static void handle_signal(unsigned long sig, siginfo_t 
 *info,
   /*
* Set up the stack frame
*/
 - if (setup_rt_frame(ka, regs, sig, sigmask_to_save(), info)  0)
 - return;
 + ret = setup_rt_frame(ksig, regs, sigmask_to_save());
  
 - signal_delivered(sig, info, ka, regs, 0);
 + signal_setup_done(ret, ksig, 0);
  }
  
  static void do_signal(struct pt_regs *regs)
  {
 - struct k_sigaction ka;
 - siginfo_t info;
 - int signr;
 + struct ksignal ksig;
  
   /*
* We want the common case to go fast, which is why we may in certain
 @@ -265,10 +259,9 @@ static void do_signal(struct pt_regs *regs)
   if (!user_mode(regs))
   return;
  
 - signr = get_signal_to_deliver(info, ka, regs, NULL);
 - if (signr  0) {
 + if (get_signal(ksig)) {
   /* Actually deliver the signal.  */
 - handle_signal(signr, info, ka, regs);
 + handle_signal(ksig, regs);
   return;
   }
  




signature.asc
Description: OpenPGP digital signature

Re: [PATCH 25/34] score: handle pgtable_page_ctor() fail

[Lennox Wu]

Thanks.

Acked-by: Lennox Wu 

於 2013/10/11 上午 02:05, Kirill A. Shutemov 提到:
> Signed-off-by: Kirill A. Shutemov 
> Cc: Chen Liqin 
> Cc: Lennox Wu 
> ---
>  arch/score/include/asm/pgalloc.h | 9 ++---
>  1 file changed, 6 insertions(+), 3 deletions(-)
>
> diff --git a/arch/score/include/asm/pgalloc.h 
> b/arch/score/include/asm/pgalloc.h
> index 059a61b707..2a861ffbd5 100644
> --- a/arch/score/include/asm/pgalloc.h
> +++ b/arch/score/include/asm/pgalloc.h
> @@ -54,9 +54,12 @@ static inline struct page *pte_alloc_one(struct mm_struct 
> *mm,
>   struct page *pte;
>  
>   pte = alloc_pages(GFP_KERNEL | __GFP_REPEAT, PTE_ORDER);
> - if (pte) {
> - clear_highpage(pte);
> - pgtable_page_ctor(pte);
> + if (!pte)
> + return NULL;
> + clear_highpage(pte);
> + if (!pgtable_page_ctor(pte)) {
> + __free_page(pte);
> + return NULL;
>   }
>   return pte;
>  }




signature.asc
Description: OpenPGP digital signature

Re: [PATCH 25/34] score: handle pgtable_page_ctor() fail

[Lennox Wu]

Thanks.

Acked-by: Lennox Wu lennox...@gmail.com

於 2013/10/11 上午 02:05, Kirill A. Shutemov 提到:
 Signed-off-by: Kirill A. Shutemov kirill.shute...@linux.intel.com
 Cc: Chen Liqin liqin.c...@sunplusct.com
 Cc: Lennox Wu lennox...@gmail.com
 ---
  arch/score/include/asm/pgalloc.h | 9 ++---
  1 file changed, 6 insertions(+), 3 deletions(-)

 diff --git a/arch/score/include/asm/pgalloc.h 
 b/arch/score/include/asm/pgalloc.h
 index 059a61b707..2a861ffbd5 100644
 --- a/arch/score/include/asm/pgalloc.h
 +++ b/arch/score/include/asm/pgalloc.h
 @@ -54,9 +54,12 @@ static inline struct page *pte_alloc_one(struct mm_struct 
 *mm,
   struct page *pte;
  
   pte = alloc_pages(GFP_KERNEL | __GFP_REPEAT, PTE_ORDER);
 - if (pte) {
 - clear_highpage(pte);
 - pgtable_page_ctor(pte);
 + if (!pte)
 + return NULL;
 + clear_highpage(pte);
 + if (!pgtable_page_ctor(pte)) {
 + __free_page(pte);
 + return NULL;
   }
   return pte;
  }




signature.asc
Description: OpenPGP digital signature

[git pull] Score: Fix compiling errors and update the information of maintainers (Fixed format 2)

[Lennox Wu]

Hi Linus,
Please pull these updates for S+core architecture. These updates include
updating information of maintainers, fix some trivial errors, and add
a necessary function for supporting ipv6. We fix the format of the link
and use a tag.

The following changes since commit 22356f447ceb8d97a4885792e7d9e4607f712e1b

mm: Place preemption point in do_mlockall() loop (2013-9-24 18:29:11 -0700)

are available in the git repository at:

git://github.com/sctscore/official-linux.git tags/for-linus-20130929

For you to fetch changes up to ed38665eb50ef7ad530f9b2db531a09b8520ed32

Score: Update the information of Score maintaners (2013-9-26 01:41:22 +0800)

Thanks for your patience.

Best,
Lennox Wu

---
Summary these commits:
1. Fix some trivial errors for successfully compiling the latest
version Linux kernel.
2. Provide necessary function for support IPV6.
3. Update the information of maintainers.

Signed-off-by: Lennox Wu 


 MAINTAINERS   |4 +-
 arch/score/Kconfig|4 ++
 arch/score/Makefile   |4 +-
 arch/score/include/asm/checksum.h |   93
-
 arch/score/include/asm/io.h   |1 -
 arch/score/include/asm/pgalloc.h  |2 +-
 arch/score/kernel/entry.S |4 +-
 arch/score/kernel/process.c   |4 +-
 8 files changed, 64 insertions(+), 52 deletions(-)



signature.asc
Description: OpenPGP digital signature

Re: [git pull] Score: Fix compiling errors and update the information of maintainers (Fixed format)

[Lennox Wu]

Sorry, please ignor the mail. Some thing wrong in my mail client. The
mail includes wrong format.

Lennox
於 2013/9/30 下午 10:45, Lennox Wu 提到:
> Hi Linus,
> Please pull these updates for S+core architecture. These updates include
> updating information of maintainers, fix some trivial errors, and add
> a necessary function for supporting ipv6. We fix the format of the link
> and use a tag.
>
> The following changes since commit 22356f447ceb8d97a4885792e7d9e4607f712e1b
>
> mm: Place preemption point in do_mlockall() loop (2013-9-24 18:29:11 -0700)
>
> are available in the git repository at:
>
> git://github.com/sctscore/official-linux.git
> <https://github.com/sctscore/official-linux.git>tags/for-linus-20130929
>
> For you to fetch changes up to ed38665eb50ef7ad530f9b2db531a09b8520ed32
>
> Score: Update the information of Score maintaners (2013-9-26 01:41:22 +0800)
>
>
> Thanks for your patience.
>
> Best,
> Lennox Wu
>
> ---
> Summary these commits:
> 1. Fix some trivial errors for successfully compiling the latest
> version Linux kernel.
> 2. Provide necessary function for support IPV6.
> 3. Update the information of maintainers.
>
> Signed-off-by: Lennox Wu mailto:lennox...@gmail.com>>
>
> 
>  MAINTAINERS   |4 +-
>  arch/score/Kconfig|4 ++
>  arch/score/Makefile   |4 +-
>  arch/score/include/asm/checksum.h |   93
> -
>  arch/score/include/asm/io.h   |1 -
>  arch/score/include/asm/pgalloc.h  |2 +-
>  arch/score/kernel/entry.S |4 +-
>  arch/score/kernel/process.c   |4 +-
>  8 files changed, 64 insertions(+), 52 deletions(-)
>




signature.asc
Description: OpenPGP digital signature

[git pull] Score: Fix compiling errors and update the information of maintainers (Fixed format)

[Lennox Wu]

Hi Linus,
Please pull these updates for S+core architecture. These updates include
updating information of maintainers, fix some trivial errors, and add
a necessary function for supporting ipv6. We fix the format of the link
and use a tag.

The following changes since commit 22356f447ceb8d97a4885792e7d9e4607f712e1b

mm: Place preemption point in do_mlockall() loop (2013-9-24 18:29:11 -0700)

are available in the git repository at:

git://github.com/sctscore/official-linux.git
<https://github.com/sctscore/official-linux.git>tags/for-linus-20130929

For you to fetch changes up to ed38665eb50ef7ad530f9b2db531a09b8520ed32

Score: Update the information of Score maintaners (2013-9-26 01:41:22 +0800)


Thanks for your patience.

Best,
Lennox Wu

---
Summary these commits:
1. Fix some trivial errors for successfully compiling the latest
version Linux kernel.
2. Provide necessary function for support IPV6.
3. Update the information of maintainers.

Signed-off-by: Lennox Wu mailto:lennox...@gmail.com>>


 MAINTAINERS   |4 +-
 arch/score/Kconfig|4 ++
 arch/score/Makefile   |4 +-
 arch/score/include/asm/checksum.h |   93
-
 arch/score/include/asm/io.h   |1 -
 arch/score/include/asm/pgalloc.h  |2 +-
 arch/score/kernel/entry.S |4 +-
 arch/score/kernel/process.c   |4 +-
 8 files changed, 64 insertions(+), 52 deletions(-)



signature.asc
Description: OpenPGP digital signature

[git pull] Score: Fix compiling errors and update the information of maintainers (Fixed format)

[Lennox Wu]

Hi Linus,
Please pull these updates for S+core architecture. These updates include
updating information of maintainers, fix some trivial errors, and add
a necessary function for supporting ipv6. We fix the format of the link
and use a tag.

The following changes since commit 22356f447ceb8d97a4885792e7d9e4607f712e1b

mm: Place preemption point in do_mlockall() loop (2013-9-24 18:29:11 -0700)

are available in the git repository at:

git://github.com/sctscore/official-linux.git
https://github.com/sctscore/official-linux.gittags/for-linus-20130929

For you to fetch changes up to ed38665eb50ef7ad530f9b2db531a09b8520ed32

Score: Update the information of Score maintaners (2013-9-26 01:41:22 +0800)


Thanks for your patience.

Best,
Lennox Wu

---
Summary these commits:
1. Fix some trivial errors for successfully compiling the latest
version Linux kernel.
2. Provide necessary function for support IPV6.
3. Update the information of maintainers.

Signed-off-by: Lennox Wu lennox...@gmail.com mailto:lennox...@gmail.com


 MAINTAINERS   |4 +-
 arch/score/Kconfig|4 ++
 arch/score/Makefile   |4 +-
 arch/score/include/asm/checksum.h |   93
-
 arch/score/include/asm/io.h   |1 -
 arch/score/include/asm/pgalloc.h  |2 +-
 arch/score/kernel/entry.S |4 +-
 arch/score/kernel/process.c   |4 +-
 8 files changed, 64 insertions(+), 52 deletions(-)



signature.asc
Description: OpenPGP digital signature

Re: [git pull] Score: Fix compiling errors and update the information of maintainers (Fixed format)

[Lennox Wu]

Sorry, please ignor the mail. Some thing wrong in my mail client. The
mail includes wrong format.

Lennox
於 2013/9/30 下午 10:45, Lennox Wu 提到:
 Hi Linus,
 Please pull these updates for S+core architecture. These updates include
 updating information of maintainers, fix some trivial errors, and add
 a necessary function for supporting ipv6. We fix the format of the link
 and use a tag.

 The following changes since commit 22356f447ceb8d97a4885792e7d9e4607f712e1b

 mm: Place preemption point in do_mlockall() loop (2013-9-24 18:29:11 -0700)

 are available in the git repository at:

 git://github.com/sctscore/official-linux.git
 https://github.com/sctscore/official-linux.gittags/for-linus-20130929

 For you to fetch changes up to ed38665eb50ef7ad530f9b2db531a09b8520ed32

 Score: Update the information of Score maintaners (2013-9-26 01:41:22 +0800)


 Thanks for your patience.

 Best,
 Lennox Wu

 ---
 Summary these commits:
 1. Fix some trivial errors for successfully compiling the latest
 version Linux kernel.
 2. Provide necessary function for support IPV6.
 3. Update the information of maintainers.

 Signed-off-by: Lennox Wu lennox...@gmail.com mailto:lennox...@gmail.com

 
  MAINTAINERS   |4 +-
  arch/score/Kconfig|4 ++
  arch/score/Makefile   |4 +-
  arch/score/include/asm/checksum.h |   93
 -
  arch/score/include/asm/io.h   |1 -
  arch/score/include/asm/pgalloc.h  |2 +-
  arch/score/kernel/entry.S |4 +-
  arch/score/kernel/process.c   |4 +-
  8 files changed, 64 insertions(+), 52 deletions(-)





signature.asc
Description: OpenPGP digital signature

[git pull] Score: Fix compiling errors and update the information of maintainers (Fixed format 2)

[Lennox Wu]

Hi Linus,
Please pull these updates for S+core architecture. These updates include
updating information of maintainers, fix some trivial errors, and add
a necessary function for supporting ipv6. We fix the format of the link
and use a tag.

The following changes since commit 22356f447ceb8d97a4885792e7d9e4607f712e1b

mm: Place preemption point in do_mlockall() loop (2013-9-24 18:29:11 -0700)

are available in the git repository at:

git://github.com/sctscore/official-linux.git tags/for-linus-20130929

For you to fetch changes up to ed38665eb50ef7ad530f9b2db531a09b8520ed32

Score: Update the information of Score maintaners (2013-9-26 01:41:22 +0800)

Thanks for your patience.

Best,
Lennox Wu

---
Summary these commits:
1. Fix some trivial errors for successfully compiling the latest
version Linux kernel.
2. Provide necessary function for support IPV6.
3. Update the information of maintainers.

Signed-off-by: Lennox Wu lennox...@gmail.com


 MAINTAINERS   |4 +-
 arch/score/Kconfig|4 ++
 arch/score/Makefile   |4 +-
 arch/score/include/asm/checksum.h |   93
-
 arch/score/include/asm/io.h   |1 -
 arch/score/include/asm/pgalloc.h  |2 +-
 arch/score/kernel/entry.S |4 +-
 arch/score/kernel/process.c   |4 +-
 8 files changed, 64 insertions(+), 52 deletions(-)



signature.asc
Description: OpenPGP digital signature

[git pull] Score: Fix compiling error and updatr thee information of maintainers

[Lennox Wu]

Hi Linus,
Please pull these updates for S+core architecture. These updates include
updating information of maintainers, fix some trivial errors, and add
a necessary function for supporting ipv6.

The following changes since commit 22356f447ceb8d97a4885792e7d9e4607f712e1b

mm: Place preemption point in do_mlockall() loop (2013-9-24 18:29:11 -0700)

are available in the git repository at:

https://github.com/sctscore/official-linux.git release-up-to-date

For you to fetch changes up to ed38665eb50ef7ad530f9b2db531a09b8520ed32

Score: Update the information of Score maintaners (2013-9-26 01:41:22 +0800)

Best,
Lennox Wu

---
Summary these commits:
1. Fix some trivial errors for successfully compiling the latest
version Linux kernel.
2. Provide necessary function for support IPV6.
3. Update the information of maintainers.

Signed-off-by: Lennox Wu 


 MAINTAINERS   |4 +-
 arch/score/Kconfig|4 ++
 arch/score/Makefile   |4 +-
 arch/score/include/asm/checksum.h |   93 -
 arch/score/include/asm/io.h   |1 -
 arch/score/include/asm/pgalloc.h  |2 +-
 arch/score/kernel/entry.S |4 +-
 arch/score/kernel/process.c   |4 +-
 8 files changed, 64 insertions(+), 52 deletions(-)




signature.asc
Description: OpenPGP digital signature

[git pull] Score: Fix compiling error and updatr thee information of maintainers

[Lennox Wu]

Hi Linus,
Please pull these updates for S+core architecture. These updates include
updating information of maintainers, fix some trivial errors, and add
a necessary function for supporting ipv6.

The following changes since commit 22356f447ceb8d97a4885792e7d9e4607f712e1b

mm: Place preemption point in do_mlockall() loop (2013-9-24 18:29:11 -0700)

are available in the git repository at:

https://github.com/sctscore/official-linux.git release-up-to-date

For you to fetch changes up to ed38665eb50ef7ad530f9b2db531a09b8520ed32

Score: Update the information of Score maintaners (2013-9-26 01:41:22 +0800)

Best,
Lennox Wu

---
Summary these commits:
1. Fix some trivial errors for successfully compiling the latest
version Linux kernel.
2. Provide necessary function for support IPV6.
3. Update the information of maintainers.

Signed-off-by: Lennox Wu lennox...@gmail.com


 MAINTAINERS   |4 +-
 arch/score/Kconfig|4 ++
 arch/score/Makefile   |4 +-
 arch/score/include/asm/checksum.h |   93 -
 arch/score/include/asm/io.h   |1 -
 arch/score/include/asm/pgalloc.h  |2 +-
 arch/score/kernel/entry.S |4 +-
 arch/score/kernel/process.c   |4 +-
 8 files changed, 64 insertions(+), 52 deletions(-)




signature.asc
Description: OpenPGP digital signature