[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

Re: [tor-bugs] #27629 [Core Tor/Tor]: add len argument to consensus parsing functions



#27629: add len argument to consensus parsing functions
--------------------------+------------------------------------
 Reporter:  cyberpunks    |          Owner:  (none)
     Type:  enhancement   |         Status:  needs_review
 Priority:  Medium        |      Milestone:  Tor: 0.3.5.x-final
Component:  Core Tor/Tor  |        Version:
 Severity:  Normal        |     Resolution:
 Keywords:                |  Actual Points:
Parent ID:  #27244        |         Points:
 Reviewer:                |        Sponsor:  Sponsor8
--------------------------+------------------------------------

Comment (by teor):

 Replying to [comment:7 cyberpunks]:
 > Replying to [comment:5 teor]:
 > > The rest of the branch makes Tor use less RAM
 >
 > How does it do that? Well, there's removal of an allocation in the fuzz
 test. But no reduction in any actually running Tor client. This branch is
 just prerequisite refactoring.
 >
 > There is another branch that does the actual switch to using mmap, but
 it's based on top of #27247 due to conflicts, so waiting on that to be
 merged.

 Oh, right. I looked at all the commits, including the mmap ones that are
 in parselen1-on-ticket27247.

 So for parselen1, it's just refactoring. For the mmap branch, it's an
 enhancement.

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/27629#comment:8>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs