Re: Remove TO DO in jfs_xtree.c

2014-12-29 Thread Dave Kleikamp
On 12/29/2014 09:48 PM, ty...@mit.edu wrote: > On Mon, Dec 29, 2014 at 04:13:37PM -0600, Dave Kleikamp wrote: >> On 12/27/2014 06:58 PM, nick wrote: >>> Greetings Dave, >>> I am wondering why there is a TO DO above this code: >>> * ToDo: tlocks should be on doubly-linked list, so we can >>

Re: Remove TO DO in jfs_xtree.c

2014-12-29 Thread tytso
On Mon, Dec 29, 2014 at 04:13:37PM -0600, Dave Kleikamp wrote: > On 12/27/2014 06:58 PM, nick wrote: > > Greetings Dave, > > I am wondering why there is a TO DO above this code: > > * ToDo: tlocks should be on doubly-linked list, so we can > > * quickly remove it and add it to the

Re: Remove TO DO in jfs_xtree.c

2014-12-29 Thread Dave Kleikamp
On 12/27/2014 06:58 PM, nick wrote: > Greetings Dave, > I am wondering why there is a TO DO above this code: > * ToDo: tlocks should be on doubly-linked list, so we can > * quickly remove it and add it to the end. I'm sure the idea was to avoid the for loop needed to find the prev