Changeset 77421 in vbox for trunk/src

Timestamp:

Feb 22, 2019 2:27:06 AM (6 years ago)

Author:

vboxsync

Message:

linux/vboxsf: Better use VbglR0SfHostReqReadPgLst in sf_readpage. FsPerf indicates that it's lightly faster here. bugref:9172

File:

• trunk/src/VBox/Additions/linux/sharedfolders/regops.c (modified) (2 diffs)

trunk/src/VBox/Additions/linux/sharedfolders/regops.c

@@ -1110 +1110 @@
                     uint8_t             *pbMapped = (g_fHostFeatures & VMMDEV_HVF_HGCM_CONTIGUOUS_PAGE_LIST)
                                                   ? NULL : (uint8_t *)kmap(page);
-                    int vrc = VbglR0SfHostReqReadContig(sf_g->map.root,
-                                                        pReq,
-                                                        sf_r->handle,
-                                                        (uint64_t)page->index << PAGE_SHIFT,
-                                                        PAGE_SIZE,
-                                                        pbMapped,
-                                                        page_to_phys(page));
-                    uint32_t cbRead = pReq->Parms.cb32Read.u.value32;
+                    uint32_t             cbRead;
+                    int                  vrc;
+
+                    pReq->PgLst.offFirstPage = 0;
+                    pReq->PgLst.aPages[0]    = page_to_phys(page);
+                    vrc = VbglR0SfHostReqReadPgLst(sf_g->map.root,
+                                                   pReq,
+                                                   sf_r->handle,
+                                                   (uint64_t)page->index << PAGE_SHIFT,
+                                                   PAGE_SIZE,
+                                                   1 /*cPages*/);
+
+                    cbRead = pReq->Parms.cb32Read.u.value32;
                     AssertStmt(cbRead <= PAGE_SIZE, cbRead = PAGE_SIZE);
                     VbglR0PhysHeapFree(pReq);
@@ -1130 +1135 @@
                                     /** @todo truncate the inode file size? */
                             }
+
                             flush_dcache_page(page);
                             SetPageUptodate(page);