[ovs-dev] [PATCH] ofproto: Fix mod flow table name not to take effect.

2024-04-03 Thread Yuhao zhou via dev
From: "zhouyuhao.philozhou" When mod a flow table's name with table's prefix name, there will be no change. Because when check whether the new and old name are the same, only compare the length of the new name. Case: table 10: "good" There will be no change if mod the table's name with "g"

Re: [ovs-dev] [PATCH] ofproto: Fix mod flow table name not to take effect.

2024-03-15 Thread Ilya Maximets
On 3/16/24 01:44, Ilya Maximets wrote: > On 3/8/24 09:32, Yuhao zhou via dev wrote: >> From: "zhouyuhao.philozhou" >> >> When mod a flow table's name with table's prefix name, there >> will be no change. Because when check whether the new and old >> name are the same, only compare the length of

Re: [ovs-dev] [PATCH] ofproto: Fix mod flow table name not to take effect.

2024-03-15 Thread Ilya Maximets
On 3/8/24 09:32, Yuhao zhou via dev wrote: > From: "zhouyuhao.philozhou" > > When mod a flow table's name with table's prefix name, there > will be no change. Because when check whether the new and old > name are the same, only compare the length of the new name. > > Case: > table 10: "good"

[ovs-dev] [PATCH] ofproto: Fix mod flow table name not to take effect.

2024-03-08 Thread Yuhao zhou via dev
From: "zhouyuhao.philozhou" When mod a flow table's name with table's prefix name, there will be no change. Because when check whether the new and old name are the same, only compare the length of the new name. Case: table 10: "good" There will be no change if mod the table's name with "g"

[ovs-dev] [PATCH] ofproto: Fix mod flow table name not to take effect.

2024-03-08 Thread Yuhao zhou via dev
From: "zhouyuhao.philozhou" When mod a flow table's name with table's prefix name, there will be no change. Because when check whether the new and old name are the same, only compare the length of the new name. Case: table 10: "good" There will be no change if mod the table's name with "g"

Re: [ovs-dev] [PATCH] ofproto: Fix mod flow table name not to take effect

2024-03-07 Thread 0-day Robot
Bleep bloop. Greetings Yuhao zhou, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: The subject summary should end with a dot. Subject: ofproto: Fix mod flow table name

[ovs-dev] [PATCH] ofproto: Fix mod flow table name not to take effect

2024-03-07 Thread Yuhao zhou via dev
From: "zhouyuhao.philozhou" When mod a flow table's name with table's prefix name, there will be no change. Because when check whether the new and old name are the same, only compare the length of the new name. Case: table 10: "good" There will be no change if mod the table's name with "g"

Re: [ovs-dev] [PATCH] ofproto: Fix mod flow table name not to take effect. When mod a flow table's name with table's prefix name, there will be no change.

2024-03-07 Thread 0-day Robot
Bleep bloop. Greetings Yuhao zhou, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: The subject, ': ', is over 70 characters, i.e., 132. Subject: ofproto: Fix mod flow

[ovs-dev] [PATCH] ofproto: Fix mod flow table name not to take effect. When mod a flow table's name with table's prefix name, there will be no change.

2024-03-07 Thread Yuhao zhou via dev
From: "zhouyuhao.philozhou" Because when check whether the new and old name are the same, only compare the length of the new name. Case: table 10: "good" There will be no change if mod the table's name with "g" "go" "goo". Signed-off-by: zhouyuhao.philozhou --- ofproto/ofproto.c | 4