|
From: <sv...@va...> - 2015-05-10 11:38:44
|
Author: florian
Date: Sun May 10 12:38:38 2015
New Revision: 15204
Log:
Rename VG_(am_is_valid_for_client_or_free_or_resvn) to
VG_(am_is_allowed_for_client).
Also a few spelling and white space fixes.
Modified:
branches/ASPACEM_TWEAKS/coregrind/m_aspacemgr/aspacemgr-linux.c
branches/ASPACEM_TWEAKS/coregrind/m_syswrap/syswrap-generic.c
branches/ASPACEM_TWEAKS/coregrind/pub_core_aspacemgr.h
Modified: branches/ASPACEM_TWEAKS/coregrind/m_aspacemgr/aspacemgr-linux.c
==============================================================================
--- branches/ASPACEM_TWEAKS/coregrind/m_aspacemgr/aspacemgr-linux.c (original)
+++ branches/ASPACEM_TWEAKS/coregrind/m_aspacemgr/aspacemgr-linux.c Sun May 10 12:38:38 2015
@@ -1232,8 +1232,7 @@
be consider part of the client's addressable space. It also
considers reservations to be allowable, since from the client's
point of view they don't exist. */
-Bool VG_(am_is_valid_for_client_or_free_or_resvn)
- ( Addr start, SizeT len, UInt prot )
+Bool VG_(am_is_allowed_for_client)( Addr start, SizeT len, UInt prot )
{
const UInt kinds = SkFileC | SkAnonC | SkShmC | SkFree | SkResvn;
@@ -2616,8 +2615,7 @@
aspacem_assert(VG_IS_PAGE_ALIGNED(len));
if (forClient) {
- if (!VG_(am_is_valid_for_client_or_free_or_resvn)
- ( start, len, VKI_PROT_NONE ))
+ if (!VG_(am_is_allowed_for_client)( start, len, VKI_PROT_NONE ))
goto eINVAL;
} else {
if (!VG_(am_is_valid_for_valgrind)
Modified: branches/ASPACEM_TWEAKS/coregrind/m_syswrap/syswrap-generic.c
==============================================================================
--- branches/ASPACEM_TWEAKS/coregrind/m_syswrap/syswrap-generic.c (original)
+++ branches/ASPACEM_TWEAKS/coregrind/m_syswrap/syswrap-generic.c Sun May 10 12:38:38 2015
@@ -99,8 +99,7 @@
}
/* Returns True iff address range is something the client can
- plausibly mess with: all of it is either already belongs to the
- client or is free or a reservation. */
+ plausibly mess with. */
Bool ML_(valid_client_addr)(Addr start, SizeT size, ThreadId tid,
const HChar *syscallname)
@@ -110,8 +109,7 @@
if (size == 0)
return True;
- ret = VG_(am_is_valid_for_client_or_free_or_resvn)
- (start,size,VKI_PROT_NONE);
+ ret = VG_(am_is_allowed_for_client)(start, size, VKI_PROT_NONE);
if (0)
VG_(printf)("%s: test=%#lx-%#lx ret=%d\n",
Modified: branches/ASPACEM_TWEAKS/coregrind/pub_core_aspacemgr.h
==============================================================================
--- branches/ASPACEM_TWEAKS/coregrind/pub_core_aspacemgr.h (original)
+++ branches/ASPACEM_TWEAKS/coregrind/pub_core_aspacemgr.h Sun May 10 12:38:38 2015
@@ -70,8 +70,7 @@
// Querying current status
-/* Find the next segment along from 'here', if it is a file/anon/resvn
- segment. */
+/* Find the next non-free segment along from 'here'. */
extern NSegment const* VG_(am_next_nsegment) ( const NSegment* here,
Bool fwds );
@@ -81,15 +80,13 @@
VKI_PROT_NONE as 'prot'. Will return False if any part of the
area does not belong to valgrind or does not have at least
the stated permissions. */
-extern Bool VG_(am_is_valid_for_valgrind)
- ( Addr start, SizeT len, UInt prot );
+extern Bool VG_(am_is_valid_for_valgrind)( Addr start, SizeT len, UInt prot );
-/* Variant of VG_(am_is_valid_for_client) which allows free areas to
- be consider part of the client's addressable space. It also
+/* Variant of VG_(am_is_valid_for_valgrind) which allows free areas to
+ be considered part of the client's addressable space. It also
considers reservations to be allowable, since from the client's
point of view they don't exist. */
-extern Bool VG_(am_is_valid_for_client_or_free_or_resvn)
- ( Addr start, SizeT len, UInt prot );
+extern Bool VG_(am_is_allowed_for_client)( Addr start, SizeT len, UInt prot );
/* Check whether ADDR looks like an address or address-to-be located in an
extensible client stack segment. */
|