From: "Mike (mwester)" <mwester@dls.net>
To: openembedded-devel@openembedded.org
Subject: Re: [RFC] update opkg* SRCREV to 201
Date: Thu, 26 Feb 2009 14:25:10 -0600 [thread overview]
Message-ID: <49A6FAA6.2010104@dls.net> (raw)
In-Reply-To: <565965130902261031h3d21c452w51b8d1217cebde54@mail.gmail.com>
I'm glad to hear that there's work on some of these issues, as outlined
below!
-Mike
Tick wrote:
> Hi,
> Koen and Graeme Thanks.
> To my understanding, the fix from R197 to R201 are some easy defect fixing.
> Fixing some typos, memory leak, and giving initial value some variable.
> There were no big change in these versions.
> Therefore I didn't bump up versions here.
> There are still some issues in my mind and I am still finding large
> chunk of time to deal with them.
> 1. the algorithm calculate the dependency is not very efficient.
> (Much faster than R180 now, but I know it can be faster) Because of
> there were some duplicated code and ill structured code from ipkg, I
> need to refactorying them first before changing the algorithm.
> 2. memory leak problem, ipkg were designed for running once and
> exit, and therefore there were many place are not took carefully.
> Thomas and I already fixed a lot, but not all of them. Most of the
> largest memory leaks were fixed, however there were still some small
> leaks need to be find out. Especially libopkg is used as library, this
> should be take care well. (If anyone find a leak, please report or
> help. thanks)
> 3. memory hungry issue, opkg will eat a large amount of memory if
> you have many packages metadata. This may cause problem, however there
> is dilemma. Storing metadata will make opkg faster, but may cause very
> limited device run out of memory. Saving memory will cause cpu busying
> malloc and free all the time, and it will be very slow. Though the
> algorithm now seems to be okay, for device having so many packages
> should be powerful enough. XD I am considering adding a flag decide
> which policy (fast or save memory) to use while building opkg.
> 4. buffer over issue, some of the code did not check the boundary
> well. Actually R197 is fixing an buffer overflow issue, in which not
> be found for a long long time until it cause problem (on x86_64 arch
> only). >_<
> Many code needs to be reviewed/tested again and again to make sure the
> quality is good enough, and it's need your reports, help and patches.
> If I break anything, I am sorry. And please mail to
> opkg-devel@googlegroups.com poking me, creating a ticket with
> backtrace and some analysis to
> http://code.google.com/p/opkg/issues/list is good, sending a patch is
> even welcome. When I realize if it cause serious problem to you, I
> will try to fix it asap. I will be glad if every opkg user does not
> aware it's existence, except the developers.
> Thank a lot.
>
> Cheers,
> Tick
>
>
> 2009/2/27 Graeme Gregory <dp@xora.org.uk>:
>> Koen Kooi wrote:
>>> On 25-02-09 21:46, Koen Kooi wrote:
>>>> Hi,
>>>>
>>>> I was getting seriously annoyed with the useless error codes opkg was
>>>> giving me and Tick pointed me to:
>>>>
>>>> http://code.google.com/p/opkg/issues/detail?id=6&can=1
>>>>
>>>> We're at 197 now in OE (with r201 applied as patch). I'd like to get
>>>> r199 and r200 in as well, and since I'm lazy I though we'd get r198 in
>>>> as well, which means instead of adding a few patches I can just bump up
>>>> SRCREV to 201.
>>>>
>>>> What do you think?
>>> Noone is thinking anything?
>>>
>> Sorry been AFK, my opinion of these things is always no-one will test
>> until stuff is committed anyway. So you might as well go ahead! Without
>> the bug reports tick won't be able to improve opkg.
>>
>> Graeme (XorA)
>>
>>
>> _______________________________________________
>> Openembedded-devel mailing list
>> Openembedded-devel@lists.openembedded.org
>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>>
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>
next prev parent reply other threads:[~2009-02-26 20:29 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-02-25 20:46 [RFC] update opkg* SRCREV to 201 Koen Kooi
2009-02-26 14:55 ` Koen Kooi
2009-02-26 15:40 ` Tom Rini
2009-02-27 7:20 ` Koen Kooi
2009-02-27 19:56 ` Tom Rini
2009-02-26 15:46 ` Philip Balister
2009-02-26 19:09 ` Mike (mwester)
2009-02-26 16:13 ` Graeme Gregory
2009-02-26 18:31 ` Tick
2009-02-26 19:41 ` Michael 'Mickey' Lauer
2009-02-26 20:25 ` Mike (mwester) [this message]
2009-02-27 19:31 ` Koen Kooi
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=49A6FAA6.2010104@dls.net \
--to=mwester@dls.net \
--cc=openembedded-devel@lists.openembedded.org \
--cc=openembedded-devel@openembedded.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.