Patch Set #15, Line 97:
*bufsize = 0;
Did you want to get rid of this? The SMI external interface might assume that things are zero'd out on failure? I'm not really sure of the expectation of what would happen with SMMSTORE_RET_FAILURE from the callee.
Patch Set #15, Line 136:
There's no need for this.
you can do this here:
if (rdev_chain(store, store, end, data_sz - end))
Upon returning from this function. The region device is pointing at the next section that is writable.
And this would be 0 once scan_end() has positioned the region device correctly. And it can also be named correctly: offset.
To view, visit change 30432. To unsubscribe, or for help writing mail filters, visit settings.