[Mesa-dev] [PATCH 2/2] r600/compute: Mark several functions as static
Jan Vesely
jan.vesely at rutgers.edu
Sat May 19 02:52:15 UTC 2018
On Fri, 2018-05-18 at 21:31 -0500, Aaron Watry wrote:
> On Fri, May 18, 2018 at 1:15 PM, Jan Vesely <jan.vesely at rutgers.edu> wrote:
> > On Thu, 2018-05-17 at 19:20 -0500, Aaron Watry wrote:
> > > They're not used anywhere else, so keep them private
> > >
> > > Signed-off-by: Aaron Watry <awatry at gmail.com>
> >
> > I'm not sure what the original purpose of the removed functions was. If
> > you recall please add it to the commit message.
>
> I didn't really bother looking into it when I made this change (I
> didn't write this code in the first place). This is something that
> I've had sitting in my local repo for a while, and just want to stop
> rebasing it.
>
> It was originally found a while ago when I started looking into the
> thread-safety (or lack thereof) of the compute pool for r600. Figured
> there was no point trying to fix code that was unused.
OK, nvm then. I was just curious how we ended up with so much dead
code.
>
> > Either way:
> >
> > Reviewed-by: Jan Vesely <jan.vesely at rutgers.edu>
>
> For this one, or both?
for the series.
thanks,
Jan
>
> --Aaron
>
> >
> > Jan
> >
> > > ---
> > > .../drivers/r600/compute_memory_pool.c | 35 +++++++++++++++----
> > > .../drivers/r600/compute_memory_pool.h | 24 -------------
> > > 2 files changed, 29 insertions(+), 30 deletions(-)
> > >
> > > diff --git a/src/gallium/drivers/r600/compute_memory_pool.c b/src/gallium/drivers/r600/compute_memory_pool.c
> > > index d1ef25ae1e..981d944b8d 100644
> > > --- a/src/gallium/drivers/r600/compute_memory_pool.c
> > > +++ b/src/gallium/drivers/r600/compute_memory_pool.c
> > > @@ -43,6 +43,29 @@
> > > #include <inttypes.h>
> > >
> > > #define ITEM_ALIGNMENT 1024
> > > +
> > > +/* A few forward declarations of static functions */
> > > +static void compute_memory_shadow(struct compute_memory_pool* pool,
> > > + struct pipe_context *pipe, int device_to_host);
> > > +
> > > +static void compute_memory_defrag(struct compute_memory_pool *pool,
> > > + struct pipe_resource *src, struct pipe_resource *dst,
> > > + struct pipe_context *pipe);
> > > +
> > > +static int compute_memory_promote_item(struct compute_memory_pool *pool,
> > > + struct compute_memory_item *item, struct pipe_context *pipe,
> > > + int64_t allocated);
> > > +
> > > +static void compute_memory_move_item(struct compute_memory_pool *pool,
> > > + struct pipe_resource *src, struct pipe_resource *dst,
> > > + struct compute_memory_item *item, uint64_t new_start_in_dw,
> > > + struct pipe_context *pipe);
> > > +
> > > +static void compute_memory_transfer(struct compute_memory_pool* pool,
> > > + struct pipe_context * pipe, int device_to_host,
> > > + struct compute_memory_item* chunk, void* data,
> > > + int offset_in_chunk, int size);
> > > +
> > > /**
> > > * Creates a new pool.
> > > */
> > > @@ -106,7 +129,7 @@ void compute_memory_pool_delete(struct compute_memory_pool* pool)
> > > * \returns -1 if it fails, 0 otherwise
> > > * \see compute_memory_finalize_pending
> > > */
> > > -int compute_memory_grow_defrag_pool(struct compute_memory_pool *pool,
> > > +static int compute_memory_grow_defrag_pool(struct compute_memory_pool *pool,
> > > struct pipe_context *pipe, int new_size_in_dw)
> > > {
> > > new_size_in_dw = align(new_size_in_dw, ITEM_ALIGNMENT);
> > > @@ -168,7 +191,7 @@ int compute_memory_grow_defrag_pool(struct compute_memory_pool *pool,
> > > * \param device_to_host 1 for device->host, 0 for host->device
> > > * \see compute_memory_grow_defrag_pool
> > > */
> > > -void compute_memory_shadow(struct compute_memory_pool* pool,
> > > +static void compute_memory_shadow(struct compute_memory_pool* pool,
> > > struct pipe_context * pipe, int device_to_host)
> > > {
> > > struct compute_memory_item chunk;
> > > @@ -262,7 +285,7 @@ int compute_memory_finalize_pending(struct compute_memory_pool* pool,
> > > * \param dst The destination resource
> > > * \see compute_memory_grow_defrag_pool and compute_memory_finalize_pending
> > > */
> > > -void compute_memory_defrag(struct compute_memory_pool *pool,
> > > +static void compute_memory_defrag(struct compute_memory_pool *pool,
> > > struct pipe_resource *src, struct pipe_resource *dst,
> > > struct pipe_context *pipe)
> > > {
> > > @@ -292,7 +315,7 @@ void compute_memory_defrag(struct compute_memory_pool *pool,
> > > * \return -1 if it fails, 0 otherwise
> > > * \see compute_memory_finalize_pending
> > > */
> > > -int compute_memory_promote_item(struct compute_memory_pool *pool,
> > > +static int compute_memory_promote_item(struct compute_memory_pool *pool,
> > > struct compute_memory_item *item, struct pipe_context *pipe,
> > > int64_t start_in_dw)
> > > {
> > > @@ -397,7 +420,7 @@ void compute_memory_demote_item(struct compute_memory_pool *pool,
> > > * \param new_start_in_dw The new position of the item in \a item_list
> > > * \see compute_memory_defrag
> > > */
> > > -void compute_memory_move_item(struct compute_memory_pool *pool,
> > > +static void compute_memory_move_item(struct compute_memory_pool *pool,
> > > struct pipe_resource *src, struct pipe_resource *dst,
> > > struct compute_memory_item *item, uint64_t new_start_in_dw,
> > > struct pipe_context *pipe)
> > > @@ -569,7 +592,7 @@ struct compute_memory_item* compute_memory_alloc(
> > > * \param device_to_host 1 for device->host, 0 for host->device.
> > > * \see compute_memory_shadow
> > > */
> > > -void compute_memory_transfer(
> > > +static void compute_memory_transfer(
> > > struct compute_memory_pool* pool,
> > > struct pipe_context * pipe,
> > > int device_to_host,
> > > diff --git a/src/gallium/drivers/r600/compute_memory_pool.h b/src/gallium/drivers/r600/compute_memory_pool.h
> > > index 3a17c5176b..2064e56352 100644
> > > --- a/src/gallium/drivers/r600/compute_memory_pool.h
> > > +++ b/src/gallium/drivers/r600/compute_memory_pool.h
> > > @@ -86,39 +86,15 @@ struct compute_memory_pool* compute_memory_pool_new(struct r600_screen *rscreen)
> > >
> > > void compute_memory_pool_delete(struct compute_memory_pool* pool);
> > >
> > > -int compute_memory_grow_defrag_pool(struct compute_memory_pool* pool,
> > > - struct pipe_context *pipe, int new_size_in_dw);
> > > -
> > > -void compute_memory_shadow(struct compute_memory_pool* pool,
> > > - struct pipe_context *pipe, int device_to_host);
> > > -
> > > int compute_memory_finalize_pending(struct compute_memory_pool* pool,
> > > struct pipe_context * pipe);
> > >
> > > -void compute_memory_defrag(struct compute_memory_pool *pool,
> > > - struct pipe_resource *src, struct pipe_resource *dst,
> > > - struct pipe_context *pipe);
> > > -
> > > -int compute_memory_promote_item(struct compute_memory_pool *pool,
> > > - struct compute_memory_item *item, struct pipe_context *pipe,
> > > - int64_t allocated);
> > > -
> > > void compute_memory_demote_item(struct compute_memory_pool *pool,
> > > struct compute_memory_item *item, struct pipe_context *pipe);
> > >
> > > -void compute_memory_move_item(struct compute_memory_pool *pool,
> > > - struct pipe_resource *src, struct pipe_resource *dst,
> > > - struct compute_memory_item *item, uint64_t new_start_in_dw,
> > > - struct pipe_context *pipe);
> > > -
> > > void compute_memory_free(struct compute_memory_pool* pool, int64_t id);
> > >
> > > struct compute_memory_item* compute_memory_alloc(struct compute_memory_pool* pool,
> > > int64_t size_in_dw);
> > >
> > > -void compute_memory_transfer(struct compute_memory_pool* pool,
> > > - struct pipe_context * pipe, int device_to_host,
> > > - struct compute_memory_item* chunk, void* data,
> > > - int offset_in_chunk, int size);
> > > -
> > > #endif
--
Jan Vesely <jan.vesely at rutgers.edu>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: This is a digitally signed message part
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20180518/c8359098/attachment.sig>
More information about the mesa-dev
mailing list