Jonathan Cameron <jonathan.came...@huawei.com> writes: >> > >> >> > +# >> >> > +# Inject an event record for a General Media Event (CXL r3.0 >> >> > 8.2.9.2.1.1) >> >> >> >> What's "CXL r3.0", and where could a reader find it? >> > >> > We have docs in docs/system/devices/cxl.rst that include the consortium >> > website which has download links on the front page. >> >> cxl.rst has >> >> References >> ---------- >> >> - Consortium website for specifications etc: >> http://www.computeexpresslink.org >> - Compute Express link Revision 2 specification, October 2020 >> - CEDT CFMWS & QTG _DSM ECN May 2021 >> >> Should the second reference be updated to 3.0? Exact title seems to be >> "The Compute Express Link™ (CXL™) 3.0 specification". Not sure we need >> to bother with the "™" in a reference. > > Yes. On the todo list is to update all the references to latest released > specification because old ones are unobtainable to non consortium members > unless they grabbed a copy in the past. > > Annoyingly this will be a repeated requirement as new spec versions are > released > but the cadence should be fairly low. > >> >> > I'm not sure we want >> > to >> > have lots of references to the URL spread throughout QEMU. I can add one >> > somewhere in cxl.json if you think it is important to have one here as >> > well. >> >> You could add an introduction right under the "# = CXL devices" heading, >> and include a full reference to the specification there. Suitably >> abbreviated references like the ones you use in this patch should then >> be fine. > > I tried doing that - it resulted in the index including an entry with all the > text. > So on the webpage, the contents list to the left includes whatever text you > put > in that block. > > I'm not sure why, or how to fix that.
Show me what you tried (as a patch or something I can git-pull), and I'll have a look. [...]