Sorry, strange to reply into a blank mail thread, my two comments are solved and for this series of patch:
Reviewed-by: Yi He <yi...@linaro.org> On 11 November 2016 at 22:50, Christophe Milard < christophe.mil...@linaro.org> wrote: > since V1: > -changes to match names suggested by Maxim. (Christophe) > > This patch series brings a set of functions to retrieve physical addresses > (from virtual addresses). These functions will be needed to populate the > physical addresses of packet segments. In the absence of iommu, drivers > need to know physical addresses. > The first patch brings the function to retrieve physical addresses. > The second patch implements a debug function for printing out the mapping > of some area. > Here follows a dump for a 8Mb memry area: > With Huge pages > _ishmphy.c:290:_odp_ishmphy_dumpphy():Phy Dump: > _ishmphy.c:309:_odp_ishmphy_dumpphy():Virtual: 7ffb4fc00000 <--> Phy: > 2a2c00000 512 PFNs, 2097152 bytes > _ishmphy.c:309:_odp_ishmphy_dumpphy():Virtual: 7ffb4fe00000 <--> Phy: > 2a2a00000 512 PFNs, 2097152 bytes > _ishmphy.c:309:_odp_ishmphy_dumpphy():Virtual: 7ffb50000000 <--> Phy: > 2a2800000 512 PFNs, 2097152 bytes > _ishmphy.c:323:_odp_ishmphy_dumpphy():Virtual: 7ffb50200000 <--> Phy: > 2a2600000 512 PFNs, 2097152 bytes > > Without Huge pages > _ishmphy.c:290:_odp_ishmphy_dumpphy():Phy Dump: > _ishmphy.c:309:_odp_ishmphy_dumpphy():Virtual: 7fb812598000 <--> Phy: > 2a6c31000 1 PFNs, 4096 bytes > _ishmphy.c:309:_odp_ishmphy_dumpphy():Virtual: 7fb812599000 <--> Phy: > 2a6c30000 1 PFNs, 4096 bytes > _ishmphy.c:309:_odp_ishmphy_dumpphy():Virtual: 7fb81259a000 <--> Phy: > 287bfd000 1 PFNs, 4096 bytes > _ishmphy.c:309:_odp_ishmphy_dumpphy():Virtual: 7fb81259b000 <--> Phy: > 287bfc000 1 PFNs, 4096 bytes > _ishmphy.c:309:_odp_ishmphy_dumpphy():Virtual: 7fb81259c000 <--> Phy: > 285a79000 1 PFNs, 4096 bytes > _ishmphy.c:309:_odp_ishmphy_dumpphy():Virtual: 7fb81259d000 <--> Phy: > 285a78000 1 PFNs, 4096 bytes > _ishmphy.c:309:_odp_ishmphy_dumpphy():Virtual: 7fb81259e000 <--> Phy: > 3a8a11000 1 PFNs, 4096 bytes > ... > > > Christophe Milard (2): > linux-gen: _ishmphy: adding function for physical address query > linux-gen: _ishmphy: adding debug function for pysical address mapping > > platform/linux-generic/_ishmphy.c | 140 > +++++++++++++++++++++ > platform/linux-generic/include/_ishmphy_internal.h | 19 +++ > 2 files changed, 159 insertions(+) > > -- > 2.7.4 > >
