Andrea,
I'll try this. I forgot about the mm_end_io field for asynch I/O. THis
is called "natural selection at work." :)
Thanks,
Jeff
Andrea Arcangeli wrote:
>
> On Mon, 21 Feb 2000, Jeff V. Merkey wrote:
>
> >//
> >// THIS CODE CRASHES WITH AN OOPS MESSAGE UNDER 2.2.14 (and all 2.2.X
> >kernels)
> >//
> [..]
> > bh[i]->b_this_page = bh[(i + 1) % sectors]; // create circular
> >list
> > bh[i]->b_state = 0;
> > bh[i]->b_next_free = NULL;
> > bh[i]->b_count = 0;
> > bh[i]->b_size = 512;
> > bh[i]->b_data = (char *)&Sector[i * bps];
> > bh[i]->b_list = BUF_CLEAN;
> > bh[i]->b_dev = (int)NWDisk->PhysicalDiskHandle;
> > bh[i]->b_blocknr = (StartingLBA + i);
> > bh[i]->b_count = 1;
> > bh[i]->b_flushtime = 0;
> >
> > clear_bit(BH_Uptodate, &bh[i]->b_state);
>
> 1) You forgot to set the bh[i]->b_end_io callback, that's why it
> crashes immediatly within the IDE irq handler (probably killing the
> idle task or similar).
>
> 2) It seems you don't need to set the b_this_page at all (unless you need
> it in the end_io callback). You can set it to zero to trap accesses
> to it. Also if relly you want to set it you'd better set
> bh[i]->b_this_page to bh itself since your b_data storage memory is
> 512byte wide thus is sounds more correct.
>
> 3) Note that WRITEA is obsoleted in 2.3.x.
>
> Andrea
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.rutgers.edu
Please read the FAQ at http://www.tux.org/lkml/
This archive was generated by hypermail 2b29 : Wed Feb 23 2000 - 21:00:31 EST