Received-SPF: pass (google.com: domain of linux-kernel+bounces-36280-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
Precedence: bulk
MIME-Version: 1.0
Date: Tue, 23 Jan 2024 17:38:56 -0800
In-Reply-To: <tencent_E352FDC65FC9B06EFB6ED30C9CA7EF46DC06@qq.com>
Message-ID: <0000000000004d0ad7060fa720f3@google.com>
Subject: Re: [syzbot] [jfs?] WARNING in dbAdjTree
From: syzbot <syzbot+ab18fa9c959320611727@syzkaller.appspotmail.com>
To: eadavis@qq.com
Cc: eadavis@qq.com, linux-kernel@vger.kernel.org, 
	syzkaller-bugs@googlegroups.com
Content-Type: text/plain; charset="UTF-8"

> please test WARNING in dbAdjTree
>
> #syz test upstream

want either no args or 2 args (repo, branch), got 1

>
> diff --git a/fs/jfs/jfs_dmap.c b/fs/jfs/jfs_dmap.c
> index 8eec84c651bf..bed14d5169f8 100644
> --- a/fs/jfs/jfs_dmap.c
> +++ b/fs/jfs/jfs_dmap.c
> @@ -2294,6 +2294,7 @@ static int dbFreeBits(struct bmap * bmp, struct dmap * dp, s64 blkno,
>  	 */
>  	dbitno = blkno & (BPERDMAP - 1);
>  	word = dbitno >> L2DBWORD;
> +	printk("dbitno: %d, blkno: %ld, word: %d, %s\n", dbitno, blkno, word, __func__);
>  
>  	/* block range better be within the dmap.
>  	 */
> @@ -2875,8 +2876,9 @@ static void dbAdjTree(dmtree_t *tp, int leafno, int newval, bool is_ctl)
>  	/* pick up the index of the leaf for this leafno.
>  	 */
>  	lp = leafno + le32_to_cpu(tp->dmt_leafidx);
> +	printk("lp: %d, leafno: %d, tp: %p, %s\n", lp, leafno, tp,  __func__);
>  
> -	if (WARN_ON_ONCE(lp >= size || lp < 0))
> +	if (lp >= size || lp < 0)
>  		return;
>  
>  	/* is the current value the same as the old value ?  if so,
>