Michal Piotrowski napisaƂ(a):
> On 19/03/07, Michal Piotrowski <[EMAIL PROTECTED]> wrote:
>> On 19/03/07, Pekka J Enberg <[EMAIL PROTECTED]> wrote:
>> > On Mon, 19 Mar 2007, Michal Piotrowski wrote:
>> > > Yes, revoke-special-mmap-handling.patch is bad.
>> >
>> > Aah, the VM_REVOKED flag stomps on VM_CAN_INVALIDATE in -mm. Changing
>> > VM_REVOKED to 0x10000000 should fix it.
> 
> Unfortunately it doesn't. 0x10000000 is used by VM_CAN_NONLINEAR.
> 0x20000000 should be a better value.

Here is a tested patch.

Regards,
Michal

-- 
Michal K. K. Piotrowski
LTG - Linux Testers Group (PL)
(http://www.stardust.webpages.pl/ltg/)
LTG - Linux Testers Group (EN)
(http://www.stardust.webpages.pl/linux_testers_group_en/)

Signed-off-by: Michal Piotrowski <[EMAIL PROTECTED]>
Signed-off-by: Pekka J Enberg <[EMAIL PROTECTED]>

--- linux-work1-clean/include/linux/mm.h        2007-03-19 18:52:28.000000000 
+0100
+++ linux-work1/include/linux/mm.h      2007-03-19 18:23:04.000000000 +0100
@@ -173,7 +173,6 @@ extern int do_mprotect(unsigned long sta
 #define VM_MAPPED_COPY 0x01000000      /* T if mapped copy of data (nommu 
mmap) */
 #define VM_INSERTPAGE  0x02000000      /* The vma has had "vm_insert_page()" 
done on it */
 #define VM_ALWAYSDUMP  0x04000000      /* Always include in core dumps */
-#define VM_REVOKED     0x08000000      /* Mapping has been revoked */
 
 #define VM_CAN_INVALIDATE 0x08000000   /* The mapping may be invalidated,
                                         * eg. truncate or invalidate_inode_*.
@@ -181,6 +180,7 @@ extern int do_mprotect(unsigned long sta
                                         * return with the page locked.
                                         */
 #define VM_CAN_NONLINEAR 0x10000000    /* Has ->fault & does nonlinear pages */
+#define VM_REVOKED     0x20000000      /* Mapping has been revoked */
 
 #ifndef VM_STACK_DEFAULT_FLAGS         /* arch can override this */
 #define VM_STACK_DEFAULT_FLAGS VM_DATA_DEFAULT_FLAGS
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to