[Ocfs2-devel] osb->curr_trans_id

Mark Fasheh mark.fasheh at oracle.com
Tue Feb 17 15:19:10 CST 2004


On Tue, Feb 17, 2004 at 02:46:38PM -0800, Villalovos, John L wrote:
> So I was looking at the TODO list and I see these two entries:
> 
> * get rid of osb->curr_trans_id as it was never used (always zero)
> 
> * get rid of all the:
>         changeSeqNum = osb->curr_trans_id;
>         DISK_LOCK_SEQNUM(fe) = changeSeqNum;
>   code as it is equally useless.
> 
> So I was going to tackle this but I was wondering if there was anything
> I should look out for.
Yes, we've held off on this because we're not completely sure that it's
useless (more like almost sure), so you can either wait until we're sure,
or if you want to tackle it, be prepared to prove that it is indeed,
completely useless :)

> >From my quick perusing of the code it seems fairly simple but I am
> probably missing something.
> 
> So before submitting a "dumb" patch I thought I see if there was any
> comments on this.
There's actually a long trail of code and values set from values set from
values (some of which eventually hit disk) to follow, and basically none of
us has had the chance to do so :)
	--Mark

--
Mark Fasheh
Software Developer, Oracle Corp
mark.fasheh at oracle.com


More information about the Ocfs2-devel mailing list