On Wed, Apr 10, 2019 at 12:28:22PM +0200, Thomas Gleixner wrote: > Replace the indirection through struct stack_trace with an invocation of > the storage array based interface. > > Signed-off-by: Thomas Gleixner <[email protected]> > Cc: [email protected] > Cc: Robin Murphy <[email protected]> > Cc: Christoph Hellwig <[email protected]> > Cc: Marek Szyprowski <[email protected]> > --- > kernel/dma/debug.c | 13 +++++-------- > 1 file changed, 5 insertions(+), 8 deletions(-) > > --- a/kernel/dma/debug.c > +++ b/kernel/dma/debug.c > @@ -89,8 +89,8 @@ struct dma_debug_entry { > int sg_mapped_ents; > enum map_err_types map_err_type; > #ifdef CONFIG_STACKTRACE > - struct stack_trace stacktrace; > - unsigned long st_entries[DMA_DEBUG_STACKTRACE_ENTRIES]; > + unsigned int st_len; > + unsigned long st_entries[DMA_DEBUG_STACKTRACE_ENTRIES];
nit: st_entries isn't very readable. Thanks to the magic of compilers, the characters are free, so why not call them "stacktrace_entries" and "stacktrace_len". -- Josh _______________________________________________ iommu mailing list [email protected] https://lists.linuxfoundation.org/mailman/listinfo/iommu
