netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH net] net: dsa: point out the tail taggers
@ 2020-10-16 16:28 Christian Eggers
  2020-10-16 16:46 ` Florian Fainelli
                   ` (2 more replies)
  0 siblings, 3 replies; 11+ messages in thread
From: Christian Eggers @ 2020-10-16 16:28 UTC (permalink / raw)
  To: Woojung Huh, Vladimir Oltean
  Cc: Microchip Linux Driver Support, Andrew Lunn, Vivien Didelot,
	Florian Fainelli, David S . Miller, Jakub Kicinski, netdev,
	linux-kernel, Christian Eggers

From a  recent commit with the same summary:

"The Marvell 88E6060 uses tag_trailer.c and the KSZ8795, KSZ9477 and
KSZ9893 switches also use tail tags."

Set "tail_tag" to true for KSZ8795 and KSZ9477 which were missing in the
original commit.

Fixes: 7a6ffe764be3 [net] ("net: dsa: point out the tail taggers")
Signed-off-by: Christian Eggers <ceggers@arri.de>
---
 net/dsa/tag_ksz.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/net/dsa/tag_ksz.c b/net/dsa/tag_ksz.c
index 945a9bd5ba35..0a5aa982c60d 100644
--- a/net/dsa/tag_ksz.c
+++ b/net/dsa/tag_ksz.c
@@ -123,6 +123,7 @@ static const struct dsa_device_ops ksz8795_netdev_ops = {
 	.xmit	= ksz8795_xmit,
 	.rcv	= ksz8795_rcv,
 	.overhead = KSZ_INGRESS_TAG_LEN,
+	.tail_tag = true,
 };
 
 DSA_TAG_DRIVER(ksz8795_netdev_ops);
@@ -199,6 +200,7 @@ static const struct dsa_device_ops ksz9477_netdev_ops = {
 	.xmit	= ksz9477_xmit,
 	.rcv	= ksz9477_rcv,
 	.overhead = KSZ9477_INGRESS_TAG_LEN,
+	.tail_tag = true,
 };
 
 DSA_TAG_DRIVER(ksz9477_netdev_ops);
-- 
Christian Eggers
Embedded software developer

Arnold & Richter Cine Technik GmbH & Co. Betriebs KG
Sitz: Muenchen - Registergericht: Amtsgericht Muenchen - Handelsregisternummer: HRA 57918
Persoenlich haftender Gesellschafter: Arnold & Richter Cine Technik GmbH
Sitz: Muenchen - Registergericht: Amtsgericht Muenchen - Handelsregisternummer: HRB 54477
Geschaeftsfuehrer: Dr. Michael Neuhaeuser; Stephan Schenk; Walter Trauninger; Markus Zeiler


^ permalink raw reply related	[flat|nested] 11+ messages in thread

* Re: [PATCH net] net: dsa: point out the tail taggers
  2020-10-16 16:28 [PATCH net] net: dsa: point out the tail taggers Christian Eggers
@ 2020-10-16 16:46 ` Florian Fainelli
  2020-10-16 16:56 ` Vladimir Oltean
  2020-10-16 17:25 ` kernel test robot
  2 siblings, 0 replies; 11+ messages in thread
From: Florian Fainelli @ 2020-10-16 16:46 UTC (permalink / raw)
  To: Christian Eggers, Woojung Huh, Vladimir Oltean
  Cc: Microchip Linux Driver Support, Andrew Lunn, Vivien Didelot,
	David S . Miller, Jakub Kicinski, netdev, linux-kernel

On 10/16/20 9:28 AM, Christian Eggers wrote:
> From a  recent commit with the same summary:
> 
> "The Marvell 88E6060 uses tag_trailer.c and the KSZ8795, KSZ9477 and
> KSZ9893 switches also use tail tags."
> 
> Set "tail_tag" to true for KSZ8795 and KSZ9477 which were missing in the
> original commit.
> 
> Fixes: 7a6ffe764be3 [net] ("net: dsa: point out the tail taggers")
> Signed-off-by: Christian Eggers <ceggers@arri.de>

Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
-- 
Florian

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH net] net: dsa: point out the tail taggers
  2020-10-16 16:28 [PATCH net] net: dsa: point out the tail taggers Christian Eggers
  2020-10-16 16:46 ` Florian Fainelli
@ 2020-10-16 16:56 ` Vladimir Oltean
  2020-10-16 17:25 ` kernel test robot
  2 siblings, 0 replies; 11+ messages in thread
From: Vladimir Oltean @ 2020-10-16 16:56 UTC (permalink / raw)
  To: Christian Eggers
  Cc: Woojung Huh, Microchip Linux Driver Support, Andrew Lunn,
	Vivien Didelot, Florian Fainelli, David S . Miller,
	Jakub Kicinski, netdev, linux-kernel

Hi Christian,

On Fri, Oct 16, 2020 at 06:28:00PM +0200, Christian Eggers wrote:
> From a  recent commit with the same summary:
> 
> "The Marvell 88E6060 uses tag_trailer.c and the KSZ8795, KSZ9477 and
> KSZ9893 switches also use tail tags."
> 
> Set "tail_tag" to true for KSZ8795 and KSZ9477 which were missing in the
> original commit.
> 
> Fixes: 7a6ffe764be3 [net] ("net: dsa: point out the tail taggers")
> Signed-off-by: Christian Eggers <ceggers@arri.de>
> ---

The idea is perfect but the commit isn't.

First of all, put this in your .gitconfig.

[core]
	abbrev = 12
[pretty]
	fixes = Fixes: %h (\"%s\")

Now if you run
"git show 7a6ffe764be35af0527d8cfd047945e8f8797ddf --pretty=fixes",
you'll see:

Fixes: 7a6ffe764be3 ("net: dsa: point out the tail taggers")

Notice how there's no [net] tag?
People complain when the format of the Fixes: tag is not standardized.

Secondly, can you please come up with a commit description that is
_different_ from the commit you're fixing? As a backporter I would hate
to have 2 commits with the same title, I would surely mess them up.

How about:
net: dsa: tag_ksz: KSZ8795 and KSZ9477 also use tail tags

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH net] net: dsa: point out the tail taggers
  2020-10-16 16:28 [PATCH net] net: dsa: point out the tail taggers Christian Eggers
  2020-10-16 16:46 ` Florian Fainelli
  2020-10-16 16:56 ` Vladimir Oltean
@ 2020-10-16 17:25 ` kernel test robot
  2020-10-16 17:33   ` Vladimir Oltean
  2 siblings, 1 reply; 11+ messages in thread
From: kernel test robot @ 2020-10-16 17:25 UTC (permalink / raw)
  To: Christian Eggers, Woojung Huh, Vladimir Oltean
  Cc: kbuild-all, Microchip Linux Driver Support, Andrew Lunn,
	Vivien Didelot, Florian Fainelli, David S . Miller,
	Jakub Kicinski, netdev, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 2871 bytes --]

Hi Christian,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on net/master]

url:    https://github.com/0day-ci/linux/commits/Christian-Eggers/net-dsa-point-out-the-tail-taggers/20201017-003007
base:   https://git.kernel.org/pub/scm/linux/kernel/git/davem/net.git 2ecbc1f684482b4ed52447a39903bd9b0f222898
config: xtensa-allyesconfig (attached as .config)
compiler: xtensa-linux-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # https://github.com/0day-ci/linux/commit/aaa07cad29bf365264beb2c2e2668db83ca31923
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Christian-Eggers/net-dsa-point-out-the-tail-taggers/20201017-003007
        git checkout aaa07cad29bf365264beb2c2e2668db83ca31923
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=xtensa 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All error/warnings (new ones prefixed by >>):

>> net/dsa/tag_ksz.c:126:3: error: 'const struct dsa_device_ops' has no member named 'tail_tag'
     126 |  .tail_tag = true,
         |   ^~~~~~~~
>> net/dsa/tag_ksz.c:126:14: warning: initialization of 'const char *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
     126 |  .tail_tag = true,
         |              ^~~~
   net/dsa/tag_ksz.c:126:14: note: (near initialization for 'ksz8795_netdev_ops.name')
>> net/dsa/tag_ksz.c:126:14: warning: initialized field overwritten [-Woverride-init]
   net/dsa/tag_ksz.c:126:14: note: (near initialization for 'ksz8795_netdev_ops.name')
   net/dsa/tag_ksz.c:203:3: error: 'const struct dsa_device_ops' has no member named 'tail_tag'
     203 |  .tail_tag = true,
         |   ^~~~~~~~
   net/dsa/tag_ksz.c:203:14: warning: initialization of 'const char *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
     203 |  .tail_tag = true,
         |              ^~~~
   net/dsa/tag_ksz.c:203:14: note: (near initialization for 'ksz9477_netdev_ops.name')
   net/dsa/tag_ksz.c:203:14: warning: initialized field overwritten [-Woverride-init]
   net/dsa/tag_ksz.c:203:14: note: (near initialization for 'ksz9477_netdev_ops.name')

vim +126 net/dsa/tag_ksz.c

   119	
   120	static const struct dsa_device_ops ksz8795_netdev_ops = {
   121		.name	= "ksz8795",
   122		.proto	= DSA_TAG_PROTO_KSZ8795,
   123		.xmit	= ksz8795_xmit,
   124		.rcv	= ksz8795_rcv,
   125		.overhead = KSZ_INGRESS_TAG_LEN,
 > 126		.tail_tag = true,
   127	};
   128	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 65044 bytes --]

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH net] net: dsa: point out the tail taggers
  2020-10-16 17:25 ` kernel test robot
@ 2020-10-16 17:33   ` Vladimir Oltean
  2020-10-16 20:14     ` Andrew Lunn
  0 siblings, 1 reply; 11+ messages in thread
From: Vladimir Oltean @ 2020-10-16 17:33 UTC (permalink / raw)
  To: kernel test robot
  Cc: Christian Eggers, Woojung Huh, kbuild-all,
	Microchip Linux Driver Support, Andrew Lunn, Vivien Didelot,
	Florian Fainelli, David S . Miller, Jakub Kicinski, netdev,
	linux-kernel

On Sat, Oct 17, 2020 at 01:25:08AM +0800, kernel test robot wrote:
> Hi Christian,
>
> Thank you for the patch! Yet something to improve:
>
> [auto build test ERROR on net/master]
>
> url:    https://github.com/0day-ci/linux/commits/Christian-Eggers/net-dsa-point-out-the-tail-taggers/20201017-003007
> base:   https://git.kernel.org/pub/scm/linux/kernel/git/davem/net.git 2ecbc1f684482b4ed52447a39903bd9b0f222898
> config: xtensa-allyesconfig (attached as .config)
> compiler: xtensa-linux-gcc (GCC) 9.3.0
> reproduce (this is a W=1 build):
>         wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>         chmod +x ~/bin/make.cross
>         # https://github.com/0day-ci/linux/commit/aaa07cad29bf365264beb2c2e2668db83ca31923
>         git remote add linux-review https://github.com/0day-ci/linux
>         git fetch --no-tags linux-review Christian-Eggers/net-dsa-point-out-the-tail-taggers/20201017-003007
>         git checkout aaa07cad29bf365264beb2c2e2668db83ca31923
>         # save the attached .config to linux build tree
>         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=xtensa
>
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
>
> All error/warnings (new ones prefixed by >>):
>
> >> net/dsa/tag_ksz.c:126:3: error: 'const struct dsa_device_ops' has no member named 'tail_tag'
>      126 |  .tail_tag = true,
>          |   ^~~~~~~~
> >> net/dsa/tag_ksz.c:126:14: warning: initialization of 'const char *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
>      126 |  .tail_tag = true,
>          |              ^~~~
>    net/dsa/tag_ksz.c:126:14: note: (near initialization for 'ksz8795_netdev_ops.name')
> >> net/dsa/tag_ksz.c:126:14: warning: initialized field overwritten [-Woverride-init]
>    net/dsa/tag_ksz.c:126:14: note: (near initialization for 'ksz8795_netdev_ops.name')
>    net/dsa/tag_ksz.c:203:3: error: 'const struct dsa_device_ops' has no member named 'tail_tag'
>      203 |  .tail_tag = true,
>          |   ^~~~~~~~
>    net/dsa/tag_ksz.c:203:14: warning: initialization of 'const char *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
>      203 |  .tail_tag = true,
>          |              ^~~~
>    net/dsa/tag_ksz.c:203:14: note: (near initialization for 'ksz9477_netdev_ops.name')
>    net/dsa/tag_ksz.c:203:14: warning: initialized field overwritten [-Woverride-init]
>    net/dsa/tag_ksz.c:203:14: note: (near initialization for 'ksz9477_netdev_ops.name')
>
> vim +126 net/dsa/tag_ksz.c
>
>    119
>    120	static const struct dsa_device_ops ksz8795_netdev_ops = {
>    121		.name	= "ksz8795",
>    122		.proto	= DSA_TAG_PROTO_KSZ8795,
>    123		.xmit	= ksz8795_xmit,
>    124		.rcv	= ksz8795_rcv,
>    125		.overhead = KSZ_INGRESS_TAG_LEN,
>  > 126		.tail_tag = true,
>    127	};
>    128
>
> ---
> 0-DAY CI Kernel Test Service, Intel Corporation
> https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

Is the test bot being a bit "slow" today?
https://git.kernel.org/pub/scm/linux/kernel/git/netdev/net.git/tree/include/net/dsa.h#n93?id=2ecbc1f684482b4ed52447a39903bd9b0f222898

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH net] net: dsa: point out the tail taggers
  2020-10-16 17:33   ` Vladimir Oltean
@ 2020-10-16 20:14     ` Andrew Lunn
  2020-10-16 20:19       ` Vladimir Oltean
  0 siblings, 1 reply; 11+ messages in thread
From: Andrew Lunn @ 2020-10-16 20:14 UTC (permalink / raw)
  To: Vladimir Oltean
  Cc: kernel test robot, Christian Eggers, Woojung Huh, kbuild-all,
	Microchip Linux Driver Support, Vivien Didelot, Florian Fainelli,
	David S . Miller, Jakub Kicinski, netdev, linux-kernel

On Fri, Oct 16, 2020 at 08:33:17PM +0300, Vladimir Oltean wrote:
> On Sat, Oct 17, 2020 at 01:25:08AM +0800, kernel test robot wrote:
> > Hi Christian,
> >
> > Thank you for the patch! Yet something to improve:
> >
> > [auto build test ERROR on net/master]
> >
> > url:    https://github.com/0day-ci/linux/commits/Christian-Eggers/net-dsa-point-out-the-tail-taggers/20201017-003007
> > base:   https://git.kernel.org/pub/scm/linux/kernel/git/davem/net.git 2ecbc1f684482b4ed52447a39903bd9b0f222898

> Is the test bot being a bit "slow" today?

It is using the net.git commit from yesterday afternoon. net-next got
merged into net yesterday evening, so it is a bit behind, but not too
far behind.

       Andrew

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH net] net: dsa: point out the tail taggers
  2020-10-16 20:14     ` Andrew Lunn
@ 2020-10-16 20:19       ` Vladimir Oltean
  2020-10-16 21:03         ` Andrew Lunn
  0 siblings, 1 reply; 11+ messages in thread
From: Vladimir Oltean @ 2020-10-16 20:19 UTC (permalink / raw)
  To: Andrew Lunn
  Cc: kernel test robot, Christian Eggers, Woojung Huh, kbuild-all,
	Microchip Linux Driver Support, Vivien Didelot, Florian Fainelli,
	David S . Miller, Jakub Kicinski, netdev, linux-kernel

On Fri, Oct 16, 2020 at 10:14:28PM +0200, Andrew Lunn wrote:
> On Fri, Oct 16, 2020 at 08:33:17PM +0300, Vladimir Oltean wrote:
> > On Sat, Oct 17, 2020 at 01:25:08AM +0800, kernel test robot wrote:
> > > Hi Christian,
> > >
> > > Thank you for the patch! Yet something to improve:
> > >
> > > [auto build test ERROR on net/master]
> > >
> > > url:    https://github.com/0day-ci/linux/commits/Christian-Eggers/net-dsa-point-out-the-tail-taggers/20201017-003007
> > > base:   https://git.kernel.org/pub/scm/linux/kernel/git/davem/net.git 2ecbc1f684482b4ed52447a39903bd9b0f222898
> 
> > Is the test bot being a bit "slow" today?
> 
> It is using the net.git commit from yesterday afternoon. net-next got
> merged into net yesterday evening, so it is a bit behind, but not too
> far behind.

Exactly. The sha1sum it uses _does_ contain the tail_tag member inside
struct dsa_device_ops. What's it complaining about?

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH net] net: dsa: point out the tail taggers
  2020-10-16 20:19       ` Vladimir Oltean
@ 2020-10-16 21:03         ` Andrew Lunn
  2020-10-16 21:16           ` Vladimir Oltean
  0 siblings, 1 reply; 11+ messages in thread
From: Andrew Lunn @ 2020-10-16 21:03 UTC (permalink / raw)
  To: Vladimir Oltean
  Cc: kernel test robot, Christian Eggers, Woojung Huh, kbuild-all,
	Microchip Linux Driver Support, Vivien Didelot, Florian Fainelli,
	David S . Miller, Jakub Kicinski, netdev, linux-kernel

On Fri, Oct 16, 2020 at 11:19:30PM +0300, Vladimir Oltean wrote:
> On Fri, Oct 16, 2020 at 10:14:28PM +0200, Andrew Lunn wrote:
> > On Fri, Oct 16, 2020 at 08:33:17PM +0300, Vladimir Oltean wrote:
> > > On Sat, Oct 17, 2020 at 01:25:08AM +0800, kernel test robot wrote:
> > > > Hi Christian,
> > > >
> > > > Thank you for the patch! Yet something to improve:
> > > >
> > > > [auto build test ERROR on net/master]
> > > >
> > > > url:    https://github.com/0day-ci/linux/commits/Christian-Eggers/net-dsa-point-out-the-tail-taggers/20201017-003007
> > > > base:   https://git.kernel.org/pub/scm/linux/kernel/git/davem/net.git 2ecbc1f684482b4ed52447a39903bd9b0f222898
> > 
> > > Is the test bot being a bit "slow" today?
> > 
> > It is using the net.git commit from yesterday afternoon. net-next got
> > merged into net yesterday evening, so it is a bit behind, but not too
> > far behind.
> 
> Exactly. The sha1sum it uses _does_ contain the tail_tag member inside
> struct dsa_device_ops. What's it complaining about?

2ecbc1f684482b4ed52447a39903bd9b0f222898 does not have net-next, as
far as i see, and tail_tag only hit net when net-next was merged into
net.

Or i'm reading the git history wrong.

	Andrew

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH net] net: dsa: point out the tail taggers
  2020-10-16 21:03         ` Andrew Lunn
@ 2020-10-16 21:16           ` Vladimir Oltean
  2020-10-16 21:33             ` Vladimir Oltean
  0 siblings, 1 reply; 11+ messages in thread
From: Vladimir Oltean @ 2020-10-16 21:16 UTC (permalink / raw)
  To: Andrew Lunn
  Cc: kernel test robot, Christian Eggers, Woojung Huh, kbuild-all,
	Microchip Linux Driver Support, Vivien Didelot, Florian Fainelli,
	David S . Miller, Jakub Kicinski, netdev, linux-kernel

On Fri, Oct 16, 2020 at 11:03:18PM +0200, Andrew Lunn wrote:
> 2ecbc1f684482b4ed52447a39903bd9b0f222898 does not have net-next, as
> far as i see,

Not sure what you mean by that.

> and tail_tag only hit net when net-next was merged into
> net.

net-next is only merged into net via Linus Torvalds, as far as I
understand.

> Or i'm reading the git history wrong.

So the only plausible scenario is that yesterday's 'net/master' did not
contain 2ecbc1f684482b4ed52447a39903bd9b0f222898, but today it does, due
to Linus Torvalds merging net-next and Jakub merging that merge into net.

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH net] net: dsa: point out the tail taggers
  2020-10-16 21:16           ` Vladimir Oltean
@ 2020-10-16 21:33             ` Vladimir Oltean
  2020-10-16 22:56               ` Jakub Kicinski
  0 siblings, 1 reply; 11+ messages in thread
From: Vladimir Oltean @ 2020-10-16 21:33 UTC (permalink / raw)
  To: Andrew Lunn
  Cc: kernel test robot, Christian Eggers, Woojung Huh, kbuild-all,
	Microchip Linux Driver Support, Vivien Didelot, Florian Fainelli,
	David S . Miller, Jakub Kicinski, netdev, linux-kernel

On Sat, Oct 17, 2020 at 12:16:28AM +0300, Vladimir Oltean wrote:
> On Fri, Oct 16, 2020 at 11:03:18PM +0200, Andrew Lunn wrote:
> > 2ecbc1f684482b4ed52447a39903bd9b0f222898 does not have net-next, as
> > far as i see,
> 
> Not sure what you mean by that.

Ah, I do understand what you mean now. In git, that is what I see as
well. But in my cgit link, why would tail_tag be there?
https://git.kernel.org/pub/scm/linux/kernel/git/netdev/net.git/tree/include/net/dsa.h#n93?id=2ecbc1f684482b4ed52447a39903bd9b0f222898
I think either cgit is plainly dumb at showing the kernel tree at a
particular commit, or I'm plainly incapable of using it.

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH net] net: dsa: point out the tail taggers
  2020-10-16 21:33             ` Vladimir Oltean
@ 2020-10-16 22:56               ` Jakub Kicinski
  0 siblings, 0 replies; 11+ messages in thread
From: Jakub Kicinski @ 2020-10-16 22:56 UTC (permalink / raw)
  To: Vladimir Oltean
  Cc: Andrew Lunn, kernel test robot, Christian Eggers, Woojung Huh,
	kbuild-all, Microchip Linux Driver Support, Vivien Didelot,
	Florian Fainelli, David S . Miller, netdev, linux-kernel

On Sat, 17 Oct 2020 00:33:02 +0300 Vladimir Oltean wrote:
> On Sat, Oct 17, 2020 at 12:16:28AM +0300, Vladimir Oltean wrote:
> > On Fri, Oct 16, 2020 at 11:03:18PM +0200, Andrew Lunn wrote:  
> > > 2ecbc1f684482b4ed52447a39903bd9b0f222898 does not have net-next, as
> > > far as i see,  
> > 
> > Not sure what you mean by that.  
> 
> Ah, I do understand what you mean now. In git, that is what I see as
> well. But in my cgit link, why would tail_tag be there?
> https://git.kernel.org/pub/scm/linux/kernel/git/netdev/net.git/tree/include/net/dsa.h#n93?id=2ecbc1f684482b4ed52447a39903bd9b0f222898
> I think either cgit is plainly dumb at showing the kernel tree at a
> particular commit, or I'm plainly incapable of using it.

The link is bamboozled.

The #n93 needs to be after the ? parameters.

Like this:

https://git.kernel.org/pub/scm/linux/kernel/git/netdev/net.git/tree/include/net/dsa.h?id=2ecbc1f684482b4ed52447a39903bd9b0f222898#n86

^ permalink raw reply	[flat|nested] 11+ messages in thread

end of thread, other threads:[~2020-10-16 22:56 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-16 16:28 [PATCH net] net: dsa: point out the tail taggers Christian Eggers
2020-10-16 16:46 ` Florian Fainelli
2020-10-16 16:56 ` Vladimir Oltean
2020-10-16 17:25 ` kernel test robot
2020-10-16 17:33   ` Vladimir Oltean
2020-10-16 20:14     ` Andrew Lunn
2020-10-16 20:19       ` Vladimir Oltean
2020-10-16 21:03         ` Andrew Lunn
2020-10-16 21:16           ` Vladimir Oltean
2020-10-16 21:33             ` Vladimir Oltean
2020-10-16 22:56               ` Jakub Kicinski

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).