First Last Prev Next    No search results available      Search page      Enter new bug
Bug#: 76181
Alias:
Product:
Component:
Status: RESOLVED
Resolution: FIXED
Assigned To: Mobile Herd <mobile@gentoo.org>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: Vincenzo Romano <vincenzo.romano@gmail.com>
Add CC:
CC:
URL:
Summary:
Status Whiteboard:
Keywords:

Filename Description Type Creator Created Size Actions
emerge--info emerge --info output text/plain Vincenzo Romano 2005-01-08 09:28 0000 2.01 KB Details
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 76181 depends on: Show dependency tree
Bug 76181 blocks:
Votes: 0    Show votes for this bug    Vote for this bug

Additional Comments: (this is where you put emerge --info)


Not eligible to see or edit group visibility for this bug.






View Bug Activity   |   Format For Printing   |   XML   |   Clone This Bug


Description:   Opened: 2004-12-30 15:37 0000
Even running "radeontool --help" gives back the "mmap error" message.

Reproducible: Always
Steps to Reproduce:
1. radeontool --help


Actual Results:  
mmap error



Expected Results:  
The help about command line arguments.

The following patch fixes the problem to me:
--- radeontool.c        2004-12-31 00:32:43.476822439 +0100
+++ radeontool.c.orig   2004-12-31 00:31:55.265222292 +0100
@@ -107,8 +107,7 @@
         mem_fd,
         base
     );
-    if( (void*) device_mem == MAP_FAILED ) {
-    //if ((long)device_mem < 0) {
+    if ((long)device_mem < 0) {
         if(debug)
             fprintf(stderr,"mmap returned %d\n",(int)device_mem);
         fatal("mmap error \n");

------- Comment #1 From Vincenzo Romano 2004-12-30 15:39:16 0000 -------
Ooops: I reversed the filenames in the diff  command!
--- radeontool.c.orig   2004-12-31 00:31:55.265222292 +0100
+++ radeontool.c        2004-12-31 00:32:43.476822439 +0100
@@ -107,7 +107,8 @@
         mem_fd,
         base
     );
-    if ((long)device_mem < 0) {
+    if( (void*) device_mem == MAP_FAILED ) {
+    //if ((long)device_mem < 0) {
         if(debug)
             fprintf(stderr,"mmap returned %d\n",(int)device_mem);
         fatal("mmap error \n");

Sorry for the inconvenience!

------- Comment #2 From Henrik Brix Andersen 2005-01-02 14:12:17 0000 -------
Please attach the output of 'emerge --info'.

------- Comment #3 From Henrik Brix Andersen 2005-01-04 03:45:51 0000 -------
This bug report is awaiting feedback from the reporter.

------- Comment #4 From Vincenzo Romano 2005-01-08 09:28:41 0000 -------
Created an attachment (id=47946) [edit]
emerge --info output

Sorry for the delay;: I was far from my email.
In any case it seems more a thinko in the mmap() return check than an
environment issue.

------- Comment #5 From Henrik Brix Andersen 2005-01-08 09:42:28 0000 -------
Thank you for reporting back.

------- Comment #6 From Henrik Brix Andersen 2005-01-20 04:23:00 0000 -------
Thank you for reporting this. I have fixed this in app-laptop/radeontool-1.5-r1
and submitted the patch upstream.

First Last Prev Next    No search results available      Search page      Enter new bug