chance new effective permissions code to ease aggregation on demand( ie on changes). But still doing full aggregation on checks, so still heavy

4 files changed, 121 insertions, 149 deletions

diff --git a/OpenSim/Region/Framework/Interfaces/IEntityInventory.cs b/OpenSim/Region/Framework/Interfaces/IEntityInventory.cs
index 2af6ff0..e7c2428 100644
--- a/OpenSim/Region/Framework/Interfaces/IEntityInventory.cs
+++ b/OpenSim/Region/Framework/Interfaces/IEntityInventory.cs
@@ -278,10 +278,7 @@ namespace OpenSim.Region.Framework.Interfaces
         /// <param name="datastore"></param>
         void ProcessInventoryBackup(ISimulationDataService datastore);
 
-        void AggregateEveryOnePerms(ref uint current);
-        void AggregateGroupOrEveryonePerms(ref uint current);
-        void AggregateGroupPerms(ref uint current);
-        void AggregateOwnerPerms(ref uint current);
+        void AggregateInnerPerms(ref uint owner, ref uint group, ref uint everyone);
 
         uint MaskEffectivePermissions();
 
diff --git a/OpenSim/Region/Framework/Scenes/SceneObjectGroup.Inventory.cs b/OpenSim/Region/Framework/Scenes/SceneObjectGroup.Inventory.cs
index f44604b..7b9ea8b 100644
--- a/OpenSim/Region/Framework/Scenes/SceneObjectGroup.Inventory.cs
+++ b/OpenSim/Region/Framework/Scenes/SceneObjectGroup.Inventory.cs
@@ -259,41 +259,11 @@ namespace OpenSim.Region.Framework.Scenes
                 // this can't be done here but on every place where a change may happen (rez, (de)link, contents , perms,  etc)
                 // bc this is on heavy duty code paths
                 // but for now we need to test the concept
-                AggregateEveryOnePerms();
+                AggregateDeepPerms();
                 return m_EffectiveEveryOnePerms;
             }
         }
 
-        public void AggregateEveryOnePerms()
-        {
-            lock(PermissionsLock)
-            {
-                // get object everyone permissions
-                uint baseperms = RootPart.EveryoneMask & (uint)PermissionMask.All;
-
-                if(baseperms == 0)
-                {
-                    m_EffectiveEveryOnePerms = 0;
-                    return;
-                }
-                
-                uint current = baseperms;
-                SceneObjectPart[] parts = m_parts.GetArray();
-                for (int i = 0; i < parts.Length; i++)
-                {
-                    SceneObjectPart part = parts[i];
-                    part.Inventory.AggregateEveryOnePerms(ref current);
-                    if( current == 0)
-                        break;
-                }
-                // recover move
-                baseperms &= (uint)PermissionMask.Move;
-                current |= baseperms;
-                current &= (uint)PermissionMask.All;
-                m_EffectiveEveryOnePerms = current;
-            }
-        }
-
         private uint m_EffectiveGroupPerms;
         public uint EffectiveGroupPerms
         {
@@ -302,126 +272,136 @@ namespace OpenSim.Region.Framework.Scenes
                 // this can't be done here but on every place where a change may happen (rez, (de)link, contents , perms,  etc)
                 // bc this is on heavy duty code paths
                 // but for now we need to test the concept
-                AggregateGroupPerms();
+                AggregateDeepPerms();
                 return m_EffectiveGroupPerms;
             }
         }
 
-        public void AggregateGroupPerms()
+        public uint EffectiveGroupOrEveryOnePerms
         {
-            lock(PermissionsLock)
+            get
             {
-                // get object everyone permissions
-                uint baseperms = RootPart.GroupMask & (uint)PermissionMask.All;
-
-                if(baseperms == 0)
-                {
-                    m_EffectiveGroupPerms = 0;
-                    return;
-                }
-                
-                uint current = baseperms;
-                SceneObjectPart[] parts = m_parts.GetArray();
-                for (int i = 0; i < parts.Length; i++)
-                {
-                    SceneObjectPart part = parts[i];
-                    part.Inventory.AggregateGroupPerms(ref current);
-                    if( current == 0)
-                        break;
-                }
-                // recover modify and move
-                baseperms &= (uint)(PermissionMask.Move | PermissionMask.Modify );
-                current |= baseperms;
-                current &= (uint)PermissionMask.All;
-                m_EffectiveGroupPerms = current;
+                // this can't be done here but on every place where a change may happen (rez, (de)link, contents , perms,  etc)
+                // bc this is on heavy duty code paths
+                // but for now we need to test the concept
+                AggregateDeepPerms();
+                return m_EffectiveEveryOnePerms | m_EffectiveGroupPerms;
             }
         }
 
-        private uint m_EffectiveGroupOrEveryOnePerms;
-        public uint EffectiveGroupOrEveryOnePerms
+        private uint m_EffectiveOwnerPerms;
+        public uint EffectiveOwnerPerms
         {
             get
             {
                 // this can't be done here but on every place where a change may happen (rez, (de)link, contents , perms,  etc)
                 // bc this is on heavy duty code paths
                 // but for now we need to test the concept
-                AggregateGroupOrEveryOnePerms();
-                return m_EffectiveGroupOrEveryOnePerms;
+                AggregateDeepPerms();
+                return m_EffectiveOwnerPerms;
             }
         }
 
-        public void AggregateGroupOrEveryOnePerms()
+        // aggregates perms scanning parts and their contents
+        public void AggregateDeepPerms()
         {
             lock(PermissionsLock)
             {
-                // get object everyone permissions
-                uint baseperms = (RootPart.EveryoneMask | RootPart.GroupMask) & (uint)PermissionMask.All;
+                // aux
+                const uint allmask = (uint)PermissionMask.AllEffective;
+                const uint movemodmask = (uint)(PermissionMask.Move | PermissionMask.Modify);
+                const uint copytransfermast = (uint)(PermissionMask.Copy | PermissionMask.Transfer);
+
+                uint baseOwnerPerms = RootPart.OwnerMask;
+                uint owner = baseOwnerPerms;
+                uint baseGroupPerms = RootPart.GroupMask;
+                uint group = baseGroupPerms;
+                uint baseEveryonePerms = RootPart.EveryoneMask;
+                uint everyone = baseEveryonePerms;
 
-                if(baseperms == 0)
-                {
-                    m_EffectiveGroupOrEveryOnePerms = 0;
-                    return;
-                }
-                
-                uint current = baseperms;
                 SceneObjectPart[] parts = m_parts.GetArray();
                 for (int i = 0; i < parts.Length; i++)
                 {
                     SceneObjectPart part = parts[i];
-                    part.Inventory.AggregateGroupOrEveryonePerms(ref current);
-                    if( current == 0)
-                        break;
+                    part.AggregateInnerPerms();
+                    owner &= part.AggregatedInnerOwnerPerms; 
+                    group &= part.AggregatedInnerGroupPerms;
+                    everyone &= part.AggregatedInnerEveryonePerms;
                 }
                 // recover modify and move
-                baseperms &= (uint)(PermissionMask.Move | PermissionMask.Modify );
-                current |= baseperms;
-                current &= (uint)PermissionMask.All;
-                m_EffectiveGroupOrEveryOnePerms = current;
-            }
-        }
+                baseOwnerPerms &= movemodmask;
+                owner |= baseOwnerPerms;
+                if((owner & copytransfermast) == 0)
+                    owner |= (uint)PermissionMask.Transfer;
+                owner &= allmask;
+                m_EffectiveOwnerPerms = owner;
 
-        private uint m_EffectiveOwnerPerms;
-        public uint EffectiveOwnerPerms
-        {
-            get
-            {
-                // this can't be done here but on every place where a change may happen (rez, (de)link, contents , perms,  etc)
-                // bc this is on heavy duty code paths
-                // but for now we need to test the concept
-                AggregateOwnerPerms();
-                return m_EffectiveOwnerPerms;
+                // recover modify and move
+                baseGroupPerms &= movemodmask;
+                group |= baseGroupPerms;
+                if((group & copytransfermast) == 0)
+                    group |= (uint)PermissionMask.Transfer;
+                group &= allmask;
+                m_EffectiveGroupPerms = group;
+
+                // recover move
+                baseEveryonePerms &= (uint)PermissionMask.Move;
+                everyone |= baseEveryonePerms;
+                if((everyone & copytransfermast) == 0) // not much sense but as sl
+                    everyone |= (uint)PermissionMask.Transfer;
+                everyone &= allmask;
+                m_EffectiveEveryOnePerms = everyone;
             }
         }
 
-        public void AggregateOwnerPerms()
+        // aggregates perms scanning parts, assuming their contents was already aggregated
+        public void AggregatePerms()
         {
             lock(PermissionsLock)
             {
-                // get object everyone permissions
-                uint baseperms = RootPart.OwnerMask  & (uint)PermissionMask.All;
+                // aux
+                const uint allmask = (uint)PermissionMask.AllEffective;
+                const uint movemodmask = (uint)(PermissionMask.Move | PermissionMask.Modify);
+                const uint copytransfermast = (uint)(PermissionMask.Copy | PermissionMask.Transfer);
+
+                uint baseOwnerPerms = RootPart.OwnerMask;
+                uint owner = baseOwnerPerms;
+                uint baseGroupPerms = RootPart.GroupMask;
+                uint group = baseGroupPerms;
+                uint baseEveryonePerms = RootPart.EveryoneMask;
+                uint everyone = baseEveryonePerms;
 
-                if(baseperms == 0)
-                {
-                    m_EffectiveOwnerPerms = 0;
-                    return;
-                }
-                
-                uint current = baseperms;
                 SceneObjectPart[] parts = m_parts.GetArray();
                 for (int i = 0; i < parts.Length; i++)
                 {
                     SceneObjectPart part = parts[i];
-                    part.Inventory.AggregateOwnerPerms(ref current);
-                    if( current == 0)
-                        break;
+                    owner &= part.AggregatedInnerOwnerPerms; 
+                    group &= part.AggregatedInnerGroupPerms;
+                    everyone &= part.AggregatedInnerEveryonePerms;
                 }
                 // recover modify and move
-                baseperms &= (uint)(PermissionMask.Move | PermissionMask.Modify );
-                current |= baseperms;
-                current &= (uint)PermissionMask.All;
-                if((current & (uint)(PermissionMask.Copy | PermissionMask.Transfer)) == 0)
-                    current |= (uint)PermissionMask.Transfer;
-                m_EffectiveOwnerPerms = current;
+                baseOwnerPerms &= movemodmask;
+                owner |= baseOwnerPerms;
+                if((owner & copytransfermast) == 0)
+                    owner |= (uint)PermissionMask.Transfer;
+                owner &= allmask;
+                m_EffectiveOwnerPerms = owner;
+
+                // recover modify and move
+                baseGroupPerms &= movemodmask;
+                group |= baseGroupPerms;
+                if((group & copytransfermast) == 0)
+                    group |= (uint)PermissionMask.Transfer;
+                group &= allmask;
+                m_EffectiveGroupPerms = group;
+
+                // recover move
+                baseEveryonePerms &= (uint)PermissionMask.Move;
+                everyone |= baseEveryonePerms;
+                if((everyone & copytransfermast) == 0) // not much sense but as sl
+                    everyone |= (uint)PermissionMask.Transfer;
+                everyone &= allmask;
+                m_EffectiveEveryOnePerms = everyone;
             }
         }
 
diff --git a/OpenSim/Region/Framework/Scenes/SceneObjectPart.cs b/OpenSim/Region/Framework/Scenes/SceneObjectPart.cs
index 557b55e..9f99665 100644
--- a/OpenSim/Region/Framework/Scenes/SceneObjectPart.cs
+++ b/OpenSim/Region/Framework/Scenes/SceneObjectPart.cs
@@ -2537,6 +2537,33 @@ namespace OpenSim.Region.Framework.Scenes
             return (uint)Flags | (uint)LocalFlags;
         }
 
+        // some of this lines need be moved to other place later
+
+        // effective permitions considering only this part inventory contents perms
+        public uint AggregatedInnerOwnerPerms {get; private set; }
+        public uint AggregatedInnerGroupPerms {get; private set; }
+        public uint AggregatedInnerEveryonePerms {get; private set; }
+        private object InnerPermsLock = new object();
+
+        public void AggregateInnerPerms()
+        {
+            const uint mask = (uint)PermissionMask.AllEffective;
+
+            uint owner = mask;
+            uint group = mask;
+            uint everyone = mask;
+
+            lock(InnerPermsLock) // do we really need this?
+            {
+                if(Inventory != null)
+                    Inventory.AggregateInnerPerms(ref owner, ref group, ref everyone);
+            
+                AggregatedInnerOwnerPerms = owner & mask;
+                AggregatedInnerGroupPerms = group & mask;
+                AggregatedInnerEveryonePerms = everyone & mask;
+            }
+        }
+
         public Vector3 GetGeometricCenter()
         {
             // this is not real geometric center but a average of positions relative to root prim acording to
diff --git a/OpenSim/Region/Framework/Scenes/SceneObjectPartInventory.cs b/OpenSim/Region/Framework/Scenes/SceneObjectPartInventory.cs
index 636afac..a55936f 100644
--- a/OpenSim/Region/Framework/Scenes/SceneObjectPartInventory.cs
+++ b/OpenSim/Region/Framework/Scenes/SceneObjectPartInventory.cs
@@ -1317,48 +1317,16 @@ namespace OpenSim.Region.Framework.Scenes
             }
         }
 
-        // reduce to minimal set
-        public void AggregateEveryOnePerms(ref uint current)
+        public void AggregateInnerPerms(ref uint owner, ref uint group, ref uint everyone)
         {
             foreach (TaskInventoryItem item in m_items.Values)
             {
-                current &= item.EveryonePermissions;
-                if(current == 0)
-                    break;
-            }
-        }
-
-        public void AggregateGroupPerms(ref uint current)
-        {
-            foreach (TaskInventoryItem item in m_items.Values)
-            {
-                current &= item.GroupPermissions;
-                if(current == 0)
-                    break;
+                owner &= item.CurrentPermissions;
+                group &= item.GroupPermissions;
+                everyone &= item.EveryonePermissions;
             }
         }
 
-        public void AggregateGroupOrEveryonePerms(ref uint current)
-        {
-            foreach (TaskInventoryItem item in m_items.Values)
-            {
-                current &= (item.GroupPermissions | item.EveryonePermissions);
-                if(current == 0)
-                    break;
-            }
-        }
-
-        public void AggregateOwnerPerms(ref uint current)
-        {
-            foreach (TaskInventoryItem item in m_items.Values)
-            {
-                current &= item.CurrentPermissions;
-                if(current == 0)
-                    break;
-            }
-        }
-
-
         public uint MaskEffectivePermissions()
         {
             uint mask=0x7fffffff;