Commit da22013f authored by Alan Cox's avatar Alan Cox Committed by Greg Kroah-Hartman

atomisp: remove indirection from sh_css_malloc

We have one hard coded set of behaviour so unpick the indirection and function
pointers. This isn't the whole story. A lot of the callers are known sizes and
use cases so we can switch them directly to kmalloc later on.
Signed-off-by: default avatarAlan Cox <alan@linux.intel.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 83fceac0
...@@ -923,9 +923,6 @@ int atomisp_css_load_firmware(struct atomisp_device *isp) ...@@ -923,9 +923,6 @@ int atomisp_css_load_firmware(struct atomisp_device *isp)
isp->css_env.isp_css_fw.data = (void *)isp->firmware->data; isp->css_env.isp_css_fw.data = (void *)isp->firmware->data;
isp->css_env.isp_css_fw.bytes = isp->firmware->size; isp->css_env.isp_css_fw.bytes = isp->firmware->size;
isp->css_env.isp_css_env.cpu_mem_env.alloc = atomisp_kernel_zalloc;
isp->css_env.isp_css_env.cpu_mem_env.free = atomisp_kernel_free;
isp->css_env.isp_css_env.hw_access_env.store_8 = isp->css_env.isp_css_env.hw_access_env.store_8 =
atomisp_css2_hw_store_8; atomisp_css2_hw_store_8;
isp->css_env.isp_css_env.hw_access_env.store_16 = isp->css_env.isp_css_env.hw_access_env.store_16 =
......
...@@ -39,25 +39,8 @@ enum ia_css_mem_attr { ...@@ -39,25 +39,8 @@ enum ia_css_mem_attr {
* This is never expected to allocate more than one page of memory (4K bytes). * This is never expected to allocate more than one page of memory (4K bytes).
*/ */
struct ia_css_cpu_mem_env { struct ia_css_cpu_mem_env {
void * (*alloc)(size_t bytes, bool zero_mem);
/**< Allocation function with boolean argument to indicate whether
the allocated memory should be zeroed out or not, true (or 1)
meaning the memory given to CSS must be zeroed */
void (*free)(void *ptr);
/**< Corresponding free function. The function must also accept
a NULL argument, similar to C89 free(). */
void (*flush)(struct ia_css_acc_fw *fw); void (*flush)(struct ia_css_acc_fw *fw);
/**< Flush function to flush the cache for given accelerator. */ /**< Flush function to flush the cache for given accelerator. */
#ifdef ISP2401
#if !defined(__SVOS__)
/* a set of matching functions with additional debug params */
void * (*alloc_ex)(size_t bytes, bool zero_mem, const char *caller_func, int caller_line);
/**< same as alloc above, only with additional debug parameters */
void (*free_ex)(void *ptr, const char *caller_func, int caller_line);
/**< same as free above, only with additional debug parameters */
#endif
#endif
}; };
/** Environment with function pointers to access the CSS hardware. This includes /** Environment with function pointers to access the CSS hardware. This includes
...@@ -103,7 +86,7 @@ struct ia_css_print_env { ...@@ -103,7 +86,7 @@ struct ia_css_print_env {
* Windows and several simulation environments. * Windows and several simulation environments.
*/ */
struct ia_css_env { struct ia_css_env {
struct ia_css_cpu_mem_env cpu_mem_env; /**< local malloc and free. */ struct ia_css_cpu_mem_env cpu_mem_env; /**< local flush. */
struct ia_css_hw_access_env hw_access_env; /**< CSS HW access functions */ struct ia_css_hw_access_env hw_access_env; /**< CSS HW access functions */
struct ia_css_print_env print_env; /**< Message printing env. */ struct ia_css_print_env print_env; /**< Message printing env. */
}; };
......
...@@ -13,6 +13,10 @@ ...@@ -13,6 +13,10 @@
*/ */
/*! \file */ /*! \file */
#include <linux/mm.h>
#include <linux/slab.h>
#include <linux/vmalloc.h>
#include "ia_css.h" #include "ia_css.h"
#include "sh_css_hrt.h" /* only for file 2 MIPI */ #include "sh_css_hrt.h" /* only for file 2 MIPI */
#include "ia_css_buffer.h" #include "ia_css_buffer.h"
...@@ -1679,15 +1683,8 @@ ia_css_load_firmware(const struct ia_css_env *env, ...@@ -1679,15 +1683,8 @@ ia_css_load_firmware(const struct ia_css_env *env,
ia_css_debug_dtrace(IA_CSS_DEBUG_TRACE, "ia_css_load_firmware() enter\n"); ia_css_debug_dtrace(IA_CSS_DEBUG_TRACE, "ia_css_load_firmware() enter\n");
/* make sure we initialize my_css */ /* make sure we initialize my_css */
if ((my_css.malloc != env->cpu_mem_env.alloc) || if (my_css.flush != env->cpu_mem_env.flush) {
(my_css.free != env->cpu_mem_env.free) ||
(my_css.flush != env->cpu_mem_env.flush)
)
{
ia_css_reset_defaults(&my_css); ia_css_reset_defaults(&my_css);
my_css.malloc = env->cpu_mem_env.alloc;
my_css.free = env->cpu_mem_env.free;
my_css.flush = env->cpu_mem_env.flush; my_css.flush = env->cpu_mem_env.flush;
} }
...@@ -1715,8 +1712,6 @@ ia_css_init(const struct ia_css_env *env, ...@@ -1715,8 +1712,6 @@ ia_css_init(const struct ia_css_env *env,
ia_css_blctrl_cfg blctrl_cfg; ia_css_blctrl_cfg blctrl_cfg;
#endif #endif
void *(*malloc_func)(size_t size, bool zero_mem);
void (*free_func)(void *ptr);
void (*flush_func)(struct ia_css_acc_fw *fw); void (*flush_func)(struct ia_css_acc_fw *fw);
hrt_data select, enable; hrt_data select, enable;
...@@ -1765,8 +1760,6 @@ ia_css_init(const struct ia_css_env *env, ...@@ -1765,8 +1760,6 @@ ia_css_init(const struct ia_css_env *env,
IA_CSS_ENTER("void"); IA_CSS_ENTER("void");
malloc_func = env->cpu_mem_env.alloc;
free_func = env->cpu_mem_env.free;
flush_func = env->cpu_mem_env.flush; flush_func = env->cpu_mem_env.flush;
pipe_global_init(); pipe_global_init();
...@@ -1786,16 +1779,9 @@ ia_css_init(const struct ia_css_env *env, ...@@ -1786,16 +1779,9 @@ ia_css_init(const struct ia_css_env *env,
ia_css_save_mmu_base_addr(mmu_l1_base); ia_css_save_mmu_base_addr(mmu_l1_base);
#endif #endif
if (malloc_func == NULL || free_func == NULL) {
IA_CSS_LEAVE_ERR(IA_CSS_ERR_INVALID_ARGUMENTS);
return IA_CSS_ERR_INVALID_ARGUMENTS;
}
ia_css_reset_defaults(&my_css); ia_css_reset_defaults(&my_css);
my_css_save.driver_env = *env; my_css_save.driver_env = *env;
my_css.malloc = malloc_func;
my_css.free = free_func;
my_css.flush = flush_func; my_css.flush = flush_func;
err = ia_css_rmgr_init(); err = ia_css_rmgr_init();
...@@ -2018,25 +2004,35 @@ ia_css_enable_isys_event_queue(bool enable) ...@@ -2018,25 +2004,35 @@ ia_css_enable_isys_event_queue(bool enable)
void *sh_css_malloc(size_t size) void *sh_css_malloc(size_t size)
{ {
ia_css_debug_dtrace(IA_CSS_DEBUG_TRACE, "sh_css_malloc() enter: size=%d\n",size); ia_css_debug_dtrace(IA_CSS_DEBUG_TRACE, "sh_css_malloc() enter: size=%d\n",size);
if (size > 0 && my_css.malloc) /* FIXME: This first test can probably go away */
return my_css.malloc(size, false); if (size == 0)
return NULL; return NULL;
if (size > PAGE_SIZE)
return vmalloc(size);
return kmalloc(size, GFP_KERNEL);
} }
void *sh_css_calloc(size_t N, size_t size) void *sh_css_calloc(size_t N, size_t size)
{ {
void *p;
ia_css_debug_dtrace(IA_CSS_DEBUG_TRACE, "sh_css_calloc() enter: N=%d, size=%d\n",N,size); ia_css_debug_dtrace(IA_CSS_DEBUG_TRACE, "sh_css_calloc() enter: N=%d, size=%d\n",N,size);
if (size > 0 && my_css.malloc)
return my_css.malloc(N*size, true); /* FIXME: this test can probably go away */
if (size > 0) {
p = sh_css_malloc(N*size);
if (p)
memset(p, 0, size);
}
return NULL; return NULL;
} }
void sh_css_free(void *ptr) void sh_css_free(void *ptr)
{ {
IA_CSS_ENTER_PRIVATE("ptr = %p", ptr); if (is_vmalloc_addr(ptr))
if (ptr && my_css.free) vfree(ptr);
my_css.free(ptr); else
IA_CSS_LEAVE_PRIVATE("void"); kfree(ptr);
} }
/* For Acceleration API: Flush FW (shared buffer pointer) arguments */ /* For Acceleration API: Flush FW (shared buffer pointer) arguments */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment