[Bug 1001650] New: warning in kernel test kcache2.c

[Bug 1001650] New: warning in kernel test kcache2.c

[bugzilla-daemon]

Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001650

           Summary: warning in kernel test kcache2.c
           Product: eCos
           Version: CVS
          Platform: All
        OS/Version: Other
            Status: UNCONFIRMED
          Severity: enhancement
          Priority: low
         Component: Kernel
        AssignedTo: unassigned@bugs.ecos.sourceware.org
        ReportedBy: bernd.edlinger@hotmail.de
                CC: ecos-bugs@ecos.sourceware.org
             Class: Advice Request


Created an attachment (id=1896)
 --> (http://bugs.ecos.sourceware.org/attachment.cgi?id=1896)
include a header for the missing define.

Hello,

due to the change by vae at Sun May 27 18:34:02 2012 UTC,
there is now a compiler warning in the file:
ecos/packages/kernel/current/tests/kcache2.c

It says "use of undefined define CYGMEM_REGION_ram_SIZE".

This means that the test now always uses the small memory
configuration, which was not the intention of this change.

See the attached patch for a possible solution.

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

[Bug 1001650] warning in kernel test kcache2.c

[bugzilla-daemon]

Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001650

Ilija Kocho <ilijak@siva.com.mk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ilijak@siva.com.mk

--- Comment #1 from Ilija Kocho <ilijak@siva.com.mk> 2012-08-14 09:24:33 BST ---
Hi Bernd

Thank you for the catch.

(In reply to comment #0)
> Created an attachment (id=1896)
 --> (http://bugs.ecos.sourceware.org/attachment.cgi?id=1896) [details]
> include a header for the missing define.
> 
> Hello,
> 
> due to the change by vae at Sun May 27 18:34:02 2012 UTC,

Just a Bugzilla technical note.
It would be useful to put a ref to Bug 1001594 (Bugzilla will do the magic)
or reopen the Bug 1001594 instead of creating new one.

> there is now a compiler warning in the file:
> ecos/packages/kernel/current/tests/kcache2.c
> 
> It says "use of undefined define CYGMEM_REGION_ram_SIZE".
> 
> This means that the test now always uses the small memory
> configuration, which was not the intention of this change.
> 
> See the attached patch for a possible solution.

Thank you for the patch too.

Ilija

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

[Bug 1001650] warning in kernel test kcache2.c

[bugzilla-daemon]

Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001650

--- Comment #2 from Bernd Edlinger <bernd.edlinger@hotmail.de> 2012-08-14 10:14:25 BST ---
(In reply to comment #1)
Thanks for clarifying this Ilja,

I would have reopened it, but Bug #1001594 was not mentioned
anywhere in the changelog, therefore I just did not find it. Sorry.

Bernd.

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

[Bug 1001650] warning in kernel test kcache2.c

[bugzilla-daemon]

Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001650

Ilija Kocho <ilijak@siva.com.mk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |CURRENTRELEASE

--- Comment #3 from Ilija Kocho <ilijak@siva.com.mk> 2012-10-04 08:47:51 BST ---
Checked in. Thanks to Bernd for reporting and the patch.

Ilija

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

[Bug 1001650] warning in kernel test kcache2.c

[bugzilla-daemon]

Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001650

Ilija Kocho <ilijak@siva.com.mk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |CURRENTRELEASE

--- Comment #3 from Ilija Kocho <ilijak@siva.com.mk> 2012-10-04 08:47:51 BST ---
Checked in. Thanks to Bernd for reporting and the patch.

Ilija

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Bug 1001650] warning in kernel test kcache2.c

[bugzilla-daemon]

Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001650

--- Comment #2 from Bernd Edlinger <bernd.edlinger@hotmail.de> 2012-08-14 10:14:25 BST ---
(In reply to comment #1)
Thanks for clarifying this Ilja,

I would have reopened it, but Bug #1001594 was not mentioned
anywhere in the changelog, therefore I just did not find it. Sorry.

Bernd.

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Bug 1001650] warning in kernel test kcache2.c

[bugzilla-daemon]

Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001650

Ilija Kocho <ilijak@siva.com.mk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ilijak@siva.com.mk

--- Comment #1 from Ilija Kocho <ilijak@siva.com.mk> 2012-08-14 09:24:33 BST ---
Hi Bernd

Thank you for the catch.

(In reply to comment #0)
> Created an attachment (id=1896)
 --> (http://bugs.ecos.sourceware.org/attachment.cgi?id=1896) [details]
> include a header for the missing define.
> 
> Hello,
> 
> due to the change by vae at Sun May 27 18:34:02 2012 UTC,

Just a Bugzilla technical note.
It would be useful to put a ref to Bug 1001594 (Bugzilla will do the magic)
or reopen the Bug 1001594 instead of creating new one.

> there is now a compiler warning in the file:
> ecos/packages/kernel/current/tests/kcache2.c
> 
> It says "use of undefined define CYGMEM_REGION_ram_SIZE".
> 
> This means that the test now always uses the small memory
> configuration, which was not the intention of this change.
> 
> See the attached patch for a possible solution.

Thank you for the patch too.

Ilija

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.