From: Naga Sureshkumar Relli <naga.sureshkumar.relli@xilinx.com>
To: Richard Weinberger <richard@nod.at>,
        Christoph Hellwig <hch@infradead.org>
CC: "dwmw2@infradead.org" <dwmw2@infradead.org>,
        "computersforpeace@gmail.com" <computersforpeace@gmail.com>,
        "dedekind1@gmail.com" <dedekind1@gmail.com>,
        "adrian.hunter@intel.com" <adrian.hunter@intel.com>,
        "michal.simek@xilinx.com" <michal.simek@xilinx.com>,
        "Punnaiah Choudary Kalluri" <punnaia@xilinx.com>,
        "linux-mtd@lists.infradead.org" <linux-mtd@lists.infradead.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Boris Brezillon <boris.brezillon@free-electrons.com>
Subject: RE: UBIFS LEB properties tree with odd addresses
Thread-Topic: UBIFS LEB properties tree with odd addresses
Thread-Index: AdIuhYzSHMtf8MMlQH+bJ6F6DdMcjwAPB2aAAC/Mm9AAIz6WgAFeeMUg
Date: Thu, 3 Nov 2016 05:11:20 +0000
Message-ID: <DA6901612C71B84D91459DE817C418AE26E02536@XAP-PVEXMBX01.xlnx.xilinx.com>
References: <DA6901612C71B84D91459DE817C418AE26E0016D@XAP-PVEXMBX01.xlnx.xilinx.com>
 <e7f63f72-0dc1-54f4-b94c-2efef3e7af6b@nod.at>
 <DA6901612C71B84D91459DE817C418AE26E0069D@XAP-PVEXMBX01.xlnx.xilinx.com>
 <c6e394c9-bbec-1121-7a93-268abcf934bf@nod.at>
In-Reply-To: <c6e394c9-bbec-1121-7a93-268abcf934bf@nod.at>
Accept-Language: en-US
Content-Language: en-US
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
SpamDiagnosticOutput: 1:99
SpamDiagnosticMetadata: NSPM
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Nov 2016 05:11:23.9000
 (UTC)
X-MS-Exchange-CrossTenant-Id: 657af505-d5df-48d0-8300-c31994686c5c
X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=657af505-d5df-48d0-8300-c31994686c5c;Ip=[149.199.60.83];Helo=[xsj-pvapsmtpgw01]
X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY2PR02MB153
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Transfer-Encoding: 8bit
X-MIME-Autoconverted: from quoted-printable to 8bit by mail.home.local id uA35hGfK013091
Content-Length: 1276
Lines: 27

Hi Richard,

>> Hmm, the calculations in lpt.c use basic properties of the MTD (number of blocks, etc...).
>> Can you please double check which property causes the failure and also verify whether the MTD driver provides correct properties. i.e. page sizes, block size, number of blocks, ...

> Hm, these values depend only on basic MTD prperties AFAICT.
> Mtd->numeraseregions = 0
> Nor->pagesize = 512

As per the above query only, I have given the below inputs.
>Maybe you miss some bit and set incorrect MTD properties.
I double checked the MTD properties (as mentioned above) which were causing the failure, but didn't find any clue.
All mtd properties looks good for dual parallel case(there is no odd value).

Is there any impact by putting below logic in do_calc_lpt_geom() ?
If(c->pnode_sz % 2)
    c->pnode_sz += 1;

please suggest me to do any further debugging?

Thanks,
Naga Sureshkumar Relli


This email and any attachments are intended for the sole use of the named recipient(s) and contain(s) confidential information that may be proprietary, privileged or copyrighted under applicable law. If you are not the intended recipient, do not read, copy, or forward this email message or any attachments. Delete this email message and any attachments immediately.