Re: [PATCH v4 10/11] 9pfs: T_readdir latency optimization

2020-03-17 Thread Christian Schoenebeck
On Dienstag, 17. März 2020 15:14:23 CET Greg Kurz wrote: > > > > I think the cause of disagreements we have are solely our use cases of > > > > 9pfs: your personal use case does not seem to require any performance > > > > considerations or multi-user aspects, whereas my use case requires at > > > >

Re: [PATCH v4 10/11] 9pfs: T_readdir latency optimization

2020-03-17 Thread Greg Kurz
On Wed, 11 Mar 2020 20:54:58 +0100 Christian Schoenebeck wrote: > On Mittwoch, 11. März 2020 17:14:08 CET Greg Kurz wrote: > > On Wed, 11 Mar 2020 02:18:04 +0100 > > > > Christian Schoenebeck wrote: > > > On Dienstag, 10. März 2020 19:33:36 CET Greg Kurz wrote: > > > > > This patch is also too

Re: [PATCH v4 10/11] 9pfs: T_readdir latency optimization

2020-03-11 Thread Christian Schoenebeck
On Mittwoch, 11. März 2020 17:14:08 CET Greg Kurz wrote: > On Wed, 11 Mar 2020 02:18:04 +0100 > > Christian Schoenebeck wrote: > > On Dienstag, 10. März 2020 19:33:36 CET Greg Kurz wrote: > > > > This patch is also too big for my preference, but I don't see a viable > > > > way > > > > to split i

Re: [PATCH v4 10/11] 9pfs: T_readdir latency optimization

2020-03-11 Thread Greg Kurz
On Wed, 11 Mar 2020 02:18:04 +0100 Christian Schoenebeck wrote: > On Dienstag, 10. März 2020 19:33:36 CET Greg Kurz wrote: > > > This patch is also too big for my preference, but I don't see a viable way > > > to split it further into separate patches. I already separated all the > > > patches I

Re: [PATCH v4 10/11] 9pfs: T_readdir latency optimization

2020-03-10 Thread Christian Schoenebeck
On Dienstag, 10. März 2020 19:33:36 CET Greg Kurz wrote: > > This patch is also too big for my preference, but I don't see a viable way > > to split it further into separate patches. I already separated all the > > patches I could. If you have suggestions, very much appreciated! > > Well, the patc

Re: [PATCH v4 10/11] 9pfs: T_readdir latency optimization

2020-03-10 Thread Greg Kurz
On Tue, 10 Mar 2020 16:10:41 +0100 Christian Schoenebeck wrote: > On Montag, 9. März 2020 16:42:59 CET Greg Kurz wrote: > > The main issue I see with this patch is that it is too big. At > > least from my POV, with the little time I have for 9p, that's > > the first thing that pushes me back... S

Re: [PATCH v4 10/11] 9pfs: T_readdir latency optimization

2020-03-10 Thread Christian Schoenebeck
On Montag, 9. März 2020 16:42:59 CET Greg Kurz wrote: > The main issue I see with this patch is that it is too big. At > least from my POV, with the little time I have for 9p, that's > the first thing that pushes me back... So if you could split > this patch down to some reviewable chunks, this may

Re: [PATCH v4 10/11] 9pfs: T_readdir latency optimization

2020-03-09 Thread Greg Kurz
On Mon, 09 Mar 2020 15:09:21 +0100 Christian Schoenebeck wrote: > On Dienstag, 21. Januar 2020 01:30:10 CET Christian Schoenebeck wrote: > > Make top half really top half and bottom half really bottom half: > > > > Each T_readdir request handling is hopping between threads (main > > I/O thread a

Re: [PATCH v4 10/11] 9pfs: T_readdir latency optimization

2020-03-09 Thread Christian Schoenebeck
On Dienstag, 21. Januar 2020 01:30:10 CET Christian Schoenebeck wrote: > Make top half really top half and bottom half really bottom half: > > Each T_readdir request handling is hopping between threads (main > I/O thread and background I/O driver threads) several times for > every individual direc

Re: [PATCH v4 10/11] 9pfs: T_readdir latency optimization

2020-01-23 Thread Christian Schoenebeck
On Donnerstag, 23. Januar 2020 12:33:42 CET Greg Kurz wrote: > On Tue, 21 Jan 2020 01:30:10 +0100 > > Christian Schoenebeck wrote: > > Make top half really top half and bottom half really bottom half: > > > > Each T_readdir request handling is hopping between threads (main > > I/O thread and bac

Re: [PATCH v4 10/11] 9pfs: T_readdir latency optimization

2020-01-23 Thread Greg Kurz
On Tue, 21 Jan 2020 01:30:10 +0100 Christian Schoenebeck wrote: > Make top half really top half and bottom half really bottom half: > > Each T_readdir request handling is hopping between threads (main > I/O thread and background I/O driver threads) several times for > every individual directory

[PATCH v4 10/11] 9pfs: T_readdir latency optimization

2020-01-20 Thread Christian Schoenebeck
Make top half really top half and bottom half really bottom half: Each T_readdir request handling is hopping between threads (main I/O thread and background I/O driver threads) several times for every individual directory entry, which sums up to huge latencies for handling just a single T_readdir