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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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


--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/dm-devel

[Index of Archives]     [DM Crypt]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Packaging]     [Fedora SELinux]     [Yosemite Discussion]     [KDE Users]     [Fedora Docs]

  Powered by Linux