Build fixes for gcc 7.2 v2

src/perfctr.c: In function ‘destroy_shared_region’:
src/perfctr.c:154:10: error: logical ‘and’ of equal expressions [-Werror=logical-op]
  if (sem && sem != SEM_FAILED) {
          ^~
src/perfctr.c: In function ‘update_block_slots’:
src/perfctr.c:323:11: error: logical ‘or’ of equal expressions [-Werror=logical-op]
  if (!sem || sem == SEM_FAILED)
           ^~
v2:
  - Initialize and reset sem to SEM_FAILED.

Change-Id: Id70361079b715c4946b13e4460e4fd85d9542c46
Signed-off-by: Felix Kuehling <Felix.Kuehling@amd.com>


[ROCm/ROCR-Runtime commit: 4815d9b887]

projects/rocr-runtime/src/perfctr.c

@@ -90,7 +90,7 @@ static unsigned int counter_props_count;
 static const char shmem_name[] = "/hsakmt_shared_mem";
 static int shmem_fd;
 static const char sem_name[] = "hsakmt_semaphore";
-static sem_t *sem;
+static sem_t *sem = SEM_FAILED;
 struct perf_shared_table *shared_table;
 
 static ssize_t readn(int fd, void *buf, size_t n)
@@ -140,7 +140,7 @@ exit_2:
 exit_1:
 	sem_close(sem);
 	sem_unlink(sem_name);
-	sem = NULL;
+	sem = SEM_FAILED;
 	return HSAKMT_STATUS_ERROR;
 }
 
@@ -151,9 +151,10 @@ static void destroy_shared_region(void)
 
 	if (shmem_fd > 0)
 		shm_unlink(shmem_name);
-	if (sem && sem != SEM_FAILED) {
+	if (sem != SEM_FAILED) {
 		sem_close(sem);
 		sem_unlink(sem_name);
+		sem = SEM_FAILED;
 	}
 }
 
@@ -325,7 +326,7 @@ static HSAKMT_STATUS update_block_slots(enum perf_trace_action action,
 
 	if (shmem_fd <= 0)
 		return HSAKMT_STATUS_UNAVAILABLE;
-	if (!sem || sem == SEM_FAILED)
+	if (sem == SEM_FAILED)
 		return HSAKMT_STATUS_UNAVAILABLE;
 
 	sem_wait(sem);