Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1354276
| From | Matthew Wilcox <willy@linux.intel.com> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: + x86-add-support-for-pud-sized-transparent-hugepages-checkpatch-fixes.patch added to -mm tree |
| Date | 2016-03-09 18:00 +0100 |
| Message-ID | <raPCQ-2eF-37@gated-at.bofh.it> (permalink) |
| References | <qY0mm-7ok-21@gated-at.bofh.it> <qY0mm-7ok-19@gated-at.bofh.it> <r94FX-3Op-7@gated-at.bofh.it> <raL6a-7IU-7@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On Wed, Mar 09, 2016 at 01:08:08PM +0100, Ingo Molnar wrote: > * Matthew Wilcox <willy@linux.intel.com> wrote: > > I have no idea what it means. This is copy-and-change of the pmd version, > > which was originally commit db3eb96f4e6281b84dd33c8980dacc27f2efe177 by > > Andrea. > > It means that we don't want to copy-and-change a crappy comment that slipped > through 5 years ago, we want to copy-and-improve. I even suggested the comment > improvement (which needs to be checked though). The "it" in my sentence referred to the comment. I have no idea what the comment is supposed to mean. I am the worst person to figure out what the comment is supposed to mean as I have the least experience with the code here. The PUD and PMD code should be as similar as possible, down to the comments and the spacing. If you want the original fixed, that's fine, and I'm willing to include it as part of this patch set. But it's not my responsibility to fix up the comments that you don't like. > > It seems unfair to ask me to do better than what is there right now. > > It's absolutely fair for maintainers to require the improvement of existing code > you want to modify, especially when you are complicating existing code ... I'm not complicating it. I'm duplicating it.
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
Re: + x86-add-support-for-pud-sized-transparent-hugepages-checkpatch-fixes.patch added to -mm tree Matthew Wilcox <willy@linux.intel.com> - 2016-03-04 21:40 +0100
Re: + x86-add-support-for-pud-sized-transparent-hugepages-checkpatch-fixes.patch added to -mm tree Ingo Molnar <mingo@kernel.org> - 2016-03-09 13:10 +0100
Re: + x86-add-support-for-pud-sized-transparent-hugepages-checkpatch-fixes.patch added to -mm tree Matthew Wilcox <willy@linux.intel.com> - 2016-03-09 18:00 +0100
Re: + x86-add-support-for-pud-sized-transparent-hugepages-checkpatch-fixes.patch added to -mm tree Ingo Molnar <mingo@kernel.org> - 2016-03-10 10:40 +0100
Re: + x86-add-support-for-pud-sized-transparent-hugepages-checkpatch-fixes.patch added to -mm tree Matthew Wilcox <willy@linux.intel.com> - 2016-03-10 15:40 +0100
Re: + x86-add-support-for-pud-sized-transparent-hugepages-checkpatch-fixes.patch added to -mm tree Andrea Arcangeli <aarcange@redhat.com> - 2016-03-09 18:50 +0100
Re: + x86-add-support-for-pud-sized-transparent-hugepages-checkpatch-fixes.patch added to -mm tree Matthew Wilcox <willy@linux.intel.com> - 2016-03-09 19:50 +0100
Re: + x86-add-support-for-pud-sized-transparent-hugepages-checkpatch-fixes.patch added to -mm tree Matthew Wilcox <willy@linux.intel.com> - 2016-03-09 21:10 +0100
Re: + x86-add-support-for-pud-sized-transparent-hugepages-checkpatch-fixes.patch added to -mm tree Andrea Arcangeli <aarcange@redhat.com> - 2016-03-10 00:10 +0100
csiph-web