Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 0864dee0 authored by Veera Sundaram Sankaran's avatar Veera Sundaram Sankaran
Browse files

Merge 'dev/msm-3.18-android_sync_revert' into msm-3.18 



This reverts the sync driver changes introduced during kernel 3.18
upgrade as it is causing stability issues.

* (12 commits):
  android: sync: fix compilation errors when reverting sync fence api
  Move sync.h from staging into include/linux
  Revert "android: convert sync to fence api, v6"
  Revert "staging: android: Clean up else statement from sync_fence_poll()"
  Revert "staging: android: Cleanup style issues"
  Revert "staging: android: sync: Fix checkpatch warning: Missing a blank line after declarations"
  Revert "staging: android: use braces on all arms of if"
  Revert "android: fix reference leak in sync_fence_create"
  Revert "sync: Fix memory corruption in sync_timeline_signal()."
  Revert "staging: sync: Fix header file inclusion in sync_debug"
  Revert "Move sync.h from staging into include/linux"

Change-Id: Idce9e2394c5ba46c391c3cb117b223fa318ad119
Signed-off-by: default avatarVeera Sundaram Sankaran <veeras@codeaurora.org>
parents 2f867742 b9d2ec88

drivers/gpu/msm/kgsl_sync.c

+2 −2
Original line number Diff line number Diff line
@@ -50,14 +50,14 @@ static void kgsl_sync_pt_destroy(struct sync_pt *pt) 
static struct sync_pt *kgsl_sync_pt_dup(struct sync_pt *pt)

{

	struct kgsl_sync_pt *kpt = (struct kgsl_sync_pt *) pt;

	return kgsl_sync_pt_create(sync_pt_parent(pt), kpt->context, kpt->timestamp);

	return kgsl_sync_pt_create(pt->parent, kpt->context, kpt->timestamp);

}



static int kgsl_sync_pt_has_signaled(struct sync_pt *pt)

{

	struct kgsl_sync_pt *kpt = (struct kgsl_sync_pt *) pt;

	struct kgsl_sync_timeline *ktimeline =

		 (struct kgsl_sync_timeline *) sync_pt_parent(pt);

		 (struct kgsl_sync_timeline *) pt->parent;

	unsigned int ts = kpt->timestamp;

	int ret = 0;

drivers/staging/android/Kconfig

+0 −1
Original line number Diff line number Diff line
@@ -71,7 +71,6 @@ config SYNC 
	bool "Synchronization framework"

	default n

	select ANON_INODES

	select DMA_SHARED_BUFFER

	---help---

	  This option enables the framework for synchronization between multiple

	  drivers.  Sync implementations can take advantage of hardware

drivers/staging/android/Makefile

+1 −1
Original line number Diff line number Diff line
@@ -8,6 +8,6 @@ obj-$(CONFIG_ASHMEM) += ashmem.o 
obj-$(CONFIG_ANDROID_TIMED_OUTPUT)	+= timed_output.o

obj-$(CONFIG_ANDROID_TIMED_GPIO)	+= timed_gpio.o

obj-$(CONFIG_ANDROID_LOW_MEMORY_KILLER)	+= lowmemorykiller.o

obj-$(CONFIG_SYNC)			+= sync.o sync_debug.o

obj-$(CONFIG_SYNC)			+= sync.o

obj-$(CONFIG_SW_SYNC)			+= sw_sync.o

obj-$(CONFIG_ONESHOT_SYNC)		+= oneshot_sync.o

drivers/staging/android/oneshot_sync.c

+3 −3
Original line number Diff line number Diff line
@@ -129,7 +129,7 @@ static struct sync_pt *oneshot_pt_dup(struct sync_pt *sync_pt) 
		return NULL;



	out_pt = (struct oneshot_sync_pt *)

		sync_pt_create(sync_pt_parent(sync_pt), sizeof(*out_pt));

		sync_pt_create(sync_pt->parent, sizeof(*out_pt));



	if (out_pt == NULL) {

		oneshot_state_put(pt->state);
@@ -164,8 +164,8 @@ static void oneshot_pt_free(struct sync_pt *sync_pt) 
{

	struct oneshot_sync_pt *pt = to_oneshot_pt(sync_pt);



	struct oneshot_sync_timeline *timeline = sync_pt_parent(sync_pt) ?

		to_oneshot_timeline(sync_pt_parent(sync_pt)) : NULL;

	struct oneshot_sync_timeline *timeline = sync_pt->parent ?

		to_oneshot_timeline(sync_pt->parent) : NULL;



	if (timeline != NULL) {

		spin_lock(&timeline->lock);

drivers/staging/android/sw_sync.c

+2 −2
Original line number Diff line number Diff line
@@ -49,7 +49,7 @@ static struct sync_pt *sw_sync_pt_dup(struct sync_pt *sync_pt) 
{

	struct sw_sync_pt *pt = (struct sw_sync_pt *)sync_pt;

	struct sw_sync_timeline *obj =

		(struct sw_sync_timeline *)sync_pt_parent(sync_pt);

		(struct sw_sync_timeline *)sync_pt->parent;



	return (struct sync_pt *)sw_sync_pt_create(obj, pt->value);

}
@@ -58,7 +58,7 @@ static int sw_sync_pt_has_signaled(struct sync_pt *sync_pt) 
{

	struct sw_sync_pt *pt = (struct sw_sync_pt *)sync_pt;

	struct sw_sync_timeline *obj =

		(struct sw_sync_timeline *)sync_pt_parent(sync_pt);

		(struct sw_sync_timeline *)sync_pt->parent;



	return sw_sync_cmp(obj->value, pt->value) >= 0;

}