Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 24 Jan 2014 15:04:02 +0000 (UTC)
From:      Andriy Gapon <avg@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r261122 - head/cddl/contrib/opensolaris/lib/libdtrace/common
Message-ID:  <201401241504.s0OF42PD084338@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: avg
Date: Fri Jan 24 15:04:02 2014
New Revision: 261122
URL: http://svnweb.freebsd.org/changeset/base/261122

Log:
  dtrace: remove unexplained 16MB limitation from dt_alloc/dt_zalloc
  
  The limitation was introduced in r178556 without any note or comment.
  It seems pretty artificial and now it leads to problems like the following:
    $ dtrace -x bufsize=17m -n ...
    dtrace: processing aborted: Memory allocation failure
  OpenSolaris and illumos never had this limitation.
  
  Sponsored by:	HybridCluster

Modified:
  head/cddl/contrib/opensolaris/lib/libdtrace/common/dt_subr.c

Modified: head/cddl/contrib/opensolaris/lib/libdtrace/common/dt_subr.c
==============================================================================
--- head/cddl/contrib/opensolaris/lib/libdtrace/common/dt_subr.c	Fri Jan 24 15:03:56 2014	(r261121)
+++ head/cddl/contrib/opensolaris/lib/libdtrace/common/dt_subr.c	Fri Jan 24 15:04:02 2014	(r261122)
@@ -734,11 +734,6 @@ dt_zalloc(dtrace_hdl_t *dtp, size_t size
 {
 	void *data;
 
-	if (size > 16 * 1024 * 1024) {
-		(void) dt_set_errno(dtp, EDT_NOMEM);
-		return (NULL);
-	}
-
 	if ((data = malloc(size)) == NULL)
 		(void) dt_set_errno(dtp, EDT_NOMEM);
 	else
@@ -752,11 +747,6 @@ dt_alloc(dtrace_hdl_t *dtp, size_t size)
 {
 	void *data;
 
-	if (size > 16 * 1024 * 1024) {
-		(void) dt_set_errno(dtp, EDT_NOMEM);
-		return (NULL);
-	}
-
 	if ((data = malloc(size)) == NULL)
 		(void) dt_set_errno(dtp, EDT_NOMEM);
 



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201401241504.s0OF42PD084338>