On Thu, 27 Jul 2006 11:00:25 +0200, Franck Bui-Huu <vagabon.xyz@xxxxxxxxx> wrote: > I'm suprised that sparsemem code doens't check for holes inside > sections. I would feel really more confortable to use sparsemem if a > check like the following patch exists. We could safely use pfn_valid() > in _any_ cases and if holes exist inside sections then the user have > to fix up its section sizes. > > what do you think ? I think holes inside a section is not illegal, just a bit ineffective. So I feel your patch is too strict. --- Atsushi Nemoto