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

Groups > linux.kernel > #1463095 > unrolled thread

Re: [PATCH v6 1/3] PCI: Add Precision Time Measurement (PTM) support

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 v6 1/3] PCI: Add Precision Time Measurement (PTM) support Bjorn Helgaas <helgaas@kernel.org> - 2016-08-15 21:00 +0200
    Re: [PATCH v6 1/3] PCI: Add Precision Time Measurement (PTM) support "Yong, Jonathan" <jonathan.yong@intel.com> - 2016-08-16 10:10 +0200
      Re: [PATCH v6 1/3] PCI: Add Precision Time Measurement (PTM) support Bjorn Helgaas <helgaas@kernel.org> - 2016-08-16 15:40 +0200
        Re: [PATCH v6 1/3] PCI: Add Precision Time Measurement (PTM) support "Yong, Jonathan" <jonathan.yong@intel.com> - 2016-08-17 11:00 +0200
        Re: [PATCH v6 1/3] PCI: Add Precision Time Measurement (PTM) support Bjorn Helgaas <helgaas@kernel.org> - 2016-08-22 19:10 +0200
          Re: [PATCH v6 1/3] PCI: Add Precision Time Measurement (PTM) support "Yong, Jonathan" <jonathan.yong@intel.com> - 2016-08-23 02:20 +0200

#1463095 — Re: [PATCH v6 1/3] PCI: Add Precision Time Measurement (PTM) support

Hi Jonathan,

On Wed, Aug 10, 2016 at 10:58:22AM +0800, Yong, Jonathan wrote:
> On 06/14/2016 03:05, Bjorn Helgaas wrote:
> > From: Jonathan Yong <jonathan.yong@intel.com>
> > 
> > Add Precision Time Measurement (PTM) support (see PCIe r3.1, sec 6.22).
> > 
> > Enable PTM on PTM Root devices and switch ports.  This does not enable PTM
> > on endpoints.
> > 
> > There currently are no PTM-capable devices on the market, but it is
> > expected to be supported by the Intel Apollo Lake platform.
> > 
> > [bhelgaas: complete rework]
> > Signed-off-by: Jonathan Yong <jonathan.yong@intel.com>
> > Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
> 
> Hi,
> 
> Any updates on the PTM changes?

I guess I was hoping you could test these patches.  Do you have any
way to do that?

I rebased them and pushed them to pci/ptm.  If everything looks good,
I'll merge them for v4.9.

Bjorn

[toc] | [next] | [standalone]

#1463521

On 08/16/2016 02:59, Bjorn Helgaas wrote:
> 
> I guess I was hoping you could test these patches.  Do you have any
> way to do that?
> 

No real hardware with this feature yet, so testing is entirely on software.

> I rebased them and pushed them to pci/ptm.  If everything looks good,
> I'll merge them for v4.9.
> 
> Bjorn
> 

Looks like there is still this compile warning to fix:
http://patchwork.ozlabs.org/patch/634825/

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

#1463804

On Tue, Aug 16, 2016 at 04:04:31PM +0800, Yong, Jonathan wrote:
> On 08/16/2016 02:59, Bjorn Helgaas wrote:
> > 
> > I guess I was hoping you could test these patches.  Do you have any
> > way to do that?
> > 
> 
> No real hardware with this feature yet, so testing is entirely on software.

OK, let me know the results of your software testing with these patches.

> Looks like there is still this compile warning to fix:
> http://patchwork.ozlabs.org/patch/634825/

That report was from June 16, and I think I fixed the warning already.
Or do you still see the warning?  I don't see it myself.

Bjorn

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

#1464422

On 08/16/2016 21:37, Bjorn Helgaas wrote:
>> Looks like there is still this compile warning to fix:
>> http://patchwork.ozlabs.org/patch/634825/
> 
> That report was from June 16, and I think I fixed the warning already.
> Or do you still see the warning?  I don't see it myself.
> 
> Bjorn
> 

If that is fixed, I don't anything else to add.

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

#1467830

On Tue, Aug 16, 2016 at 08:37:15AM -0500, Bjorn Helgaas wrote:
> On Tue, Aug 16, 2016 at 04:04:31PM +0800, Yong, Jonathan wrote:
> > On 08/16/2016 02:59, Bjorn Helgaas wrote:
> > > 
> > > I guess I was hoping you could test these patches.  Do you have any
> > > way to do that?
> > > 
> > 
> > No real hardware with this feature yet, so testing is entirely on software.
> 
> OK, let me know the results of your software testing with these patches.

Do you have some software testing you can do on these patches?

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

#1468198

On 08/23/2016 01:01, Bjorn Helgaas wrote:
> On Tue, Aug 16, 2016 at 08:37:15AM -0500, Bjorn Helgaas wrote:
>> On Tue, Aug 16, 2016 at 04:04:31PM +0800, Yong, Jonathan wrote:
>>> On 08/16/2016 02:59, Bjorn Helgaas wrote:
>>>>
>>>> I guess I was hoping you could test these patches.  Do you have any
>>>> way to do that?
>>>>
>>>
>>> No real hardware with this feature yet, so testing is entirely on software.
>>
>> OK, let me know the results of your software testing with these patches.
> 
> Do you have some software testing you can do on these patches?
> 

Yes, they're entirely synthetic however and do not reflect real hardware.

At the moment, they're hooks to pci_scan_bus and providing a fake config
space for the driver to manipulate, inspected with lspci.

The PTM bits are set properly as far as I can tell.

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web