On Fri, Feb 14, 2025 at 12:37:55PM +0100, Wenjia Zhang wrote: > > > On 14.02.25 10:22, D. Wythe wrote: > >On Thu, Jan 23, 2025 at 09:59:36AM +0800, D. Wythe wrote: > >>This patch aims to introduce BPF injection capabilities for SMC and > >>includes a self-test to ensure code stability. > >> > >>Since the SMC protocol isn't ideal for every situation, especially > >>short-lived ones, most applications can't guarantee the absence of > >>such scenarios. Consequently, applications may need specific strategies > >>to decide whether to use SMC. For example, an application might limit SMC > >>usage to certain IP addresses or ports. > >> > >>To maintain the principle of transparent replacement, we want applications > >>to remain unaffected even if they need specific SMC strategies. In other > >>words, they should not require recompilation of their code. > >> > >>Additionally, we need to ensure the scalability of strategy implementation. > >>While using socket options or sysctl might be straightforward, it could > >>complicate future expansions. > >> > >>Fortunately, BPF addresses these concerns effectively. Users can write > >>their own strategies in eBPF to determine whether to use SMC, and they can > >>easily modify those strategies in the future. > > > >Hi smc folks, @Wenjia @Ian > > > >Is there any feedback regarding this patches ? This series of code has > >gone through multiple rounds of community reviews. However, the parts > >related to SMC, including the new sysctl and ops name, really needs > >your input and acknowledgment. > > > >Additionally, this series includes a bug fix for SMC, which is easily > >reproducible in the BPF CI tests. > > > >Thanks, > >D. Wythe > > > Hi D.Wythe, > > Thanks for the reminder! I have a few higher-priority tasks to > handle first, but I’ll get back to you as soon as I can—hopefully > next week. > > Thanks, > Wenjia Hi Wenjia, Thank you for your reply and explanation! I completely understand that you have higher-priority tasks to handle right now. I just wanted to ensure that this patch isn't overlooked, as it contains important changes and fixes related to SMC. Best wishes, D. Wythe