On 1/1/18 7:58 AM, Arkadi Sharshevsky wrote:
>
> Just to summarize the current fixes required:
>
> 1. ERIF dpipe table size is reporting wrong size. More precisely the
> ERIF table does not take rifs, so it should not be linked to the rif
> bank resource (is not part of this patchset, future extension).
> 2. Extended ACK user-space bug.
> 3. ABI documentation- Not sure we agreed upon it, Jiri?
>
> If I missed something please respond. Nothing of the fixes mentioned
> above is relevant for this patchset actually.
>
Can you fix the userspace command and then we come back to what else is
needed? Right now, it is hard to tell what is a user space bug and what
is a kernel space bug.
For example:
$ devlink resource set pci/0000:03:00.0 path /kvd/linear size 10000
$ devlink resource show pci/0000:03:00.0
pci/0000:03:00.0:
name kvd size 245760 size_valid true
resources:
name linear size 98304 occ 0
name hash_double size 60416
name hash_single size 87040
The set command did not fail, yet there is no size_new arg in the output
like there is for this change:
$ devlink resource set pci/0000:03:00.0 path /kvd/linear size 0
$ devlink resource show pci/0000:03:00.0
pci/0000:03:00.0:
name kvd size 245760 size_valid true
resources:
name linear size 98304 size_new 0 occ 0
name hash_double size 60416
name hash_single size 87040
