diff options
| author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2015-09-22 11:02:18 +0200 | 
|---|---|---|
| committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2015-09-24 17:18:41 +0200 | 
| commit | 646db260b843d2f758559a5483174354c304acf8 (patch) | |
| tree | 73cf06896311567e8fcb75730d4461fd689a3d3f /lib/mpi/mpicoder.c | |
| parent | a645654b817feba05e5156345325d19fc85ebc9f (diff) | |
| parent | 1f93e4a96c9109378204c147b3eec0d0e8100fde (diff) | |
Merge tag 'v4.3-rc2' into topic/drm-misc
Backmerge Linux 4.3-rc2 because of conflicts in the dp helper code
between bugfixes and new code. Just adjacent lines really.
On top of that there's a silent conflict in the new fsl-dcu driver
merged into 4.3 and
commit 844f9111f6f54f88eb2f0fac121b82ce77193866
Author: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Date:   Wed Sep 2 10:42:40 2015 +0200
    drm/atomic: Make prepare_fb/cleanup_fb only take state, v3.
which Thierry Reding spotted and provided a fixup for.
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Diffstat (limited to 'lib/mpi/mpicoder.c')
| -rw-r--r-- | lib/mpi/mpicoder.c | 38 | 
1 files changed, 25 insertions, 13 deletions
| diff --git a/lib/mpi/mpicoder.c b/lib/mpi/mpicoder.c index bc0a1da8afba..95c52a95259e 100644 --- a/lib/mpi/mpicoder.c +++ b/lib/mpi/mpicoder.c @@ -146,18 +146,25 @@ int mpi_read_buffer(MPI a, uint8_t *buf, unsigned buf_len, unsigned *nbytes,  	uint8_t *p;  	mpi_limb_t alimb;  	unsigned int n = mpi_get_size(a); -	int i; +	int i, lzeros = 0; -	if (buf_len < n || !buf) +	if (buf_len < n || !buf || !nbytes)  		return -EINVAL;  	if (sign)  		*sign = a->sign; -	if (nbytes) -		*nbytes = n; +	p = (void *)&a->d[a->nlimbs] - 1; + +	for (i = a->nlimbs * sizeof(alimb) - 1; i >= 0; i--, p--) { +		if (!*p) +			lzeros++; +		else +			break; +	}  	p = buf; +	*nbytes = n - lzeros;  	for (i = a->nlimbs - 1; i >= 0; i--) {  		alimb = a->d[i]; @@ -178,6 +185,19 @@ int mpi_read_buffer(MPI a, uint8_t *buf, unsigned buf_len, unsigned *nbytes,  #else  #error please implement for this limb size.  #endif + +		if (lzeros > 0) { +			if (lzeros >= sizeof(alimb)) { +				p -= sizeof(alimb); +			} else { +				mpi_limb_t *limb1 = (void *)p - sizeof(alimb); +				mpi_limb_t *limb2 = (void *)p - sizeof(alimb) +							+ lzeros; +				*limb1 = *limb2; +				p -= lzeros; +			} +			lzeros -= sizeof(alimb); +		}  	}  	return 0;  } @@ -197,7 +217,7 @@ EXPORT_SYMBOL_GPL(mpi_read_buffer);   */  void *mpi_get_buffer(MPI a, unsigned *nbytes, int *sign)  { -	uint8_t *buf, *p; +	uint8_t *buf;  	unsigned int n;  	int ret; @@ -220,14 +240,6 @@ void *mpi_get_buffer(MPI a, unsigned *nbytes, int *sign)  		kfree(buf);  		return NULL;  	} - -	/* this is sub-optimal but we need to do the shift operation -	 * because the caller has to free the returned buffer */ -	for (p = buf; !*p && *nbytes; p++, --*nbytes) -		; -	if (p != buf) -		memmove(buf, p, *nbytes); -  	return buf;  }  EXPORT_SYMBOL_GPL(mpi_get_buffer); | 
