public inbox for linux-ia64@vger.kernel.org
 help / color / mirror / Atom feed
From: Keith Owens <kaos@sgi.com>
To: linux-ia64@vger.kernel.org
Subject: Re: [RFC] Change the effect of SAL_{GET,CLEAR}_STATE_INFO for MCA/INIT records
Date: Mon, 13 Dec 2004 03:54:47 +0000	[thread overview]
Message-ID: <31666.1102910087@ocs3.ocs.com.au> (raw)
In-Reply-To: <12433.1102594994@ocs3.ocs.com.au>

On Fri, 10 Dec 2004 11:51:01 +0900, 
Hidetoshi Seto <seto.hidetoshi@jp.fujitsu.com> wrote:
>Keith Owens wrote:
>> Making these calls context sensitive is a small change to SAL.  The SAL
>> MCA/INIT handlers have already constructed and stored the required
>> record before calling the OS handler.  All SAL has to do is :-
>> 
>> * Save the address of the current record on this cpu.
>> * Call the OS handler.
>> * On return from the OS handler, clear the pointer to the current
>>   record.
>> * SAL_GET_STATE_INFO - if the current record pointer is set then return
>>   that record, otherwise do existing processing.
>> * SAL_CLEAR_STATE_INFO - if the current record pointer is set then
>>   clear that record and the current record pointer, otherwise do
>>   existing processing.
>
>That makes sense.
>
>When I was coding my OS_MCA handler, I used to think if there were something
>like SAL_{GET,CLEAR}_STATE_INFO_FROM_BOTTOM or some extra parameter for
>SAL_{GET,CLEAR}_STATE_INFO to select which log it should deal.
>
>Your proposal sounds good.
>It's more simple, and more easy.
>
>> This change is backwards compatible.
>
>Marvelous!
>
>BTW, who is in charge of receptionist for such proposal?

Intel own the SAL specification, so they have to accept the change to
make it official.  Obviously it helps if all the vendors agree that the
change is useful and causes no problems.  I have had no feedback from
HP, Bull or Intel (yet).


  parent reply	other threads:[~2004-12-13  3:54 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-12-09 12:23 [RFC] Change the effect of SAL_{GET,CLEAR}_STATE_INFO for MCA/INIT records Keith Owens
2004-12-10  2:51 ` [RFC] Change the effect of SAL_{GET,CLEAR}_STATE_INFO for MCA/INIT Hidetoshi Seto
2004-12-13  3:54 ` Keith Owens [this message]
2004-12-14  1:00 ` [RFC] Change the effect of SAL_{GET,CLEAR}_STATE_INFO for MCA/INIT records Luck, Tony
2004-12-14  1:26 ` Keith Owens

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=31666.1102910087@ocs3.ocs.com.au \
    --to=kaos@sgi.com \
    --cc=linux-ia64@vger.kernel.org \
    /path/to/YOUR_REPLY

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

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