...

> > +#
> > +# @corrected-persistent-error-count: Total number correctable errors  
> 
> Total number of correctable errors

Gah. I've now seen these too often so read that a good 5 times before
working out what was wrong!

> 
> > +#     in persistent memory
> > +#
> > +# Since: 8.1
> > +##
> > +{ 'command': 'cxl-inject-memory-module-event',
> > +  'data': { 'path': 'str', 'log': 'CxlEventLog', 'flags' : 'uint8',
> > +            'type': 'uint8', 'health-status': 'uint8',
> > +            'media-status': 'uint8', 'additional-status': 'uint8',
> > +            'life-used': 'uint8', 'temperature' : 'int16',
> > +            'dirty-shutdown-count': 'uint32',
> > +            'corrected-volatile-error-count': 'uint32',
> > +            'corrected-persistent-error-count': 'uint32'
> > +            }}
> > +
> >  ##
> >  # @cxl-inject-poison:
> >  #  
> 
> Neither these nitpicks nor the one on PATCH 5 calls for a respin.
> Simply fix them in the PR.  Thanks!

Thanks but for now Michael Tsirkin is still picking up the CXL focused
series based on patch sets so I'll spin a v9 with this stuff tidied up.

While, not true of this particular set, a lot of the CXL series are still
impacting the generic PCI code and so far Michael has been happy with that
approach.

Jonathan

Reply via email to