For v3 of the patchset, I was also considering to add a note to the kernel-doc comments for dma_pool_create() to use dma_alloc_coherent() directly instead of a dma pool if the driver intends to allow userspace to mmap() the returned pages, due to the new use of the _mapcount union in struct page. Would you consider that useful information or pointless trivia?
