2016-07-22 17:02, Sergio Gonzalez Monroy: > On 22/07/2016 16:24, Thomas Monjalon wrote: > > 2016-07-22 16:33, Michal Jastrzebski: > >> v2: > >> -moved close(fd) just after read. > >> -when read() from fd we expect 8 bytes, so PFN_MASK_SIZE macro > >> was introduced instead sizeof(uint64_t). > >> -removed errno print when read returns less than 8 bytes > > Looks better. [...] > > Better title to explain the issue: > > mem: fix check of physical address retrieval [...] > >> + retval = read(fd, &page, sizeof(uint64_t)); > > We could use PFN_MASK_SIZE here [...] > > useless whitespace [...] > > If you and Sergio agree, I can make the minor changes before applying. > > Go for it!
Applied with above changes, thanks
