From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ed1-f45.google.com (mail-ed1-f45.google.com [209.85.208.45]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 45EFF383A3 for ; Wed, 27 Mar 2024 15:10:39 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.45 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1711552244; cv=none; b=ce4FXpHxQABTpn9WFzn3Y+coGYF/PFIi1alzAv/EzoDlFRcNxq+vWe/EKN1lSe0YrL2Gq2jHmWOgU3UOp9lrRMSd8+7/96TJOntKj9p/cKSOiit2WzZ5RrRGMyuc2gwWxpGhNVjEzetS7Ptb3a5U8sbMypl3+01KJAphYinTms4= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1711552244; c=relaxed/simple; bh=xSl+elXw6TGnfDDxTuwZc3OijKW+bN4Ubbs24XJFxWA=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=f5iWksH/Jl2mKv35ITdf3+OMCiunHF9o9kD/dL3BXzlHp4GdzZsftZ9MXyR4gMB4j+LbpeJJf8nsiprnJoEY20ermA5bhgqJRNB2KQBftsLLXDH6NGlnoCxzpyONSsOmh+2Aolxm5gPaD93UY0UYhJ6xk/Nv9P57bOqg79xV1EU= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=suse.com; spf=pass smtp.mailfrom=suse.com; dkim=pass (2048-bit key) header.d=suse.com header.i=@suse.com header.b=JU5CiExG; arc=none smtp.client-ip=209.85.208.45 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=suse.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=suse.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=suse.com header.i=@suse.com header.b="JU5CiExG" Received: by mail-ed1-f45.google.com with SMTP id 4fb4d7f45d1cf-56c12c73ed8so4225303a12.2 for ; Wed, 27 Mar 2024 08:10:39 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=google; t=1711552238; x=1712157038; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=Wg3QzX5qREXmVZsGaqo0PzIOFIXQE1IbveON87ikDwQ=; b=JU5CiExGX2fWxeAmFJzOzhGwjmo9bvN5OJ/5W3s11sF16i8OXUMOe5F/pBm76Go78K hBC6XSl0UIB6l1qrL1CW7g8K/2XbIftE+yGsoUacfC3KOd3MXzrTCcrlgVAo8ZSidJZI B9bYh0AHyTi0Dh75aQhLByihVplja8peRlycoXedc36fWXesh4Qut8qQVrkpIa2miVBC blI0uZ1p0/q9qT4iq8nAj8XB7kPceKzwfPkKjn9NMeLN39iZmbRXwEMIk3IefFw4yEli thw+qo0uqtpTn0QpODta8N3GGOX/5ITLRLURxRNq/RJR7FxSe1dMUHXkrJEHdHZAy9uY P6hg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1711552238; x=1712157038; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=Wg3QzX5qREXmVZsGaqo0PzIOFIXQE1IbveON87ikDwQ=; b=SObFz5ekWwzAtQn4jSJ5+f54YQcS0UojBM9/uFeKZBpMsxsWeFb//TlKOiHu8QfgV0 jyEahgiZWZOJJCRfXFhiFZpqSV+3BBWAtxRoRUrv6vGWT3vDJu5WaC1biSQguxBo1CMc JN2WWQD8iFta007VUPUVAn262nLRFoi3DpHX4TRg5obBSuu4GGpvjMO5IIy3YmCE4Q5O CLH2PYr88SiXZOEkNAFMbjh7jT8R4BUwrFCUHO3zEu/soLdD2JKmGoRtqOP80p9P8VK5 3mc8SNBa/L8riAqUPLKlhJw/+fQy99ntczA8EU/lpsIU7uHh/WhQ3JatL+oXtO7ts9jd 563g== X-Forwarded-Encrypted: i=1; AJvYcCWnm9JUdbwsqcyGeOq18mRXca1JDZXc80sjgYPWHmANLbFAavvWrkXU3qcuac3bHfjW9SujqQc5X1PQw95HMY1bCEBaI54Ox6QO X-Gm-Message-State: AOJu0YzR9UPa/i9CefQ86TxJqN5+m7luvT0RRibvW1wfXnLNRhqMYpVZ PZPn2/nH0nx+MLz3LcHX1WtqB2EDXXSL8yGFZWXKBlbxQ7sRqgxp2uo/8QFBoEk= X-Google-Smtp-Source: AGHT+IEG6gS6qYe0E+lUJL//xh4zBdNakL80EjwnyIFcIwgzZrLzz6D+EK8zTW1O20qHE7z/Fw/K4A== X-Received: by 2002:a50:f61d:0:b0:567:737f:e910 with SMTP id c29-20020a50f61d000000b00567737fe910mr80250edn.3.1711552238517; Wed, 27 Mar 2024 08:10:38 -0700 (PDT) Received: from ?IPV6:2001:a61:1366:6801:b2db:904d:e0b0:46ec? ([2001:a61:1366:6801:b2db:904d:e0b0:46ec]) by smtp.gmail.com with ESMTPSA id dm28-20020a05640222dc00b0056b7ed75a46sm5403800edb.27.2024.03.27.08.10.37 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 27 Mar 2024 08:10:38 -0700 (PDT) Message-ID: <04cfa214-4d45-48b1-87ba-500e3e501977@suse.com> Date: Wed, 27 Mar 2024 16:10:36 +0100 Precedence: bulk X-Mailing-List: linux-usb@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH net-next] usbnet: fix cyclical race on disconnect with work queue To: Sai Krishna Gajula , Oliver Neukum , "davem@davemloft.net" , "edumazet@google.com" , "kuba@kernel.org" , "pabeni@redhat.com" , "netdev@vger.kernel.org" , "linux-usb@vger.kernel.org" , "linux-kernel@vger.kernel.org" Cc: "syzbot+9665bf55b1c828bbcd8a@syzkaller.appspotmail.com" References: <20240321124758.6302-1-oneukum@suse.com> Content-Language: en-US From: Oliver Neukum In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 3/22/24 18:43, Sai Krishna Gajula wrote: > >> -----Original Message----- >> From: Oliver Neukum >> Sent: Thursday, March 21, 2024 6:17 PM >> To: davem@davemloft.net; edumazet@google.com; kuba@kernel.org; >> pabeni@redhat.com; netdev@vger.kernel.org; linux-usb@vger.kernel.org; >> linux-kernel@vger.kernel.org >> Cc: Oliver Neukum ; >> syzbot+9665bf55b1c828bbcd8a@syzkaller.appspotmail.com >> Subject: [PATCH net-next] usbnet: fix cyclical race on disconnect >> with work queue > > This patch seems to be a fix, in that case the subject need to be with [PATCH net] OK > >> >> The work can submit URBs and the URBs can schedule the work. >> This cycle needs to be broken, when a device is to be stopped. >> Use a flag to do so. >> >> Fixes: f29fc259976e9 ("[PATCH] USB: usbnet (1/9) clean up framing") > > Please use correct Fixes: style 'Fixes: <12 chars of sha1> ("")' - ie: 'Fixes: f29fc259976e ("[PATCH] USB: usbnet (1/9) clean up framing")' Ehm, what exactly did I do differently >> --- a/drivers/net/usb/usbnet.c >> +++ b/drivers/net/usb/usbnet.c >> @@ -467,10 +467,12 @@ static enum skb_state defer_bh(struct usbnet *dev, >> struct sk_buff *skb, void usbnet_defer_kevent (struct usbnet *dev, int work) > > space prohibited between function name and open parenthesis '(' I am sorry, but this is the context of the diff. You are not suggesting to mix gratitious format changes into a bug fix, are you? >> diff --git a/include/linux/usb/usbnet.h b/include/linux/usb/usbnet.h index >> 9f08a584d707..d26599faab33 100644 >> --- a/include/linux/usb/usbnet.h >> +++ b/include/linux/usb/usbnet.h >> @@ -76,8 +76,26 @@ struct usbnet { >> # define EVENT_LINK_CHANGE 11 >> # define EVENT_SET_RX_MODE 12 >> # define EVENT_NO_IP_ALIGN 13 >> +/* >> + * this one is special, as it indicates that the device is going away >> + * there are cyclic dependencies between tasklet, timer and bh >> + * that must be broken >> + */ > > Networking block comments don't use an empty /* line, use /* Comment... OK Regards Oliver