{"thread":{"id":"44541","subject":"RE: [char-misc-next] mei: request async autosuspend at the end of enumeration","startedAt":"2016-11-24T16:11:40Z","lastAt":"2016-11-26T13:03:01Z","messageCount":6,"participants":["Winkler, Tomas","Jeff King","Jakub Narębski"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"306406","messageId":"5B8DA87D05A7694D9FA63FD143655C1B5433132E@hasmsx108.ger.corp.intel.com","threadId":"44541","inReplyTo":"1479987242-32050-1-git-send-email-tomas.winkler@intel.com","subject":"RE: [char-misc-next] mei: request async autosuspend at the end of enumeration","fromName":"Winkler, Tomas","fromEmail":"tomas.winkler@intel.com","sentAt":"2016-11-24T16:10:49Z","receivedAt":"2016-11-24T16:11:40Z","isPatch":false,"sender":{"key":"tomas.winkler@intel.com","avatar":null},"body":"> \n> From: Alexander Usyskin \n> \n> pm_runtime_autosuspend can take synchronous or asynchronous paths,\n> Because we are calling pm_runtime_mark_last_busy just before this most of\n> the cases it takes the asynchronous way. However, when the FW or driver\n> resets during already running runtime suspend, the call will result in calling to\n> the driver's rpm callback and results in a deadlock on device_lock.\n> The simplest fix is to replace pm_runtime_autosuspend with asynchronous\n> pm_request_autosuspend.\n> \n> Cc: # 4.4+\n\nLooks like git send-email is not able to parse this address correctly though this is suggested format by Documentation/stable_kernel_rules.txt.\nCreate wrong address If git parsers is used : 'stable@vger.kernel.org#4.4+' \n\nSomething like s/#.*$// is needed before parsing Cc: \n\nThanks\nTomas \n\n"},{"id":"306411","messageId":"20161124204316.sqxjpc7uzbrzhvho@sigill.intra.peff.net","threadId":"44541","inReplyTo":"5B8DA87D05A7694D9FA63FD143655C1B5433132E@hasmsx108.ger.corp.intel.com","subject":"Re: [char-misc-next] mei: request async autosuspend at the end of enumeration","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2016-11-24T20:43:16Z","receivedAt":"2016-11-24T20:43:31Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Thu, Nov 24, 2016 at 04:10:49PM +0000, Winkler, Tomas wrote:\n\n> > Cc: # 4.4+\n> \n> Looks like git send-email is not able to parse this address correctly\n> though this is suggested format by Documentation/stable_kernel_rules.txt.\n> Create wrong address If git parsers is used : 'stable@vger.kernel.org#4.4+' \n> \n> Something like s/#.*$// is needed before parsing Cc:\n\nThis should be fixed by e3fdbcc8e (parse_mailboxes: accept extra text\nafter <...> address, 2016-10-13), which will be released next week as\npart of v2.11. As a workaround, you can also install the Mail::Address\nperl module. See [1] for the gory details.\n\n-Peff\n\n[1] http://public-inbox.org/git/41164484-309b-bfff-ddbb-55153495d41a@lwfinger.net/\n"},{"id":"306418","messageId":"5B8DA87D05A7694D9FA63FD143655C1B54331476@hasmsx108.ger.corp.intel.com","threadId":"44541","inReplyTo":"20161124204316.sqxjpc7uzbrzhvho@sigill.intra.peff.net","subject":"RE: [char-misc-next] mei: request async autosuspend at the end of enumeration","fromName":"Winkler, Tomas","fromEmail":"tomas.winkler@intel.com","sentAt":"2016-11-24T22:37:14Z","receivedAt":"2016-11-24T22:37:26Z","isPatch":false,"sender":{"key":"tomas.winkler@intel.com","avatar":null},"body":"> \n> On Thu, Nov 24, 2016 at 04:10:49PM +0000, Winkler, Tomas wrote:\n> \n> > > Cc: # 4.4+\n> >\n> > Looks like git send-email is not able to parse this address correctly\n> > though this is suggested format by Documentation/stable_kernel_rules.txt.\n> > Create wrong address If git parsers is used : 'stable@vger.kernel.org#4.4+'\n> >\n> > Something like s/#.*$// is needed before parsing Cc:\n> \n> This should be fixed by e3fdbcc8e (parse_mailboxes: accept extra text after\n> <...> address, 2016-10-13), which will be released next week as part of v2.11. As\n> a workaround, you can also install the Mail::Address perl module. See [1] for\n> the gory details.\n\nThanks for update, I failed to understand from the thread though what decision was actually applied, I will look at the patch itself.\nI've tried to install Mail::Address and it fixes the actual address, but it appends the # 4.4+ suffix into the name \nas also mentioned in the thread, which doesn't fit the stable rules doc.\n\nSo far I've fixed the particular issue with the simple substitute above, till the real fix is released.\n\nMaybe the stable kernel maintainers should also be pulled into the conversation. \n\nThanks\nTomas \n\n"},{"id":"306420","messageId":"20161125031425.gefijvssvygp6pl4@sigill.intra.peff.net","threadId":"44541","inReplyTo":"5B8DA87D05A7694D9FA63FD143655C1B54331476@hasmsx108.ger.corp.intel.com","subject":"Re: [char-misc-next] mei: request async autosuspend at the end of enumeration","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2016-11-25T03:14:26Z","receivedAt":"2016-11-25T03:15:07Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Thu, Nov 24, 2016 at 10:37:14PM +0000, Winkler, Tomas wrote:\n\n> > > > Cc: # 4.4+\n> > >\n> > > Looks like git send-email is not able to parse this address correctly\n> > > though this is suggested format by Documentation/stable_kernel_rules.txt.\n> > > Create wrong address If git parsers is used : 'stable@vger.kernel.org#4.4+'\n> > >\n> > > Something like s/#.*$// is needed before parsing Cc:\n> > \n> > This should be fixed by e3fdbcc8e (parse_mailboxes: accept extra text after\n> > <...> address, 2016-10-13), which will be released next week as part of v2.11. As\n> > a workaround, you can also install the Mail::Address perl module. See [1] for\n> > the gory details.\n> \n> Thanks for update, I failed to understand from the thread though what\n> decision was actually applied, I will look at the patch itself. I've\n> tried to install Mail::Address and it fixes the actual address, but it\n> appends the # 4.4+ suffix into the name as also mentioned in the\n> thread, which doesn't fit the stable rules doc.\n\nThe patch just brings parity to the Mail::Address behavior and git's\nfallback parser, so that you don't end up with the broken\nstable@vger.kernel.org#4.4+ address. Instead, that content goes into the\nname part of the address.\n\nIt sounds like you want the \"# 4.4+\" to be dropped entirely in the\nrfc822 header. It looks like send-email used to do that, but stopped in\nb1c8a11c8 (send-email: allow multiple emails using --cc, --to and --bcc,\n2015-06-30).\n\nSo perhaps there are further fixes required, but it's hard to know. The\ninput isn't a valid rfc822 header, so it's not entirely clear what the\noutput is supposed to be. I can buy either \"drop it completely\" or\n\"stick it in the name field of the cc header\" as reasonable.\n\n-Peff\n"},{"id":"306440","messageId":"e11d28d3-c1b5-2c04-643f-0b3bd96cb4d3@gmail.com","threadId":"44541","inReplyTo":"20161125031425.gefijvssvygp6pl4@sigill.intra.peff.net","subject":"Re: [char-misc-next] mei: request async autosuspend at the end of enumeration","fromName":"Jakub Narębski","fromEmail":"jnareb@gmail.com","sentAt":"2016-11-25T20:33:42Z","receivedAt":"2016-11-25T20:34:02Z","isPatch":false,"sender":{"key":"jnareb@gmail.com","avatar":"https://avatars.githubusercontent.com/u/2706?v=4"},"body":"W dniu 25.11.2016 o 04:14, Jeff King pisze:\n> On Thu, Nov 24, 2016 at 10:37:14PM +0000, Winkler, Tomas wrote:\n> \n>>>>> Cc: # 4.4+\n>>>>\n>>>> Looks like git send-email is not able to parse this address correctly\n>>>> though this is suggested format by Documentation/stable_kernel_rules.txt.\n>>>> Create wrong address If git parsers is used : 'stable@vger.kernel.org#4.4+'\n[...]\n\n> The patch just brings parity to the Mail::Address behavior and git's\n> fallback parser, so that you don't end up with the broken\n> stable@vger.kernel.org#4.4+ address. Instead, that content goes into the\n> name part of the address.\n> \n> It sounds like you want the \"# 4.4+\" to be dropped entirely in the\n> rfc822 header. It looks like send-email used to do that, but stopped in\n> b1c8a11c8 (send-email: allow multiple emails using --cc, --to and --bcc,\n> 2015-06-30).\n> \n> So perhaps there are further fixes required, but it's hard to know. The\n> input isn't a valid rfc822 header, so it's not entirely clear what the\n> output is supposed to be. I can buy either \"drop it completely\" or\n> \"stick it in the name field of the cc header\" as reasonable.\n\nWell, we could always convert it to email address comment, converting\nfor example the following trailer:\n\n Cc: John Doe # comment\n\nto the following address:\n\n John Doe (comment)\n\nJust FYI. Though I'm not sure how well this would work...\n\nBest,\n-- \nJakub Narębski\n"},{"id":"306449","messageId":"5B8DA87D05A7694D9FA63FD143655C1B54331DE4@hasmsx108.ger.corp.intel.com","threadId":"44541","inReplyTo":"e11d28d3-c1b5-2c04-643f-0b3bd96cb4d3@gmail.com","subject":"RE: [char-misc-next] mei: request async autosuspend at the end of enumeration","fromName":"Winkler, Tomas","fromEmail":"tomas.winkler@intel.com","sentAt":"2016-11-26T13:02:43Z","receivedAt":"2016-11-26T13:03:01Z","isPatch":false,"sender":{"key":"tomas.winkler@intel.com","avatar":null},"body":"> \n> W dniu 25.11.2016 o 04:14, Jeff King pisze:\n> > On Thu, Nov 24, 2016 at 10:37:14PM +0000, Winkler, Tomas wrote:\n> >\n> >>>>> Cc: # 4.4+\n> >>>>\n> >>>> Looks like git send-email is not able to parse this address\n> >>>> correctly though this is suggested format by\n> Documentation/stable_kernel_rules.txt.\n> >>>> Create wrong address If git parsers is used : 'stable@vger.kernel.org#4.4+'\n> [...]\n> \n> > The patch just brings parity to the Mail::Address behavior and git's\n> > fallback parser, so that you don't end up with the broken\n> > stable@vger.kernel.org#4.4+ address. Instead, that content goes into\n> > the name part of the address.\n> >\n> > It sounds like you want the \"# 4.4+\" to be dropped entirely in the\n> > rfc822 header. It looks like send-email used to do that, but stopped\n> > in\n> > b1c8a11c8 (send-email: allow multiple emails using --cc, --to and\n> > --bcc, 2015-06-30).\n> >\n> > So perhaps there are further fixes required, but it's hard to know.\n> > The input isn't a valid rfc822 header, so it's not entirely clear what\n> > the output is supposed to be. I can buy either \"drop it completely\" or\n> > \"stick it in the name field of the cc header\" as reasonable.\n> \n> Well, we could always convert it to email address comment, converting for\n> example the following trailer:\n> \n> Cc: John Doe # comment\n> \n> to the following address:\n> \n> John Doe (comment)\n> \n> Just FYI. Though I'm not sure how well this would work...\n> \nYep, it actually looks as right place to put this kind of info, \nthough I'm not on the receiving side.\nI'm not sure if and how is this used by stable maintainers. \nThanks\nTomas \n\n\n"}]}