Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

[RFC] Drop legacy cuDNN releases support in MXNet 2.0 #20618

Open
mk-61 opened this issue Sep 28, 2021 · 1 comment
Open

[RFC] Drop legacy cuDNN releases support in MXNet 2.0 #20618

mk-61 opened this issue Sep 28, 2021 · 1 comment
Labels
RFC Post requesting for comments

Comments

@mk-61
Copy link
Contributor

mk-61 commented Sep 28, 2021

In v8 cuDNN introduced new API. It enables more generalized support for heuristics, numerical properties of compute engines and engine knobs. It is substantially different to the legacy one, so new convolution op cache has been implemented in MXNet container and the old one has been removed. In order to upstream new API support we either need to maintain 2 convolution op caches, or drop the legacy one.

Maintaining 2 op caches would keep legacy cuDNN releases supported, but would make the code messier.

Secondly, it might be less painful to make this move now rather than later, with more customers moved to 2.0.

Currently this change affects only convolution and deconvolution ops, but more operations are expected to be ported to the new API, including fused operations.

@mk-61 mk-61 added the RFC Post requesting for comments label Sep 28, 2021
@szha
Copy link
Member

szha commented Sep 28, 2021

+1

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
RFC Post requesting for comments
Projects
None yet
Development

No branches or pull requests

2 participants