Changeset fdaad75d in mainline for kernel/generic/src/mm/as.c


Ignore:
Timestamp:
2010-06-22T02:18:35Z (14 years ago)
Author:
Jakub Jermar <jakub@…>
Branches:
lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export
Children:
ee42e43
Parents:
4ca26c9b
Message:

Do not disable interrupts unnecessarily.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • kernel/generic/src/mm/as.c

    r4ca26c9b rfdaad75d  
    239239       
    240240        spinlock_unlock(&asidlock);
     241        interrupts_restore(ipl);
     242
    241243       
    242244        /*
    … …  
    265267        page_table_destroy(NULL);
    266268#endif
    267        
    268         interrupts_restore(ipl);
    269269       
    270270        slab_free(as_slab, as);
    … …  
    327327                return NULL;
    328328       
    329         ipl_t ipl = interrupts_disable();
    330329        mutex_lock(&as->lock);
    331330       
    332331        if (!check_area_conflicts(as, base, size, NULL)) {
    333332                mutex_unlock(&as->lock);
    334                 interrupts_restore(ipl);
    335333                return NULL;
    336334        }
    … …  
    357355       
    358356        mutex_unlock(&as->lock);
    359         interrupts_restore(ipl);
    360357       
    361358        return area;
    … …  
    376373int as_area_resize(as_t *as, uintptr_t address, size_t size, unsigned int flags)
    377374{
    378         ipl_t ipl = interrupts_disable();
    379375        mutex_lock(&as->lock);
    380376       
    … …  
    386382        if (!area) {
    387383                mutex_unlock(&as->lock);
    388                 interrupts_restore(ipl);
    389384                return ENOENT;
    390385        }
    … …  
    398393                mutex_unlock(&area->lock);
    399394                mutex_unlock(&as->lock);
    400                 interrupts_restore(ipl);
    401395                return ENOTSUP;
    402396        }
    … …  
    410404                mutex_unlock(&area->lock);
    411405                mutex_unlock(&as->lock);
    412                 interrupts_restore(ipl);
    413406                return ENOTSUP;
    414407        }
    … …  
    422415                mutex_unlock(&area->lock);
    423416                mutex_unlock(&as->lock);
    424                 interrupts_restore(ipl);
    425417                return EPERM;
    426418        }
    … …  
    549541                        mutex_unlock(&area->lock);
    550542                        mutex_unlock(&as->lock);
    551                         interrupts_restore(ipl);
    552543                        return EADDRNOTAVAIL;
    553544                }
    … …  
    558549        mutex_unlock(&area->lock);
    559550        mutex_unlock(&as->lock);
    560         interrupts_restore(ipl);
    561551       
    562552        return 0;
    … …  
    573563int as_area_destroy(as_t *as, uintptr_t address)
    574564{
    575         ipl_t ipl = interrupts_disable();
    576565        mutex_lock(&as->lock);
    577566       
    … …  
    579568        if (!area) {
    580569                mutex_unlock(&as->lock);
    581                 interrupts_restore(ipl);
    582570                return ENOENT;
    583571        }
    … …  
    659647       
    660648        mutex_unlock(&as->lock);
    661         interrupts_restore(ipl);
    662649        return 0;
    663650}
    … …  
    690677    as_t *dst_as, uintptr_t dst_base, unsigned int dst_flags_mask)
    691678{
    692         ipl_t ipl = interrupts_disable();
    693679        mutex_lock(&src_as->lock);
    694680        as_area_t *src_area = find_area_and_lock(src_as, src_base);
    … …  
    699685                 */
    700686                mutex_unlock(&src_as->lock);
    701                 interrupts_restore(ipl);
    702687                return ENOENT;
    703688        }
    … …  
    711696                mutex_unlock(&src_area->lock);
    712697                mutex_unlock(&src_as->lock);
    713                 interrupts_restore(ipl);
    714698                return ENOTSUP;
    715699        }
    … …  
    728712                mutex_unlock(&src_area->lock);
    729713                mutex_unlock(&src_as->lock);
    730                 interrupts_restore(ipl);
    731714                return EPERM;
    732715        }
    … …  
    777760                sh_info_remove_reference(sh_info);
    778761               
    779                 interrupts_restore(ipl);
    780762                return ENOMEM;
    781763        }
    … …  
    794776        mutex_unlock(&dst_as->lock);
    795777       
    796         interrupts_restore(ipl);
    797        
    798778        return 0;
    799779}
    … …  
    816796        };
    817797
    818         ASSERT(interrupts_disabled());
    819798        ASSERT(mutex_locked(&area->lock));
    820799       
    … …  
    844823        unsigned int page_flags = area_flags_to_page_flags(flags);
    845824       
    846         ipl_t ipl = interrupts_disable();
    847825        mutex_lock(&as->lock);
    848826       
    … …  
    850828        if (!area) {
    851829                mutex_unlock(&as->lock);
    852                 interrupts_restore(ipl);
    853830                return ENOENT;
    854831        }
    … …  
    859836                mutex_unlock(&area->lock);
    860837                mutex_unlock(&as->lock);
    861                 interrupts_restore(ipl);
    862838                return ENOTSUP;
    863839        }
    … …  
    978954        mutex_unlock(&area->lock);
    979955        mutex_unlock(&as->lock);
    980         interrupts_restore(ipl);
    981956       
    982957        return 0;
    … …  
    12191194unsigned int as_area_get_flags(as_area_t *area)
    12201195{
    1221         ASSERT(interrupts_disabled());
    12221196        ASSERT(mutex_locked(&area->lock));
    12231197
    … …  
    13211295as_area_t *find_area_and_lock(as_t *as, uintptr_t va)
    13221296{
    1323         ASSERT(interrupts_disabled());
    13241297        ASSERT(mutex_locked(&as->lock));
    13251298
    … …  
    13861359    as_area_t *avoid_area)
    13871360{
    1388         ASSERT(interrupts_disabled());
    13891361        ASSERT(mutex_locked(&as->lock));
    13901362
    … …  
    14861458        size_t size;
    14871459       
    1488         ipl_t ipl = interrupts_disable();
    14891460        page_table_lock(AS, true);
    14901461        as_area_t *src_area = find_area_and_lock(AS, base);
    … …  
    14971468       
    14981469        page_table_unlock(AS, true);
    1499         interrupts_restore(ipl);
    15001470        return size;
    15011471}
    … …  
    20702040void as_get_area_info(as_t *as, as_area_info_t **obuf, size_t *osize)
    20712041{
    2072         ipl_t ipl = interrupts_disable();
    20732042        mutex_lock(&as->lock);
    20742043       
    … …  
    21142083       
    21152084        mutex_unlock(&as->lock);
    2116         interrupts_restore(ipl);
    21172085       
    21182086        *obuf = info;
    … …  
    21272095void as_print(as_t *as)
    21282096{
    2129         ipl_t ipl = interrupts_disable();
    21302097        mutex_lock(&as->lock);
    21312098       
    … …  
    21502117       
    21512118        mutex_unlock(&as->lock);
    2152         interrupts_restore(ipl);
    21532119}
    21542120
Note: See TracChangeset for help on using the changeset viewer.