public inbox for linux-acpi@vger.kernel.org
 help / color / mirror / Atom feed
From: Alexey Starikovskiy <aystarik@gmail.com>
To: Zhao Yakui <yakui.zhao@intel.com>
Cc: Alexey Starikovskiy <astarikovskiy@suse.de>,
	LenBrown <lenb@kernel.org>,
	"linux-acpi@vger.kernel.org" <linux-acpi@vger.kernel.org>,
	"Brown, Len" <len.brown@intel.com>
Subject: Re: Spreading FUD [was: The fatal  logic error in EC write transaction(EC transaction is done in interrupt context)]
Date: Mon, 10 Nov 2008 21:48:50 +0300	[thread overview]
Message-ID: <49188212.2020401@gmail.com> (raw)
In-Reply-To: <1226278911.3989.313.camel@yakui_zhao.sh.intel.com>

Zhao Yakui wrote:
> On Sun, 2008-11-09 at 23:27 +0800, Alexey Starikovskiy wrote:
>   
>> Yakui,
>>
>> Please take a look at the subject of the thread -- notice words "fatal"
>> and "(EC transaction is done in interrupt context)".
>> Looks horrible, isn't it? Now we begin to get into details, and it appears
>> that it is not fatal, introduced 3.5 years ago, and has nothing to do with
>> interrupt context transaction patch.
>> So, the whole point of your message is to spread FUD about "fast 
>> transaction" patch,
>> and show how great you are. I would guess that you've failed in 
>> achieving both goals.
>>     
> I won't argue with you about this issue. 
>   
Good.
>      In your patch the program logic states that the EC transaction is
> already finished. But in fact the EC transaction is not really finished.
> They are inconsistent. Why can't call this a fatal logic error? 
>      The more important is that the failure in EC transaction can't be
> detected in time. 
>      Why can't call this a fatal logic error?
>   
Here is Merriam-Webster' definition of "fatal", look at use case number 
4, which
is closest to your use of it:
<a href="http://www.merriam-webster.com/dictionary/fatal">fatal</a>
As we lived with the problem you decribe all these years, it can not be 
described as
"causing failure/death", thus it can not be called "fatal".

>     
> If this is not a fatal logic error, what can be called fatal? Like that
> Several commits are reverted each other as the following three commits.
>     a.7c010de7506954e973abfab5c5999c5a97f7a73e
>     b.4c611060660f0de3e9b8f02df207312bc6f5c331
>     c.f9319f903f898dd4b15dbc386499725ce6c59776
>
>   
If I had all these machines sitting on my desk side-by-side, I may have 
fixed them with one patch,
otherwise one need to make rounds. Do you know any other solution?
> If you still reply the subject so impolitely, Don't blame that I will
> forward the similar error to wider range.
>  
>   
You send false or bended statements not for the first time, do you 
expect any other treatment?
You certainly could do whatever you want, but prepare for others to 
include you in ignore/spam list.

Regards,
Alex.




  reply	other threads:[~2008-11-10 18:48 UTC|newest]

Thread overview: 43+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-11-07  4:39 ACPI patches for 2.6.28-rc3 Len Brown
2008-11-07  4:39 ` [PATCH 01/21] i7300_idle: Kconfig, show menu only on x86_64 Len Brown
2008-11-07  4:39   ` [PATCH 02/21] ACPI: EC: revert msleep patch Len Brown
2008-11-07  4:39   ` [PATCH 03/21] sony-laptop: Ignore missing _DIS method on pic device Len Brown
2008-11-07  4:39   ` [PATCH 04/21] ACPI: fix de-reference bug in power resource driver Len Brown
2008-11-07  4:39   ` [PATCH 05/21] ACPI: fan: Delete the strict check in power transition Len Brown
2008-11-07  4:39   ` [PATCH 06/21] ACPI: bugfix reporting of event handler status Len Brown
2008-11-07  4:39   ` [PATCH 07/21] ACPI EC: Fix regression due to use of uninitialized variable Len Brown
2008-11-07  6:13     ` Zhao Yakui
2008-11-07 19:45       ` Len Brown
2008-11-08 14:04         ` The fatal logic error in EC write transaction(EC transaction is done in interrupt context) Zhao, Yakui
2008-11-08 15:33           ` Alexey Starikovskiy
2008-11-08 15:58             ` Zhao, Yakui
2008-11-08 16:47               ` Alexey Starikovskiy
2008-11-08 18:42                 ` [PATCH] ACPI: EC: wait for last write gpe Alexey Starikovskiy
2008-11-09 12:45                   ` Zhao Yakui
2008-11-09 13:11                 ` The fatal logic error in EC write transaction(EC transaction is done in interrupt context) Zhao Yakui
2008-11-09 15:27                   ` Spreading FUD [was: The fatal logic error in EC write transaction(EC transaction is done in interrupt context)] Alexey Starikovskiy
2008-11-10  1:01                     ` Zhao Yakui
2008-11-10 18:48                       ` Alexey Starikovskiy [this message]
2008-11-07  4:39   ` [PATCH 08/21] intel_menlow: don't set max_state a negative value Len Brown
2008-11-07  4:39   ` [PATCH 09/21] ACPI: remove comments about debug layer/level to use Len Brown
2008-11-07  4:39   ` [PATCH 10/21] ACPI: SBS: remove useless acpi_cm_sbs_init() initcall Len Brown
2008-11-07  4:39   ` [PATCH 11/21] ACPI: remove CONFIG_ACPI_POWER Len Brown
2008-11-07  4:39   ` [PATCH 12/21] ACPI: remove CONFIG_ACPI_EC Len Brown
2008-11-07  4:39   ` [PATCH 13/21] intel_menlow: Add comment documenting legal GTHS values Len Brown
2008-11-07  4:39   ` [PATCH 14/21] intel_menlow: MAINTAINERS Len Brown
2008-11-07  4:39   ` [PATCH 15/21] Revert "ACPI: Ingore the RESET_REG_SUP bit when using ACPI reset mechanism" Len Brown
2008-11-07  4:39   ` [PATCH 16/21] PNP: add Bjorn Helgaas as PNP co-maintainer Len Brown
2008-11-07  4:39   ` [PATCH 17/21] ACPI: struct device - replace bus_id with dev_name(), dev_set_name() Len Brown
2008-11-07  4:39   ` [PATCH 18/21] fujitsu-laptop: fix section mismatch warning Len Brown
2008-11-07  4:39   ` [PATCH 19/21] ACPI: use macro to replace hard number Len Brown
2008-11-07  4:39   ` [PATCH 20/21] ACPI: avoid empty file name in sysfs Len Brown
2008-11-07  4:39   ` [PATCH 21/21] ACPI: EC: make kernel messages more useful when GPE storm is detected Len Brown
2008-11-07  9:18     ` Alan Jenkins
2008-11-07 13:54       ` Rafael J. Wysocki
2008-11-07 18:49         ` [stable] " Greg KH
2008-11-07 22:15           ` Rafael J. Wysocki
2008-11-11 21:23             ` Greg KH
2008-11-12 23:34               ` Rafael J. Wysocki
2008-11-13 21:46                 ` Greg KH
2008-11-07 17:41 ` ACPI patches for 2.6.28-rc3 Andrey Borzenkov
2008-11-08  2:48   ` Len Brown

Reply instructions:

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

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

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

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

  git send-email \
    --in-reply-to=49188212.2020401@gmail.com \
    --to=aystarik@gmail.com \
    --cc=astarikovskiy@suse.de \
    --cc=len.brown@intel.com \
    --cc=lenb@kernel.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=yakui.zhao@intel.com \
    /path/to/YOUR_REPLY

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

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