[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <66da24827b2a9_22a2294b3@dwillia2-xfh.jf.intel.com.notmuch>
Date: Thu, 5 Sep 2024 14:37:06 -0700
From: Dan Williams <dan.j.williams@...el.com>
To: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>, Dan Williams
<dan.j.williams@...el.com>
CC: "Huang, Ying" <ying.huang@...el.com>, Andrew Morton
<akpm@...ux-foundation.org>, <linux-mm@...ck.org>,
<linux-kernel@...r.kernel.org>, <linux-cxl@...r.kernel.org>, "David
Hildenbrand" <david@...hat.com>, Davidlohr Bueso <dave@...olabs.net>,
"Jonathan Cameron" <jonathan.cameron@...wei.com>, Dave Jiang
<dave.jiang@...el.com>, Alison Schofield <alison.schofield@...el.com>, Vishal
Verma <vishal.l.verma@...el.com>, Ira Weiny <ira.weiny@...el.com>, Alistair
Popple <apopple@...dia.com>, Bjorn Helgaas <bhelgaas@...gle.com>, Baoquan He
<bhe@...hat.com>
Subject: Re: [PATCH -v2] Resource: fix region_intersects() for CXL memory
Andy Shevchenko wrote:
> On Wed, Sep 04, 2024 at 04:58:20PM -0700, Dan Williams wrote:
> > Huang, Ying wrote:
> > > Andy Shevchenko <andriy.shevchenko@...ux.intel.com> writes:
>
> [..]
>
> > > > You may move Cc list after '---', so it won't unnecessarily pollute the commit
> > > > message.
> > >
> > > Emm... It appears that it's a common practice to include "Cc" in the
> > > commit log.
> >
> > Yes, just ignore this feedback, it goes against common practice. Cc list
> > as is looks sane to me.
>
> It seems nobody can give technical arguments why it's better than just keeping
> them outside of the commit message. Mantra "common practice" nowadays is
> questionable.
Yes, question asked and answered. Not to your satisfaction, but the
people that have engaged to date have been cold to the idea.
Historically, reaching into other kernel developers workflows to
instantiate a new preference is a high risk low reward endeavor, please
moderate your advocacy accordingly.
Powered by blists - more mailing lists