All of lore.kernel.org
 help / color / mirror / Atom feed
From: Deepak Kodihalli <dkodihal@linux.vnet.ibm.com>
To: Ed Tanous <ed.tanous@intel.com>, Jeremy Kerr <jk@ozlabs.org>
Cc: OpenBMC Maillist <openbmc@lists.ozlabs.org>
Subject: Re: Initial MCTP design proposal
Date: Fri, 8 Mar 2019 10:51:54 +0530	[thread overview]
Message-ID: <c3305f0e-fca2-9758-2908-1ba73e944858@linux.vnet.ibm.com> (raw)
In-Reply-To: <847a41e2-59cd-47dd-ac1c-7d6c433e003d@linux.vnet.ibm.com>

On 08/03/19 10:28 AM, Deepak Kodihalli wrote:
> On 08/03/19 1:05 AM, Ed Tanous wrote:
>>
>>
>> On 3/7/19 12:46 AM, Deepak Kodihalli wrote:
>>>
>>> I agree to this as well. I've got two PLDM stacks on my laptop
>>> (requester and responder) talking to each other over libmctp (over the
>>> serial binding - making use of fifos instead of an actual serial
>>> connection). I'd like to push a couple small commits to libmctp as well
>>> (I made a PR for one of them at the moment).
>> This is one thing I've wondered before about the way PLDM is being built
>> out.  Testing the implementation against itself doesn't really test the
>> correctness of it, unless you're creating a bunch of asserts on data
>> structure elements, and even then, it just tests that you're giving what
>> you expect, not that you faithfully implemented the spec.


Also, the application was not tested against itself. The application is 
mostly a responder now. It was tested against a requester 
implementation. This commit chain might help clarify : 
https://gerrit.openbmc-project.xyz/#/c/openbmc/pldm/+/19011/

>> Do you plan to run this implementation against something other than
>> itself to test at some point?  If so, what device/implementation are you
>> planning to target?
> 
> BMC to host communications over MCTP, over different physical channels 
> (LPC for eg), is one of the first goals. There are more details in the 
> PLDM/MCTP design docs that Jeremy and I had sent out.
> 
> The test that I described above was to integrate the PLDM stack with 
> libmctp and see how that goes, without having to wait for another device 
> implementation to be ready. I think this is a logical step towards 
> employing PLDM/MCTP for device-device communication. The PLDM daemon 
> will link with the MCTP libs, eventually.
> 
> Regards,
> Deepak
> 

  reply	other threads:[~2019-03-08  5:22 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-12-07  2:41 Initial MCTP design proposal Jeremy Kerr
2018-12-07  4:15 ` Naidoo, Nilan
2018-12-07  5:06   ` Jeremy Kerr
2018-12-07  5:40     ` Naidoo, Nilan
2018-12-07  5:13 ` Deepak Kodihalli
2018-12-07  7:41   ` Jeremy Kerr
2018-12-07 17:09   ` Supreeth Venkatesh
2018-12-07 18:53     ` Emily Shaffer
2018-12-07 20:06       ` Supreeth Venkatesh
2018-12-07 21:19       ` Jeremy Kerr
2018-12-11  1:14       ` Stewart Smith
2018-12-11 18:26         ` Tanous, Ed
2018-12-18  0:10           ` Stewart Smith
2018-12-10  6:14     ` Deepak Kodihalli
2018-12-10 17:40       ` Supreeth Venkatesh
2018-12-11  7:38         ` Deepak Kodihalli
2018-12-12 22:50           ` Supreeth Venkatesh
2018-12-07 16:38 ` Supreeth Venkatesh
2019-02-07 15:51 ` Brad Bishop
2019-02-08  6:48   ` Jeremy Kerr
2019-02-08 15:55     ` Supreeth Venkatesh
2019-02-11 18:57     ` Brad Bishop
2019-02-12  8:43       ` Jeremy Kerr
2019-03-06 20:04         ` Ed Tanous
2019-03-07  8:46           ` Deepak Kodihalli
2019-03-07 19:35             ` Ed Tanous
2019-03-08  4:58               ` Deepak Kodihalli
2019-03-08  5:21                 ` Deepak Kodihalli [this message]
2019-03-07 20:40             ` Supreeth Venkatesh
2019-03-18 12:12           ` Brad Bishop

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=c3305f0e-fca2-9758-2908-1ba73e944858@linux.vnet.ibm.com \
    --to=dkodihal@linux.vnet.ibm.com \
    --cc=ed.tanous@intel.com \
    --cc=jk@ozlabs.org \
    --cc=openbmc@lists.ozlabs.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 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.