lkml.org 
[lkml]   [2008]   [Aug]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: kernel BUG at lib/radix-tree.c:473!
Date
Hi,

Jeremy Fitzhardinge <jeremy@goop.org> writes:

> Hugh Dickins wrote:
>> As you can see, I'm still groping towards the right answer.
>> The driver probably needs to provide its own backing_dev_info
>> (or point to a suitable default), and its own address_space_ops,
>> and perhaps more (there should be examples elsewhere). But whether
>> it is actually wrong, or whether I was wrong to mess it up, I've
>> not yet decided.
>>
>
> My understanding is that the driver is doing something a bit clever:
> it uses the page dirty flags to determine which parts of the
> framebuffer have been written to, and uses that information to
> minimize the amount of stuff that needs to be copied out. The writes
> to the pages are not expected to generate actual page faults.
>
> But I haven't really looked at it closely, and I'm not at all familiar
> with the vm at this layer. I'm not sure how it actually allocates the
> framebuffer memory for example (vmalloc? incrementally on faults?).
> I'm hoping Markus will leap in, since wrote this stuff. Or, gasp,
> I'll read the code myself.
>
>> An additional useful input would be: what happens if you replace
>> that /dev/fb0 by a symlink /dev/fb0 pointing to an fb0 device node in
>> one of your disk filesystems? I rather expect that to cause the same
>> trouble, which would argue that the driver is wrong and shmem right.
>>
>
> I don't follow. Do you mean make /dev/fb0 a plain file on a
> filesystem? Or make it a disk device node? Something else?

Creating a device node on a different filesystem to see if the driver
only worked with the safe shmem set_page_dirty and now breaks due to
exposure to the generic version. Or if the driver works with the
generic version through other mappings and the shmem code screws it up
somewhere else.

Hannes


\
 
 \ /
  Last update: 2008-08-15 01:17    [from the cache]
©2003-2011 Jasper Spaans