Am 18.07.2012 15:51, schrieb Andreas Färber: > Am 18.07.2012 10:51, schrieb Wenchao Xia: >> Hi, following is API draft, prototypes were taken from qemu/block.h, >> and the API prefix is changed frpm bdrv to qbdrvs, to declare related >> object is BlockDriverState, not BlockDriver. [...]
After the refactoring that Markus is working on it won't refer to a BlockDriverState, but to a BlockBackend. (And changes like this make quite clear why the internals of the current block layer are not suitable as a public API. The API needs to be defined in a separate layer than can abstract such changes away.) > So let the bikeshedding begin: ;) > > What about qbds_ prefix rather than qbdrvs_? I find the proposed mixture > of acronym (q for QEMU, b for Block, s for State) and abbreviation (drv > as in Driver) a bit ugly. > > Or just simply go for qblock - might be better memorable. :) Yes, something like qblk that isn't tied to internals sounds better. Kevin
