RE: [PATCH v2 11/11] ARC: [plat-eznps] Handle memory error as an exception

From	Noam Camus <noamca@mellanox.com>
Newsgroups	linux.kernel
Subject	RE: [PATCH v2 11/11] ARC: [plat-eznps] Handle memory error as an exception
Date	2017-06-07 13:20 +0200
Message-ID	<tPHah-5UP-15@gated-at.bofh.it> (permalink)
References	<tM0lb-6QK-3@gated-at.bofh.it> <tM0lc-6QK-21@gated-at.bofh.it> <tPuZs-6kh-3@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

> From: Noam Camus 
> Sent: Wednesday, June 7, 2017 9:08 AM
>To: 'Vineet Gupta' <Vineet.Gupta1@synopsys.com>; linux-snps-arc@lists.infradead.org
>Cc: linux-kernel@vger.kernel.org; Elad Kanfi <eladkan@mellanox.com>
>Subject: RE: [PATCH v2 11/11] ARC: [plat-eznps] Handle memory error as an exception

>>From: Vineet Gupta [mailto:Vineet.Gupta1@synopsys.com] 
>>Sent: Wednesday, June 7, 2017 1:11 AM
>...
>>> +
>>> +config EZNPS_MEM_ERROR
>>> +       bool "ARC-EZchip Memory error as an exception"
>>> +       depends on ARC_PLAT_EZNPS
>>> +       default n

>>So you set default to "n" - thus by default it works for the simulator not silicon ?
>Correct, this way I "align" Sim environment to react as close as possible to how it work with silicon.

Sorry, but It is not correct.
Default is for silicon where it is naturally emits machine check and unlike simulator do not need OS to redirect the ISR L2 to machine check handler.
Above motivation of "align" is true, but default is silicon and not sim, as I wrote in my original configuration help.
Please re-update this CONFIG help section.

Thanks
-Noam

Thread

[PATCH v2 00/11] ARC plat-eznps upstream cont. Noam Camus <noamca@mellanox.com> - 2017-05-28 09:00 +0200
  [PATCH v2 05/11] ARC: [plat-eznps] typo fix at Kconfig Noam Camus <noamca@mellanox.com> - 2017-05-28 09:00 +0200
  [PATCH v2 06/11] ARC: [plat-eznps] Fix TLB Errata Noam Camus <noamca@mellanox.com> - 2017-05-28 09:00 +0200
  [PATCH v2 01/11] ARC: set level of log per CPU during boot to be debug level Noam Camus <noamca@mellanox.com> - 2017-05-28 09:00 +0200
  [PATCH v2 11/11] ARC: [plat-eznps] Handle memory error as an exception Noam Camus <noamca@mellanox.com> - 2017-05-28 09:00 +0200
    Re: [PATCH v2 11/11] ARC: [plat-eznps] Handle memory error as an  exception Vineet Gupta <Vineet.Gupta1@synopsys.com> - 2017-06-02 21:10 +0200
      RE: [PATCH v2 11/11] ARC: [plat-eznps] Handle memory error as an  exception Noam Camus <noamca@mellanox.com> - 2017-06-04 08:20 +0200
    Re: [PATCH v2 11/11] ARC: [plat-eznps] Handle memory error as an  exception Vineet Gupta <Vineet.Gupta1@synopsys.com> - 2017-06-07 00:20 +0200
      RE: [PATCH v2 11/11] ARC: [plat-eznps] Handle memory error as an  exception Noam Camus <noamca@mellanox.com> - 2017-06-07 08:10 +0200
      RE: [PATCH v2 11/11] ARC: [plat-eznps] Handle memory error as an  exception Noam Camus <noamca@mellanox.com> - 2017-06-07 13:20 +0200
        Re: [PATCH v2 11/11] ARC: [plat-eznps] Handle memory error as an  exception Vineet Gupta <Vineet.Gupta1@synopsys.com> - 2017-06-07 18:20 +0200
          Re: [PATCH v2 11/11] ARC: [plat-eznps] Handle memory error as an  exception Vineet Gupta <Vineet.Gupta1@synopsys.com> - 2017-06-08 18:40 +0200
            Re: [PATCH v2 11/11] ARC: [plat-eznps] Handle memory error as an  exception Vineet Gupta <Vineet.Gupta1@synopsys.com> - 2017-06-08 21:10 +0200
              Re: [PATCH v2 11/11] ARC: [plat-eznps] Handle memory error as an  exception Vineet Gupta <Vineet.Gupta1@synopsys.com> - 2017-06-12 19:50 +0200
  [PATCH v2 02/11] ARC: send ipi to all cpus sharing task mm in case of page fault Noam Camus <noamca@mellanox.com> - 2017-05-28 09:00 +0200
    Re: [PATCH v2 02/11] ARC: send ipi to all cpus sharing task mm in  case of page fault Vineet Gupta <Vineet.Gupta1@synopsys.com> - 2017-05-30 18:50 +0200

csiph-web