Re: [PATCH] Check region size during dirty log creation

[Heinz Mauelshagen <heinzm@redhat.com>]

Am Mittwoch, den 29.10.2008, 13:56 +0000 schrieb Alasdair G Kergon:
> On Wed, Oct 29, 2008 at 02:02:08PM +0100, Heinz Mauelshagen wrote:
> > This patch adds checks to the dirty log creation for region size to be
> > larger than 2 sectors and to be a power of 2.
>  
> Interesting we didn't validate that, but the userspace validation is
> tighter.
> 
>         if (lp->region_size % (pagesize >> SECTOR_SHIFT)) {
>                 log_error("Region size (%" PRIu32 ") must be a multiple of "
>                           "machine memory page size (%d)",
>                           lp->region_size, pagesize >> SECTOR_SHIFT);
> 
> The validation should be consistent between userspace and kernel.
> Alasdair

It should be sufficiently restrictive in the kernel to prevent
programming errors causing oopses. It can be called via the kernel
interface by any application anyway, so doesn't need to enforce our
uspace restrictions (think direct IOCTL calls).

Heinz