On Thu, May 17, 2012 at 03:31:50PM +0200, Arne Jansen wrote:

The assumption here is that if err == 0, eb is always != NULL. There's
even a tiny comment above the function stating this:

      107              /* in case of err, eb might be NULL */

Ah, right.  I missed the comment.

This code changes significantly with the patch

btrfs: extend readahead interface

Where it is written in a more obvious way.

Cool.

regards,
dan carpenter