All of lore.kernel.org
 help / color / mirror / Atom feed
From: Patrick Williams <patrick@stwcx.xyz>
To: "yugang.chen" <yugang.chen@linux.intel.com>
Cc: "openbmc@lists.ozlabs.org" <openbmc@lists.ozlabs.org>,
	Brad Bishop <bradleyb@fuzziesquirrel.com>,
	"Bills, Jason M" <jason.m.bills@linux.intel.com>,
	chunhui.jia@linux.intel.com
Subject: Re: apply for a new repo "openbmc/node-data-sync"
Date: Mon, 12 Apr 2021 06:29:13 -0500	[thread overview]
Message-ID: <YHQvCSTmDrZ6DVD8@heinlein> (raw)
In-Reply-To: <bd64513a-76f6-7617-9942-ced892b2c500@linux.intel.com>

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

On Mon, Apr 12, 2021 at 02:38:38PM +0800, yugang.chen wrote:
> thanks, Brad.
> 
> + openbmc@lists.ozlabs.org .
> 
> Dear All,
> 
> I'm Intel BMC engineer, we have a module is used to sync sensor data between BMC nodes in one Chassis system. In our Modular system, there are 2 nodes or 4 nodes, one node works as Primary node, the other nodes work as secondary nodes.
> 
> 1. Some sensors are only visible for primary BMC, but secondary nodes need the sensors for Fan control.
> 
> 2. Some sensor are in secondary nodes, they are needed to show them on primary node.
> 
> So, we need a new repo to sync the sensor data between primary node and secondary nodes, could you create a repo "openbmc/node-data-sync"? thanks.

Do you have a design on how this would be accomplished?  I'm curious
what your proposal ends up looking like at a dbus-level.  We should make
sure this is documented at a high level and also aligns with how we are
handling multi-host scenarios already in our dbus layout.

How are the two BMC devices connected?  How do you ensure you can safely
/ securely accept the data between the two BMCs?  If done poorly this
could open up a large security attack surface.

I suspect your software design is somewhat driven by a particular hardware
design as well, so we need to make sure it is clear the scoping of your
solution and should probably even communicate that in the name.  For
instance, if your data transport is IPMB over i2c and you don't plan on
this being a general daemon that could be expanded to a network
connection, we shouldn't name it so generally.

-- 
Patrick Williams

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

  reply	other threads:[~2021-04-12 11:29 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <cab2988f-76b8-6a30-5fa9-0ee8030af7f0@linux.intel.com>
     [not found] ` <AF9887DB-F6CB-4CE0-90B1-17FEA2497013@fuzziesquirrel.com>
2021-04-12  6:38   ` apply for a new repo "openbmc/node-data-sync" yugang.chen
2021-04-12 11:29     ` Patrick Williams [this message]
2021-04-12 19:26     ` Ed Tanous
2021-04-19  4:41       ` yugang.chen
2021-04-19 14:26         ` Brad Bishop
2021-04-23 13:04           ` Patrick Williams
2021-04-19  5:10 Milton Miller II

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=YHQvCSTmDrZ6DVD8@heinlein \
    --to=patrick@stwcx.xyz \
    --cc=bradleyb@fuzziesquirrel.com \
    --cc=chunhui.jia@linux.intel.com \
    --cc=jason.m.bills@linux.intel.com \
    --cc=openbmc@lists.ozlabs.org \
    --cc=yugang.chen@linux.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 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.