578618 - Make /sys/kernel/high_watermark check MAEMO=5 only

Status: RESOLVED FIXED

(Core :: XPCOM, defect)

Description

[Oleg Romashin (:romaxa)]

Attachment: Make maemo5 only mem check

we are checking special file and not using any system API for handling low memory on maemo... and that is common now for all maemo platforms...

I think we should make current implementation MAEMO 5 only,

and then find proper low mem API for maemo/meego platform

Comment 1

[Doug Turner (:dougt)]

Comment on attachment 457275 [details] [diff] [review]
Make maemo5 only mem check

what about MOZ_PLATFORM_MAEMO == 4

Comment 2

[Oleg Romashin (:romaxa)]

> what about MOZ_PLATFORM_MAEMO == 4
Will (MOZ_PLATFORM_MAEMO <= 5) work?

Comment 3

[Oleg Romashin (:romaxa)]

Attachment: mem <= 5

Comment 4

[Doug Turner (:dougt)]

Comment on attachment 459577 [details] [diff] [review]
mem <= 5

we are about to remove this code very soon and replacing it with a new system.

Comment 5

[Doug Turner (:dougt)]

Comment on attachment 459577 [details] [diff] [review]
mem <= 5

gnu doesn't support comparisons.  :(

Comment 6

[Oleg Romashin (:romaxa)]

Attachment: ==5 || ==4

Comment 7

[Oleg Romashin (:romaxa)]

> 
> we are about to remove this code very soon and replacing it with a new system.

my goal here is get rid of annoying NS_ERRORS on maemo platform > 5

Comment 8

[Doug Turner (:dougt)]

http://hg.mozilla.org/mozilla-central/rev/cb95b8d67bdd