David Jenkins wrote:
> Sending this to the list instead of just to Corey (forgot to reply-all).
>
> Corey,
>
> Thanks for the comments. I had been waffleing on the same issue i.e.
> make it look like the message came from the ipmc receive message queue
> or from the ipmc event queue..
>
> I ended up going with the message queue because
>
> a) Some of the status register bits that cause the OEM0 flag are
> events but many are not. So what I am creating is not exclusively an
> event alert message but a more generic request message to the payload
> to perform an action (check status reg). Which seemed to fit better
> in the "it's a command to the SMS from some other entity" /
> handle_ipmb_get_msg_cmd() path.
>
> b) When a user space app registers to receive events - it gets all
> events and all events go to all registered apps. I liked being able
> to narrow the receiver list down by having an app that just registered
> for a particular netfn / cmd.
>
> The one fly in the ointment was the error response from the stack. I
> read it that the stack response when there is no listener will be to
> send a IPMI_INVALID_CMD_COMPLETION_CODE to the ipmc. From running it,
> it appears that the ipmc just drops this message.
>
> Given a & b and seeing that the error response caused no problems to
> the ipmc I went with passing it up as a message. If you think that it
> truly ought to be an event then I will be happy to go that way.
I've been waffling a little, too, but I still think it should be an
event, and an OEM event at that. I have two basic reasons for this:
* Events are meant for asynchronous notification, and this is
asynchronous.
* Though the IPMC may ignore this now, I'd like to not send anything
that might perturb the IPMC after a firmware upgrade or some new
version.
* It's possible to pick up an event that has occurred in the past,
if you are interested.
So I really think an event is the right way to go. Make sure to follow
the proper event format for an OEM event.
Thanks,
-corey
-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
Openipmi-developer mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openipmi-developer
