Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1584260
| From | Dan Williams <dan.j.williams@intel.com> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH] switchtec: cleanup cdev init |
| Date | 2017-02-19 22:50 +0100 |
| Message-ID | <tcHwK-4bg-7@gated-at.bofh.it> (permalink) |
| References | <t9kQ2-4Jf-21@gated-at.bofh.it> <t9nNU-6VN-25@gated-at.bofh.it> <tck77-6Ef-7@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On Sat, Feb 18, 2017 at 12:22 PM, Logan Gunthorpe <logang@deltatee.com> wrote: > Hi, > > Please don't apply this patch and instead apply the switchtec driver as > we submitted in v2. As per the discussion in [1], not using the cdev's > kobj parent results in incorrect reference counting and a possible use > of the cdev after its containing structure is freed. Is this race present for all file operations? I've only seen it with mmap() and late faults. So if these other drivers do not support mmap it's not clear they can trigger the failure.
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
Re: [PATCH] switchtec: cleanup cdev init Logan Gunthorpe <logang@deltatee.com> - 2017-02-18 21:50 +0100
Re: [PATCH] switchtec: cleanup cdev init Dan Williams <dan.j.williams@intel.com> - 2017-02-19 22:50 +0100
Re: [PATCH] switchtec: cleanup cdev init Logan Gunthorpe <logang@deltatee.com> - 2017-02-20 05:30 +0100
Re: [PATCH] switchtec: cleanup cdev init Jason Gunthorpe <jgunthorpe@obsidianresearch.com> - 2017-02-21 19:40 +0100
csiph-web