Re: missing madvise functionality

[Rik van Riel]

Jakub Jelinek wrote:

> +	/* FIXME: POSIX says that MADV_DONTNEED cannot throw away data. */
>  	case MADV_DONTNEED:
> +	case MADV_FREE:
>  		error = madvise_dontneed(vma, prev, start, end);
>  		break;
>  
> I think you should only use the new behavior for madvise MADV_FREE, not for
> MADV_DONTNEED. 

I will.  However, we need to double-use MADV_DONTNEED in this
patch for now, so Ulrich's test glibc can be used easily :)

--
Politics is the struggle between those who want to make their country
the best in the world, and those who believe it already is.  Each group
calls the other unpatriotic.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/