> > -----Original Message----- > > From: Allan W. Nielsen <allan.niel...@microchip.com> > > Sent: Tuesday, August 13, 2019 2:16 PM > > To: Y.b. Lu <yangbo...@nxp.com> > > Cc: firstname.lastname@example.org; David S . Miller <da...@davemloft.net>; > > Alexandre Belloni <alexandre.bell...@bootlin.com>; Microchip Linux Driver > > Support <unglinuxdri...@microchip.com> > > Subject: Re: [PATCH 3/3] ocelot_ace: fix action of trap > > > > The 08/13/2019 02:12, Y.b. Lu wrote: > > > > -----Original Message----- > > > > From: Allan W. Nielsen <allan.niel...@microchip.com> > > > > Sent: Monday, August 12, 2019 8:32 PM > > > > To: Y.b. Lu <yangbo...@nxp.com> > > > > Cc: email@example.com; David S . Miller <da...@davemloft.net>; > > > > Alexandre Belloni <alexandre.bell...@bootlin.com>; Microchip Linux > > > > Driver Support <unglinuxdri...@microchip.com> > > > > Subject: Re: [PATCH 3/3] ocelot_ace: fix action of trap > > > > > > > > The 08/12/2019 18:48, Yangbo Lu wrote: > > > > > The trap action should be copying the frame to CPU and dropping it > > > > > for forwarding, but current setting was just copying frame to CPU. > > > > > > > > Are there any actions which do a "copy-to-cpu" and still forward the > > > > frame in HW? > > > > > > [Y.b. Lu] We're using Felix switch whose code hadn't been accepted by > > upstream. > > > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatc > > > > > hwork.ozlabs.org%2Fproject%2Fnetdev%2Flist%2F%3Fseries%3D115399%26s > > tat > > > > > e%3D*&data=02%7C01%7Cyangbo.lu%40nxp.com%7C42cd202cb17b45 > > 69821708d > > > > > 71fb5c5de%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C6370127 > > 37899910 > > > > > 736&sdata=QnsDaWPHK9rb0XWg%2BduYEha6fuYSlv4YZdsu5f4kbfc%3D > > &res > > > erved=0 > > > > > > I'd like to trap all IEEE 1588 PTP Ethernet frames to CPU through etype > > 0x88f7. > > > When I used current TRAP option, I found the frames were not only copied > > > to > > CPU, but also forwarded to other ports. > > > So I just made the TRAP option same with DROP option except enabling > > CPU_COPY_ENA in the patch. > > This is still wrong to do - and it will not work for Ocelot (and I doubt it > > will > > work for your Felix target). > > > > The policer setting in the drop action ensure that the frame is dropped > > even if > > other pipe-line steps in the switch has set the copy-to-cpu flag. > > > > I think you can fix this patch my just clearing the port mask, and not set > > the > > policer. > > [Y.b. Lu] Sorry. I missed your previous comments on the TRAP action. > With my configuration in the patch, it indeed worked. Maybe it was because > "the CPU port is not touched by MASK_MODE" which I saw in RM. Okay. If this is working, then you should properly test and see if the DROP action is working on your target. I do not have access to the SoC which incldues Felix, so I cannot check.
> I will try your suggestion too. It sound more proper. Sounds good - thanks /Allan