980914 - [tarako]gallery image editor should support EXIF data for shrinking 2MB RAM

Status: RESOLVED DUPLICATE of bug 798684

(Firefox OS Graveyard :: Gaia::Gallery, defect)

Description

[James Zhang (Spreadtrum)]

We use about 3M kernel ion memory for camera capture rotation because gallery app don't support EXIF data.

Comment 1

[[:fabrice] Fabrice Desré]

David, what is our status with exif support in the gallery app?

Comment 2

[James Zhang (Spreadtrum)]

Gallery support exif but gallery image editor don't support it. Without 3M camera rotation memory, contact apps add iamge from camera, the image rotate 90 degree，it cause bad user experience. 

But the 3M memory is very very import for tarako.

Comment 3

[thomas tsai]

Hi James: Is this 3MB always allocated in kernek space or Is it allocated only when camera is used?

Comment 4

[James Zhang (Spreadtrum)]

(In reply to thomas tsai from comment #3)
> Hi James: Is this 3MB always allocated in kernek space or Is it allocated
> only when camera is used?

No, it's ion reserved memory.

Comment 5

[Joe Cheng [:jcheng] (please needinfo)]

ni? Thomas to confirm with partner further

Comment 6

[James Zhang (Spreadtrum)]

(In reply to James Zhang from comment #4)
> (In reply to thomas tsai from comment #3)
> > Hi James: Is this 3MB always allocated in kernek space or Is it allocated
> > only when camera is used?
> 
> No, it's ion reserved memory.

It's always allocated in kernek space.

Comment 7

[Joe Cheng [:jcheng] (please needinfo)]

Triage: 1.3T+ for 3MB of memory saving

ni? Hema
it looks like if gallery image editor supports EXIF, we can save 3MB of RAM
Can you look into supporting this? Thanks

Comment 8

[James Zhang (Spreadtrum)]

Spreadtrum camera owner shrink some ratation memory on our side. 
In the latest verion, if gallery image edtor supports EXIF, we can save 2MB RAM.

Comment 9

[David Flanagan [:djf]]

I thought there was a way to rotate jpeg images without loss and without decoding them.  I think that the chipset ought to be able to rotate using only the memory required to hold the JPEG file.

In any case, we do have a patch that didn't quite make the 1.3 release that is just about ready to go.  See bug 798684.

Comment 10

[Hema Koka [:hema]]

Punam is currently investigating on whats involved in getting the fix for 798684 in. We will update with more info tomorrow.

We would possibly also need help from someone who has tarako for testing the patch and ensuring it works...

thanks
hema

Comment 11

[Joe Cheng [:jcheng] (please needinfo)]

it seems like Punam is working on this bug, can Punam take this bug?
Yes, Taipei can assist with testing the patch when its available

Comment 12

[Evelyn Hung]

I think this bug is a dup of 798684. Can we mark that?

Comment 13

[Gregor Wagner [:gwagner]]

Should we mark this bug as a dup of 798684 and mark 798684 as 1.3T+?

Comment 15

[Marvin Khoo [:Marvin_Khoo]]

The 980914 is DUP and will be tracked in Bug 798684 (1.3T+)

Comment 16

[Steven Yang [:styang]]

tracked in 798684