Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
To:     "Matthew Wilcox (Oracle)" <willy@infradead.org>
Cc:     Andrew Morton <akpm@linux-foundation.org>, linux-mm@kvack.org,
        linux-block@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux-bcache@vger.kernel.org, linux-nvdimm@lists.01.org,
        linux-scsi@vger.kernel.org
Subject: Re: [PATCH v2] include: Remove pagemap.h from blkdev.h
From:   "Martin K. Petersen" <martin.petersen@oracle.com>
Organization: Oracle Corporation
Message-ID: <yq1ft139i5g.fsf@ca-mkp.ca.oracle.com>
References: <20210309195747.283796-1-willy@infradead.org>
Date:   Wed, 10 Mar 2021 09:51:38 -0500
In-Reply-To: <20210309195747.283796-1-willy@infradead.org> (Matthew Wilcox's
        message of "Tue, 9 Mar 2021 19:57:47 +0000")
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
Received: from ca-mkp.ca.oracle.com (138.3.200.58) by CH2PR03CA0019.namprd03.prod.outlook.com (2603:10b6:610:59::29) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3912.17 via Frontend Transport; Wed, 10 Mar 2021 14:51:41 +0000
X-MS-PublicTrafficType: Email
X-MS-Office365-Filtering-Correlation-Id: 6f85afa0-9670-47ef-8669-08d8e3d404be
X-MS-TrafficTypeDiagnostic: PH0PR10MB4789:
X-MS-Exchange-Transport-Forked: True
X-Microsoft-Antispam-PRVS: <PH0PR10MB4789567DF238A9DA434A25638E919@PH0PR10MB4789.namprd10.prod.outlook.com>
X-MS-Oob-TLC-OOBClassifiers: OLM:4941;
X-MS-Exchange-SenderADCheck: 1
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info: 5ArQvDt2eSOFLXRkD2j8osAB6Us8KcLd4G7h4KOHMz/r7ttZF+QetVOMOXCFLMGXvMAkoZ4mLGIUJSAfs+pQX9qKWvoNn4p8oiovnozuCC2qDPWtYn1/Qsue7cpGwh7N3WllfhCJa5mlN7KRKnW4Qpp+YQtE/GmH2fYFyUiMdkhHOLKSC1nwHYczKUilwX8aFzxPdaUNH80HbP6NkLbX9Z0iT9lMwxmlKlQM4ubp1TxeZK9aVRIPt5es4XkKHMRK0eKPkLzNLnOz/AnpFwLa8atXcKrnDcLjDTyLngwMlBbLK1USIBrIM0Mn6Ygxf+P5qdTG+QN9LtkWqSkRbTa5MOwgc5CbT/k5Tgk5cpAA1oIUAXPG3QKacRgLt8Lf5a1njcYRpBuGNLcSdH9BFjqo7h9dDPF6DGyEmiUL9Xpl0yFwaYEn1zO+fUMOvnZcldPvSiyJiiranV33DuSVVPjeA+oNgI/GQEMr8NoIAFCquaZufrxkWOH0k8AAb8kWH2bKUbz0V/Tmz62Upc4HD6D92A==
X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:PH0PR10MB4759.namprd10.prod.outlook.com;PTR:;CAT:NONE;SFS:(136003)(376002)(346002)(39860400002)(366004)(396003)(2906002)(478600001)(55016002)(8676002)(66476007)(4744005)(4326008)(26005)(66556008)(8936002)(52116002)(66946007)(956004)(86362001)(316002)(16526019)(6916009)(186003)(36916002)(6666004)(5660300002)(7696005);DIR:OUT;SFP:1101;
X-MS-Exchange-AntiSpam-MessageData: =?us-ascii?Q?3kuccfVWdICyxNj3MduGF4t7Wp00yuqC+PRtorQZ4LNfUZxqfszsiBieOKBA?=
 =?us-ascii?Q?sTXBnT+PannOG+cUOVic7f2F/7EkbZdQYhzXyOcm81F95YZYc1SEYbL9PN8b?=
 =?us-ascii?Q?KNOXbrUur6QXTO9JdmfKsR53EpueVRExlBcRsPZ6/CBUml7x159AI3pe+ewI?=
 =?us-ascii?Q?K8Xq4vjMANTGBNcRKyu0BuWAmTTsF4oyHQWT5aIdf+hr+unLidf/JH5grjQJ?=
 =?us-ascii?Q?cKOHPrjxmQBl3xXMG58ken/F9rheQcinC2ZuWfSVlluJICPOhCV3RhjRnf8w?=
 =?us-ascii?Q?/Gz7xrMRQ9Z+exqj1NscFLEhfivsezn9dr+j6bxDRDEh+YsqllR+wLTD+xxR?=
 =?us-ascii?Q?fz97ZFtxsvcvxmFP6C4RfS6ppqr9uhBI8zy+06nvSw1jm1z7aHPlXB3LDNey?=
 =?us-ascii?Q?a3D1qd4Hkv+f28mSjCZ7uSbN7RHcneBkHzq0tw4EsgZ9k9ISGztdkyuYj4Ge?=
 =?us-ascii?Q?ImEbjWyskpuwr34Qe/8SGiJwOAMQ2hRN25IgGL8ObLzOyoO2rqkJS8eLwL8j?=
 =?us-ascii?Q?Jhrmq5zfnlBDP9hpAEIZOHVBQbL87uC/rGeIQ6IzS4meLekzXMxUBA7JJTnD?=
 =?us-ascii?Q?PJwqAjb4VdSI9mr0A9v0qNwxZoufm8m2kEjaKI13/ynsNfGN9icghkoZCGFS?=
 =?us-ascii?Q?IimUouwCjId65G34HvCtr5iugHGXDgbr9NOHYlybIhi6/710SFgvUBg/2CgP?=
 =?us-ascii?Q?nAhIFkkQUigCVynQJT88NzUpkpDPeiL6dfOpLY6/DuhYeGfp5+lB8K6p0lyB?=
 =?us-ascii?Q?KXZ7E5W/TxT/OziJOmzqCbUIgdw6JwcGACsI6yZ9mtT/359PHaoGU7DpJhYM?=
 =?us-ascii?Q?uaPtrV9s1Of3bYwMXb9WFAjvPOFWTXJG+F1GVxtC98TZnd11P14OwsJiLdtN?=
 =?us-ascii?Q?KIpqA2B8BGncCGwOfX5uwCF5D6nieic0AGXIngKQ3TFjLIud1h0hNm8E5rTV?=
 =?us-ascii?Q?8/j6VPkzzQlnZ3VINiYSdN4w5Dm3/vbc/mvxiTwPk4XMikXQljiVK/76g9gC?=
 =?us-ascii?Q?ekGbWplbNELpqqeZF+d/8r3NPtgN8wL3obz1GuurxYiljycGR81znuniuXBC?=
 =?us-ascii?Q?hS3oGM29SkJfIPb4EewhSLbKb8cGBDg9KscGNNhvrWb0yOQ0AgCBkdL0sp0r?=
 =?us-ascii?Q?laCG4ClIWy3J7AjlbJdrFMqrg9iaHlRacth58krullnc4DVF2zznYmYU/RxC?=
 =?us-ascii?Q?1UaezlVY69aK/VaUEySCAKU+wneHeQ7NSijv8tcGRBebmMwON8DnJOAiftbl?=
 =?us-ascii?Q?btaZRmgCcXXLx3CKxq/LsoBF8jZA4E7k2VH+i+2DZm4TxPuGaTr4+MVpLGKW?=
 =?us-ascii?Q?RFuKXNYVDNIVs2Ixgk8HZh8H?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 6f85afa0-9670-47ef-8669-08d8e3d404be
X-MS-Exchange-CrossTenant-AuthSource: PH0PR10MB4759.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 Mar 2021 14:51:42.6296
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: wjlm2pwEED3NYrcXvoeMh8PUvuChmEfN6Rgho/eOyj7eTsoVhU/VfcMRs4MMysXXrtbVlIZ5bhZ6rarqjNUO6ppcEsclAi8Z5L/dUjL5lkY=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR10MB4789
X-Proofpoint-Virus-Version: vendor=nai engine=6200 definitions=9919 signatures=668683
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 spamscore=0 suspectscore=0 adultscore=0
 phishscore=0 malwarescore=0 mlxscore=0 bulkscore=0 mlxlogscore=999
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2009150000
 definitions=main-2103100075
X-Proofpoint-Virus-Version: vendor=nai engine=6200 definitions=9919 signatures=668683
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 spamscore=0 suspectscore=0
 clxscore=1011 phishscore=0 adultscore=0 mlxlogscore=999 priorityscore=1501
 lowpriorityscore=0 bulkscore=0 mlxscore=0 impostorscore=0 malwarescore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2009150000
 definitions=main-2103100075
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


Matthew,

> My UEK-derived config has 1030 files depending on pagemap.h before
> this change.  Afterwards, just 326 files need to be rebuilt when I
> touch pagemap.h.  I think blkdev.h is probably included too widely,
> but untangling that dependency is harder and this solves my problem.
> x86 allmodconfig builds, but there may be implicit include problems on
> other architectures.

SCSI portion looks fine.

Acked-by: Martin K. Petersen <martin.petersen@oracle.com>

-- 
Martin K. Petersen	Oracle Linux Engineering