All of lore.kernel.org
 help / color / mirror / Atom feed
From: "chunhui.jia" <chunhui.jia@linux.intel.com>
To: "Lei Yu" <yulei.sh@bytedance.com>,
	"Vernon Mauery" <vernon.mauery@linux.intel.com>
Cc: Willy Tu <wltu@google.com>,
	OpenBMC Maillist <openbmc@lists.ozlabs.org>,
	Ed Tanous <edtanous@google.com>,
	Brad Bishop <bradleyb@fuzziesquirrel.com>,
	"apparao.puli" <apparao.puli@linux.intel.com>,
	Vijay Khemka <vijaykhemka@fb.com>
Subject: Re:  Re: Intel-ipmi-oem repo
Date: Fri, 15 Jan 2021 12:20:47 +0800	[thread overview]
Message-ID: <6001181A.3090500@linux.intel.com> (raw)
In-Reply-To: <CAGm54UGtnHK+MsNCbaWSOCxUtoTzOJddn+1h5DKUZy5sPEmm=w@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 1327 bytes --]

You use both fixed sensor and dynamic sensors?

2021-01-15 

chunhui.jia 



发件人:Lei Yu <yulei.sh@bytedance.com>
发送时间:2021-01-15 10:23
主题:Re: Intel-ipmi-oem repo
收件人:"Vernon Mauery"<vernon.mauery@linux.intel.com>
抄送:"Willy Tu"<wltu@google.com>,"OpenBMC Maillist"<openbmc@lists.ozlabs.org>,"chunhui.jia"<chunhui.jia@linux.intel.com>,"Ed Tanous"<edtanous@google.com>,"Brad Bishop"<bradleyb@fuzziesquirrel.com>,"apparao.puli"<apparao.puli@linux.intel.com>,"Vijay Khemka"<vijaykhemka@fb.com>

On Fri, Jan 15, 2021 at 3:23 AM Vernon Mauery 
<vernon.mauery@linux.intel.com> wrote: 
> I would say that if IBM is the only company using the sensor 
> implementation that is currently in ipmid, it would be best to move it 
> to the IBM OEM layer. But it is difficult in a project this size who is 
> using what. So leaving it in ipmid for now is fine. 
> 

This is not the case. Bytedance uses ipmid with fixed yaml as well. 
In our case, we have all the sensors on DBus created by 
entity-manager/dbus-sensors, and only part of them are necessary for 
ipmi. 
So we specifically define the necessary sensors (and 
inventory-sensors) in yaml and use the current ipmid to implement 
them. 

AFAIK Yadro and Ampere uses ipmid with fixed yaml too. 

--  
BRs, 
Lei YU 

[-- Attachment #2: Type: text/html, Size: 3782 bytes --]

  reply	other threads:[~2021-01-15  4:22 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-01-13  2:54 Intel-ipmi-oem repo Willy Tu
2021-01-14 14:39 ` Brad Bishop
2021-01-14 16:38   ` Ed Tanous
2021-01-14 17:38     ` Vernon Mauery
2021-01-14 19:40       ` Brad Bishop
2021-01-14 20:06         ` Ed Tanous
2021-01-14 21:46           ` Willy Tu
2021-01-15  2:23       ` Lei Yu
2021-01-15  4:20         ` chunhui.jia [this message]
2021-01-15  5:59           ` Lei Yu
2021-01-14 18:53     ` Brad Bishop
2021-01-14 20:00       ` Ed Tanous

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=6001181A.3090500@linux.intel.com \
    --to=chunhui.jia@linux.intel.com \
    --cc=apparao.puli@linux.intel.com \
    --cc=bradleyb@fuzziesquirrel.com \
    --cc=edtanous@google.com \
    --cc=openbmc@lists.ozlabs.org \
    --cc=vernon.mauery@linux.intel.com \
    --cc=vijaykhemka@fb.com \
    --cc=wltu@google.com \
    --cc=yulei.sh@bytedance.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.