Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1490747 > unrolled thread

RE: [PATCH v6 1/9] rpmb: add Replay Protected Memory Block (RPMB) subsystem

Started by"Winkler, Tomas" <tomas.winkler@intel.com>
First post2016-09-24 22:40 +0200
Last post2016-09-24 22:40 +0200
Articles 1 — 1 participant

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  RE: [PATCH v6 1/9] rpmb: add Replay Protected Memory Block (RPMB)  subsystem "Winkler, Tomas" <tomas.winkler@intel.com> - 2016-09-24 22:40 +0200

#1490747 — RE: [PATCH v6 1/9] rpmb: add Replay Protected Memory Block (RPMB) subsystem

From"Winkler, Tomas" <tomas.winkler@intel.com>
Date2016-09-24 22:40 +0200
SubjectRE: [PATCH v6 1/9] rpmb: add Replay Protected Memory Block (RPMB) subsystem
Message-ID<sl1TP-5HT-11@gated-at.bofh.it>
> 
> 
> >
> > Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
> > Signed-off-by: Alexander Usyskin <alexander.usyskin@intel.com>
> Tested-by: Avri Altman <Avri.Altman@sandisk.com>
> 
> - mmc -  full functionality. One issue found that was fixed on V6: patch V6 2/9.
> - ufs - read & read counter only.  Testing is still wip.
> 
> 
> > +static int rpmb_request_verify(struct rpmb_dev *rdev, struct rpmb_data
> > +*rpmbd) {
> 
> Seems excessive - Isn't the standard should be enforced by the device?

Yes the device is responsible to detect the issues in the protocol. The rpmb_request_verify is called only for the managed interface (rpmb_cmd_req) the raw interface (rpm_cmd_seq) is going through.  Second,  we only check more or less if we can accommodate the request/response into the arguments.  If there is a specific issue  you see in this check, please let me know.

Thanks
Tomas


[toc] | [standalone]


Back to top | Article view | linux.kernel


csiph-web