Messages in this thread | | | Date | Mon, 6 Oct 2008 09:00:14 -0400 (EDT) | From | Mikulas Patocka <> | Subject | Re: [PATCH 2/3] Fix fsync livelock |
| |
On Sun, 5 Oct 2008, Arjan van de Ven wrote:
> On Sun, 5 Oct 2008 23:30:51 -0400 (EDT > > The point is that many fsync()s may run in parallel and you have just > > one inode and just one chain. And if you add two-word list_head to a > > page, to link it on this list, many developers will hate it for > > increasing its size. > > why to a page? > a list head in the inode and chain up the bios....
And if you want to wait for a bio submitted by a different process? There's no way you can find the bio from the page.
> or not make an actual list but just a "is the previous one done" thing > it's not all that hard to get something that works on a per inode basis, > that gives "wait for all io upto this one".
So code it :)
Mikulas
| |