> -----Original Message-----
> From: Jiri Pirko <j...@resnulli.us>
> Sent: Friday, September 4, 2020 2:07 PM
> To: Sunil Kovvuri Goutham <sgout...@marvell.com>
> Cc: Jakub Kicinski <k...@kernel.org>; sundeep.l...@gmail.com;
> da...@davemloft.net; netdev@vger.kernel.org; Subbaraya Sundeep
> Bhatta <sbha...@marvell.com>
> Subject: Re: [EXT] Re: [net-next PATCH 0/2] Introduce mbox tracepoints for
> Octeontx2
>
> Fri, Sep 04, 2020 at 07:39:54AM CEST, sgout...@marvell.com wrote:
> >
> >
> >> -----Original Message-----
> >> From: Jakub Kicinski <k...@kernel.org>
> >> Sent: Friday, September 4, 2020 12:48 AM
> >> To: sundeep.l...@gmail.com
> >> Cc: da...@davemloft.net; netdev@vger.kernel.org; Sunil Kovvuri
> >> Goutham <sgout...@marvell.com>; Subbaraya Sundeep Bhatta
> >> <sbha...@marvell.com>
> >> Subject: [EXT] Re: [net-next PATCH 0/2] Introduce mbox tracepoints
> >> for
> >> Octeontx2
> >>
> >> External Email
> >>
> >> ---------------------------------------------------------------------
> >> - On Thu, 3 Sep 2020 12:48:16 +0530 sundeep.l...@gmail.com wrote:
> >> > From: Subbaraya Sundeep <sbha...@marvell.com>
> >> >
> >> > This patchset adds tracepoints support for mailbox.
> >> > In Octeontx2, PFs and VFs need to communicate with AF for
> >> > allocating and freeing resources. Once all the configuration is
> >> > done by AF for a PF/VF then packet I/O can happen on PF/VF queues.
> >> > When an interface is brought up many mailbox messages are sent to
> >> > AF for initializing queues. Say a VF is brought up then each
> >> > message is sent to PF and PF forwards to AF and response also traverses
> from AF to PF and then VF.
> >> > To aid debugging, tracepoints are added at places where messages
> >> > are allocated, sent and message interrupts.
> >> > Below is the trace of one of the messages from VF to AF and AF
> >> > response back to VF:
> >>
> >> Could you use the devlink tracepoint? trace_devlink_hwmsg() ?
> >
> >Thanks for the suggestion.
> >In our case the mailbox is central to 3 different drivers and there
> >would be a 4th one once crypto driver is accepted. We cannot add
> >devlink to all of them inorder to use the devlink trace points.
>
> I guess you have 1 pci device, right? Devlink instance is created per pci
> device.
>
No, there are 3 drivers registering to 3 PCI device IDs and there can be many
instances of the same devices. So there can be 10's of instances of AF, PF and
VFs.
Thanks,
Sunil.
