* f3ed935de e1000e: Initial support for KabeLake @ 2019-03-14 19:34 Pat Riehecky 2019-03-14 20:08 ` Greg KH 0 siblings, 1 reply; 8+ messages in thread From: Pat Riehecky @ 2019-03-14 19:34 UTC (permalink / raw) To: stable https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=9cd34b3a1 The additional hardware support would be helpful in the 4.4 long term tree. e1000e: Initial support for KabeLake i219 (4) and i219 (5) are the next LOM generations that will be available on the next Intel platform (KabeLake). This patch provides the initial support for the devices. Signed-off-by: Raanan Avargil <raanan.avargil@intel.com> Tested-by: Aaron Brown <aaron.f.brown@intel.com> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> -- Pat Riehecky Fermi National Accelerator Laboratory www.fnal.gov www.scientificlinux.org ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: f3ed935de e1000e: Initial support for KabeLake 2019-03-14 19:34 f3ed935de e1000e: Initial support for KabeLake Pat Riehecky @ 2019-03-14 20:08 ` Greg KH 2019-03-14 21:17 ` Jeff Kirsher 0 siblings, 1 reply; 8+ messages in thread From: Greg KH @ 2019-03-14 20:08 UTC (permalink / raw) To: Pat Riehecky; +Cc: stable, Raanan Avargil, Aaron Brown, Jeff Kirsher On Thu, Mar 14, 2019 at 02:34:17PM -0500, Pat Riehecky wrote: > https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=9cd34b3a1 > > The additional hardware support would be helpful in the 4.4 long term tree. > > e1000e: Initial support for KabeLake > i219 (4) and i219 (5) are the next LOM generations that will be > available on the next Intel platform (KabeLake). > This patch provides the initial support for the devices. > > Signed-off-by: Raanan Avargil <raanan.avargil@intel.com> > Tested-by: Aaron Brown <aaron.f.brown@intel.com> > Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> Does this patch work properly on 4.4.y for this hardware with just this patch? Also, always cc: the authors of the patch when asking for them to be added to stable, to get their opinion/objection if needed. thanks, greg k-h ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: f3ed935de e1000e: Initial support for KabeLake 2019-03-14 20:08 ` Greg KH @ 2019-03-14 21:17 ` Jeff Kirsher 2019-03-14 21:54 ` Greg KH 0 siblings, 1 reply; 8+ messages in thread From: Jeff Kirsher @ 2019-03-14 21:17 UTC (permalink / raw) To: Greg KH, Pat Riehecky; +Cc: stable, Raanan Avargil, Aaron Brown [-- Attachment #1: Type: text/plain, Size: 1079 bytes --] On Thu, 2019-03-14 at 13:08 -0700, Greg KH wrote: > On Thu, Mar 14, 2019 at 02:34:17PM -0500, Pat Riehecky wrote: > > https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=9cd34b3a1 > > > > The additional hardware support would be helpful in the 4.4 long > > term tree. > > > > e1000e: Initial support for KabeLake > > i219 (4) and i219 (5) are the next LOM generations that will be > > available on the next Intel platform (KabeLake). > > This patch provides the initial support for the devices. > > > > Signed-off-by: Raanan Avargil <raanan.avargil@intel.com> > > Tested-by: Aaron Brown <aaron.f.brown@intel.com> > > Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> > > Does this patch work properly on 4.4.y for this hardware with just > this > patch? > > Also, always cc: the authors of the patch when asking for them to be > added to stable, to get their opinion/objection if needed. I am fine with commit 9cd34b3a1 "e1000e: Initial support for KabeLake" being backported. It is just the addition a device ids. [-- Attachment #2: This is a digitally signed message part --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: f3ed935de e1000e: Initial support for KabeLake 2019-03-14 21:17 ` Jeff Kirsher @ 2019-03-14 21:54 ` Greg KH 2019-03-14 22:53 ` Jeff Kirsher 0 siblings, 1 reply; 8+ messages in thread From: Greg KH @ 2019-03-14 21:54 UTC (permalink / raw) To: Jeff Kirsher; +Cc: Pat Riehecky, stable, Raanan Avargil, Aaron Brown On Thu, Mar 14, 2019 at 02:17:34PM -0700, Jeff Kirsher wrote: > On Thu, 2019-03-14 at 13:08 -0700, Greg KH wrote: > > On Thu, Mar 14, 2019 at 02:34:17PM -0500, Pat Riehecky wrote: > > > https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=9cd34b3a1 > > > > > > The additional hardware support would be helpful in the 4.4 long > > > term tree. > > > > > > e1000e: Initial support for KabeLake > > > i219 (4) and i219 (5) are the next LOM generations that will be > > > available on the next Intel platform (KabeLake). > > > This patch provides the initial support for the devices. > > > > > > Signed-off-by: Raanan Avargil <raanan.avargil@intel.com> > > > Tested-by: Aaron Brown <aaron.f.brown@intel.com> > > > Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> > > > > Does this patch work properly on 4.4.y for this hardware with just > > this > > patch? > > > > Also, always cc: the authors of the patch when asking for them to be > > added to stable, to get their opinion/objection if needed. > > I am fine with commit 9cd34b3a1 "e1000e: Initial support for KabeLake" > being backported. It is just the addition a device ids. Great, thanks! But, the patch does not apply to 4.4.y at all. Pat, can you make up a backport that I can apply and send it to us? thanks, greg k-h ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: f3ed935de e1000e: Initial support for KabeLake 2019-03-14 21:54 ` Greg KH @ 2019-03-14 22:53 ` Jeff Kirsher 2019-03-14 23:25 ` Greg KH 0 siblings, 1 reply; 8+ messages in thread From: Jeff Kirsher @ 2019-03-14 22:53 UTC (permalink / raw) To: Greg KH; +Cc: Pat Riehecky, stable, Raanan Avargil, Aaron Brown [-- Attachment #1: Type: text/plain, Size: 1950 bytes --] On Thu, 2019-03-14 at 14:54 -0700, Greg KH wrote: > On Thu, Mar 14, 2019 at 02:17:34PM -0700, Jeff Kirsher wrote: > > On Thu, 2019-03-14 at 13:08 -0700, Greg KH wrote: > > > On Thu, Mar 14, 2019 at 02:34:17PM -0500, Pat Riehecky wrote: > > > > https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=9cd34b3a1 > > > > > > > > The additional hardware support would be helpful in the 4.4 > > > > long > > > > term tree. > > > > > > > > e1000e: Initial support for KabeLake > > > > i219 (4) and i219 (5) are the next LOM generations that will be > > > > available on the next Intel platform (KabeLake). > > > > This patch provides the initial support for the devices. > > > > > > > > Signed-off-by: Raanan Avargil <raanan.avargil@intel.com> > > > > Tested-by: Aaron Brown <aaron.f.brown@intel.com> > > > > Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> > > > > > > Does this patch work properly on 4.4.y for this hardware with > > > just > > > this > > > patch? > > > > > > Also, always cc: the authors of the patch when asking for them to > > > be > > > added to stable, to get their opinion/objection if needed. > > > > I am fine with commit 9cd34b3a1 "e1000e: Initial support for > > KabeLake" > > being backported. It is just the addition a device ids. > > Great, thanks! > > But, the patch does not apply to 4.4.y at all. Pat, can you make up > a > backport that I can apply and send it to us? I see that you do not have this commit on the 4.4.y tree: f3ed935de059 e1000e: initial support for i219-LM (3) Which is causing the issues that Greg is seeing. This commit is not a "simple" add new device id type of patch. This commit would be a hard requirement for the other commit to be applied. I am fine with both being applied, but this second commit adds code for properly accessing the NVM for e1000_pch_spt devices (which all the i219 devices are). [-- Attachment #2: This is a digitally signed message part --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: f3ed935de e1000e: Initial support for KabeLake 2019-03-14 22:53 ` Jeff Kirsher @ 2019-03-14 23:25 ` Greg KH 2019-03-15 1:36 ` Jeff Kirsher 0 siblings, 1 reply; 8+ messages in thread From: Greg KH @ 2019-03-14 23:25 UTC (permalink / raw) To: Jeff Kirsher; +Cc: Pat Riehecky, stable, Raanan Avargil, Aaron Brown On Thu, Mar 14, 2019 at 03:53:29PM -0700, Jeff Kirsher wrote: > On Thu, 2019-03-14 at 14:54 -0700, Greg KH wrote: > > On Thu, Mar 14, 2019 at 02:17:34PM -0700, Jeff Kirsher wrote: > > > On Thu, 2019-03-14 at 13:08 -0700, Greg KH wrote: > > > > On Thu, Mar 14, 2019 at 02:34:17PM -0500, Pat Riehecky wrote: > > > > > https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=9cd34b3a1 > > > > > > > > > > The additional hardware support would be helpful in the 4.4 > > > > > long > > > > > term tree. > > > > > > > > > > e1000e: Initial support for KabeLake > > > > > i219 (4) and i219 (5) are the next LOM generations that will be > > > > > available on the next Intel platform (KabeLake). > > > > > This patch provides the initial support for the devices. > > > > > > > > > > Signed-off-by: Raanan Avargil <raanan.avargil@intel.com> > > > > > Tested-by: Aaron Brown <aaron.f.brown@intel.com> > > > > > Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> > > > > > > > > Does this patch work properly on 4.4.y for this hardware with > > > > just > > > > this > > > > patch? > > > > > > > > Also, always cc: the authors of the patch when asking for them to > > > > be > > > > added to stable, to get their opinion/objection if needed. > > > > > > I am fine with commit 9cd34b3a1 "e1000e: Initial support for > > > KabeLake" > > > being backported. It is just the addition a device ids. > > > > Great, thanks! > > > > But, the patch does not apply to 4.4.y at all. Pat, can you make up > > a > > backport that I can apply and send it to us? > > I see that you do not have this commit on the 4.4.y tree: > > f3ed935de059 e1000e: initial support for i219-LM (3) > > Which is causing the issues that Greg is seeing. This commit is not a > "simple" add new device id type of patch. This commit would be a hard > requirement for the other commit to be applied. > > I am fine with both being applied, but this second commit adds code for > properly accessing the NVM for e1000_pch_spt devices (which all the > i219 devices are). Yeah, that's not worth it. Just use 4.9, or really 4.14, or even more realistic, 4.19 instead. Especially for any system that might be using this type of network card, you should NOT be using 4.4 or 4.9 there at the moment anyway. thanks, greg k-h ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: f3ed935de e1000e: Initial support for KabeLake 2019-03-14 23:25 ` Greg KH @ 2019-03-15 1:36 ` Jeff Kirsher 2019-03-15 13:13 ` Pat Riehecky 0 siblings, 1 reply; 8+ messages in thread From: Jeff Kirsher @ 2019-03-15 1:36 UTC (permalink / raw) To: Greg KH; +Cc: Pat Riehecky, stable, Raanan Avargil, Aaron Brown [-- Attachment #1: Type: text/plain, Size: 2760 bytes --] On Thu, 2019-03-14 at 16:25 -0700, Greg KH wrote: > On Thu, Mar 14, 2019 at 03:53:29PM -0700, Jeff Kirsher wrote: > > On Thu, 2019-03-14 at 14:54 -0700, Greg KH wrote: > > > On Thu, Mar 14, 2019 at 02:17:34PM -0700, Jeff Kirsher wrote: > > > > On Thu, 2019-03-14 at 13:08 -0700, Greg KH wrote: > > > > > On Thu, Mar 14, 2019 at 02:34:17PM -0500, Pat Riehecky wrote: > > > > > > https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=9cd34b3a1 > > > > > > > > > > > > The additional hardware support would be helpful in the 4.4 > > > > > > long > > > > > > term tree. > > > > > > > > > > > > e1000e: Initial support for KabeLake > > > > > > i219 (4) and i219 (5) are the next LOM generations that > > > > > > will be > > > > > > available on the next Intel platform (KabeLake). > > > > > > This patch provides the initial support for the devices. > > > > > > > > > > > > Signed-off-by: Raanan Avargil <raanan.avargil@intel.com> > > > > > > Tested-by: Aaron Brown <aaron.f.brown@intel.com> > > > > > > Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> > > > > > > > > > > Does this patch work properly on 4.4.y for this hardware with > > > > > just > > > > > this > > > > > patch? > > > > > > > > > > Also, always cc: the authors of the patch when asking for > > > > > them to > > > > > be > > > > > added to stable, to get their opinion/objection if needed. > > > > > > > > I am fine with commit 9cd34b3a1 "e1000e: Initial support for > > > > KabeLake" > > > > being backported. It is just the addition a device ids. > > > > > > Great, thanks! > > > > > > But, the patch does not apply to 4.4.y at all. Pat, can you make > > > up > > > a > > > backport that I can apply and send it to us? > > > > I see that you do not have this commit on the 4.4.y tree: > > > > f3ed935de059 e1000e: initial support for i219-LM (3) > > > > Which is causing the issues that Greg is seeing. This commit is > > not a > > "simple" add new device id type of patch. This commit would be a > > hard > > requirement for the other commit to be applied. > > > > I am fine with both being applied, but this second commit adds code > > for > > properly accessing the NVM for e1000_pch_spt devices (which all the > > i219 devices are). > > Yeah, that's not worth it. Just use 4.9, or really 4.14, or even > more > realistic, 4.19 instead. Especially for any system that might be > using > this type of network card, you should NOT be using 4.4 or 4.9 there > at > the moment anyway. > Or you can download our out-of-tree driver from http://e1000.sourceforge.net project, that driver has all the necessary kernel compat code to run the driver on older kernels like 4.4. [-- Attachment #2: This is a digitally signed message part --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: f3ed935de e1000e: Initial support for KabeLake 2019-03-15 1:36 ` Jeff Kirsher @ 2019-03-15 13:13 ` Pat Riehecky 0 siblings, 0 replies; 8+ messages in thread From: Pat Riehecky @ 2019-03-15 13:13 UTC (permalink / raw) To: jeffrey.t.kirsher, Greg KH; +Cc: stable, Raanan Avargil, Aaron Brown On 3/14/19 8:36 PM, Jeff Kirsher wrote: > On Thu, 2019-03-14 at 16:25 -0700, Greg KH wrote: >> On Thu, Mar 14, 2019 at 03:53:29PM -0700, Jeff Kirsher wrote: >>> On Thu, 2019-03-14 at 14:54 -0700, Greg KH wrote: >>>> On Thu, Mar 14, 2019 at 02:17:34PM -0700, Jeff Kirsher wrote: >>>>> On Thu, 2019-03-14 at 13:08 -0700, Greg KH wrote: >>>>>> On Thu, Mar 14, 2019 at 02:34:17PM -0500, Pat Riehecky wrote: >>>>>>> https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=9cd34b3a1 >>>>>>> >>>>>>> The additional hardware support would be helpful in the 4.4 >>>>>>> long >>>>>>> term tree. >>>>>>> >>>>>>> e1000e: Initial support for KabeLake >>>>>>> i219 (4) and i219 (5) are the next LOM generations that >>>>>>> will be >>>>>>> available on the next Intel platform (KabeLake). >>>>>>> This patch provides the initial support for the devices. >>>>>>> >>>>>>> Signed-off-by: Raanan Avargil <raanan.avargil@intel.com> >>>>>>> Tested-by: Aaron Brown <aaron.f.brown@intel.com> >>>>>>> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> >>>>>> Does this patch work properly on 4.4.y for this hardware with >>>>>> just >>>>>> this >>>>>> patch? >>>>>> >>>>>> Also, always cc: the authors of the patch when asking for >>>>>> them to >>>>>> be >>>>>> added to stable, to get their opinion/objection if needed. >>>>> I am fine with commit 9cd34b3a1 "e1000e: Initial support for >>>>> KabeLake" >>>>> being backported. It is just the addition a device ids. >>>> Great, thanks! >>>> >>>> But, the patch does not apply to 4.4.y at all. Pat, can you make >>>> up >>>> a >>>> backport that I can apply and send it to us? >>> I see that you do not have this commit on the 4.4.y tree: >>> >>> f3ed935de059 e1000e: initial support for i219-LM (3) >>> >>> Which is causing the issues that Greg is seeing. This commit is >>> not a >>> "simple" add new device id type of patch. This commit would be a >>> hard >>> requirement for the other commit to be applied. >>> >>> I am fine with both being applied, but this second commit adds code >>> for >>> properly accessing the NVM for e1000_pch_spt devices (which all the >>> i219 devices are). >> Yeah, that's not worth it. Just use 4.9, or really 4.14, or even >> more >> realistic, 4.19 instead. Especially for any system that might be >> using >> this type of network card, you should NOT be using 4.4 or 4.9 there >> at >> the moment anyway. >> > Or you can download our out-of-tree driver from > http://e1000.sourceforge.net project, that driver has all the necessary > kernel compat code to run the driver on older kernels like 4.4. Thanks for the review! I'll pull in the out of tree driver for my use case. Pat -- Pat Riehecky Fermi National Accelerator Laboratory www.fnal.gov www.scientificlinux.org ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2019-03-15 13:13 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2019-03-14 19:34 f3ed935de e1000e: Initial support for KabeLake Pat Riehecky 2019-03-14 20:08 ` Greg KH 2019-03-14 21:17 ` Jeff Kirsher 2019-03-14 21:54 ` Greg KH 2019-03-14 22:53 ` Jeff Kirsher 2019-03-14 23:25 ` Greg KH 2019-03-15 1:36 ` Jeff Kirsher 2019-03-15 13:13 ` Pat Riehecky
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.