public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Dave Jones <davej@redhat.com>
To: Marcelo Tosatti <marcelo.tosatti@cyclades.com>
Cc: Andreas Hartmann <andihartmann@01019freenet.de>,
	linux-kernel@vger.kernel.org, davej@codemonkey.org.uk
Subject: Re: 2.4.x oops with X
Date: Fri, 7 Jan 2005 14:53:44 -0500	[thread overview]
Message-ID: <20050107195343.GH22299@redhat.com> (raw)
In-Reply-To: <20050107170152.GI29176@logos.cnet>

On Fri, Jan 07, 2005 at 03:01:52PM -0200, Marcelo Tosatti wrote:

 > > Well, what do you mean with "disable AGP"? I can't disable it in the BIOS.
 > > I disabled DRI in the XF86Config-file. agpgart and r128 haven't been
 > > loaded (they are built as modules). The behaviour of the X-starting
 > > doesn't change and it's always the same: 
 > 
 > I meant not loading the agpgart/r128 modules, but it seems they are loaded 
 > on demand and X actually can't work without them.

If he disabled DRI in the X config file, they shouldn't be getting
loaded, so I'm curious why agp is in the picture..

 > Well the problem is the core dumping code (elf_core_dump function) is trying to write
 > your ATI card memory to disk, which is wrong. 
 > 
 > agp's mmap() method does not mark the memory region it creates as VM_IO to 
 > indicate its a device memory mapped region, and it should AFAICS.
 > 
 > The following corrects the situation and should stop the BUG() from happening, 
 > however the SIGSEGV which X is receiving seems to be a different thing.
 > 
 > Please try the following patch
 > 
 > Davej?
 > 
 > 
 > --- linux-2.4.28/drivers/char/agp/agpgart_fe.c.orig	2005-01-07 16:42:24.732957320 -0200
 > +++ linux-2.4.28/drivers/char/agp/agpgart_fe.c	2005-01-07 16:42:30.329106576 -0200
 > @@ -651,6 +651,7 @@
 >  			unlock_kernel();
 >  			return -EAGAIN;
 >  		}
 > +		vma->vm_flags |= VM_IO;
 >  		AGP_UNLOCK();
 >  		unlock_kernel();
 >  		return 0;
 > @@ -667,6 +668,7 @@
 >  			unlock_kernel();
 >  			return -EAGAIN;
 >  		}
 > +		vma->vm_flags |= VM_IO;
 >  		AGP_UNLOCK();
 >  		unlock_kernel();
 >  		return 0;

Looks ok on a first glance.

		Dave


  reply	other threads:[~2005-01-07 19:57 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <fa.m16skii.8mkd12@ifi.uio.no>
     [not found] ` <fa.f3n91fn.b42ahv@ifi.uio.no>
2005-01-07 18:13   ` 2.4.x oops with X Andreas Hartmann
2005-01-07 17:01     ` Marcelo Tosatti
2005-01-07 19:53       ` Dave Jones [this message]
     [not found] <fa.gv4g3v7.1ng0thr@ifi.uio.no>
     [not found] ` <fa.kmfmtrp.1a16aaf@ifi.uio.no>
2005-01-08 16:18   ` Andreas Hartmann
2005-01-13 11:30     ` Marcelo Tosatti
2005-02-06  6:35   ` Andreas Hartmann
     [not found] <fa.kuv2u3i.hhma1k@ifi.uio.no>
     [not found] ` <fa.f87d0no.fk6a9u@ifi.uio.no>
2005-01-07 23:21   ` Andreas Hartmann
     [not found]     ` <20050108014844.GB3210@redhat.com>
     [not found]       ` <41DF98F4.5050805@pD9F8750A.dip0.t-ipconnect.de>
2005-01-10 10:58         ` Marcelo Tosatti
2005-01-07  9:03 Andreas Hartmann
2005-01-07 11:30 ` Marcelo Tosatti

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=20050107195343.GH22299@redhat.com \
    --to=davej@redhat.com \
    --cc=andihartmann@01019freenet.de \
    --cc=davej@codemonkey.org.uk \
    --cc=linux-kernel@vger.kernel.org \
    --cc=marcelo.tosatti@cyclades.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox