Re: [PATCH v3 06/23] trace-cmd: Add new trace-cmd clock tsc2nsec

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 24 Mar 2021 18:56:39 +0200
Tzvetomir Stoyanov <tz.stoyanov@xxxxxxxxx> wrote:

> > > to free. That's why both are valid. I was wondering if to use strdup
> > > or simple assignment, and decided to allocate a memory. One day we may
> > > implement free_instance(), there are a lot of resources in an instance
> > > that should be freed.  
> >
> > Perhaps that "one day" should be this week ;-)  
> 
> That means to implement a cleanup after each trace-cmd subcommand ?
> Currently, trace-cmd just exits and not freeing that memory is not a
> big problem. I was thinking many times to implement free_instance(),
> but there is no sense for it if there is no cleanup logic at the end
> of each command.

It's not top priority. But something we should keep in mind.

-- Steve



[Index of Archives]     [Linux USB Development]     [Linux USB Development]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux