Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1650275 > unrolled thread

Re: [PATCH v2 07/13] perf header: use struct feat_fd for print

Started byJiri Olsa <jolsa@redhat.com>
First post2017-05-25 10:20 +0200
Last post2017-06-05 05:10 +0200
Articles 3 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH v2 07/13] perf header: use struct feat_fd for print Jiri Olsa <jolsa@redhat.com> - 2017-05-25 10:20 +0200
    Re: [PATCH v2 07/13] perf header: use struct feat_fd for print David Carrillo-Cisneros <davidcc@google.com> - 2017-06-05 04:40 +0200
      Re: [PATCH v2 07/13] perf header: use struct feat_fd for print David Carrillo-Cisneros <davidcc@google.com> - 2017-06-05 05:10 +0200

#1650275 — Re: [PATCH v2 07/13] perf header: use struct feat_fd for print

FromJiri Olsa <jolsa@redhat.com>
Date2017-05-25 10:20 +0200
SubjectRe: [PATCH v2 07/13] perf header: use struct feat_fd for print
Message-ID<tKW9X-5J4-1@gated-at.bofh.it>
On Tue, May 23, 2017 at 12:48:47AM -0700, David Carrillo-Cisneros wrote:
> As preparation for using header records in pipe mode, replace
> int fd with struct feat_fd fd in print functions for all header
> record types.
> 
> Add offset and size to struct feat_fd to provide pipe-mode with
> variables for offset and size that in file-mode are stored
> in struct perf_file_section.

please put that change in the pipe-mode change related patch
    perf header: add a buffer to struct feat_fd

thanks,
jirka

[toc] | [next] | [standalone]


#1657226

FromDavid Carrillo-Cisneros <davidcc@google.com>
Date2017-06-05 04:40 +0200
Message-ID<tOQ5X-5bh-5@gated-at.bofh.it>
In reply to#1650275
On Thu, May 25, 2017 at 1:09 AM, Jiri Olsa <jolsa@redhat.com> wrote:
> On Tue, May 23, 2017 at 12:48:47AM -0700, David Carrillo-Cisneros wrote:
>> As preparation for using header records in pipe mode, replace
>> int fd with struct feat_fd fd in print functions for all header
>> record types.
>>
>> Add offset and size to struct feat_fd to provide pipe-mode with
>> variables for offset and size that in file-mode are stored
>> in struct perf_file_section.
>
> please put that change in the pipe-mode change related patch
>     perf header: add a buffer to struct feat_fd
>

This is needed here to convert perf_file_section__fprintf_info . i
will explain that in the log.

[toc] | [prev] | [next] | [standalone]


#1657230

FromDavid Carrillo-Cisneros <davidcc@google.com>
Date2017-06-05 05:10 +0200
Message-ID<tOQyZ-5F6-5@gated-at.bofh.it>
In reply to#1657226
On Sun, Jun 4, 2017 at 7:37 PM, David Carrillo-Cisneros
<davidcc@google.com> wrote:
> On Thu, May 25, 2017 at 1:09 AM, Jiri Olsa <jolsa@redhat.com> wrote:
>> On Tue, May 23, 2017 at 12:48:47AM -0700, David Carrillo-Cisneros wrote:
>>> As preparation for using header records in pipe mode, replace
>>> int fd with struct feat_fd fd in print functions for all header
>>> record types.
>>>
>>> Add offset and size to struct feat_fd to provide pipe-mode with
>>> variables for offset and size that in file-mode are stored
>>> in struct perf_file_section.
>>
>> please put that change in the pipe-mode change related patch
>>     perf header: add a buffer to struct feat_fd
>>
>
> This is needed here to convert perf_file_section__fprintf_info . i
> will explain that in the log.

Never mind. Will move.

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web