linux-kernel - Re: [GIT PULL] (xen) for-jens-3.20 for v3.20 Xen blk driver updates

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <54DA2367.8020200@fb.com>
Date:	Tue, 10 Feb 2015 08:27:35 -0700
From:	Jens Axboe <axboe@...com>
To:	Roger Pau Monné <roger.pau@...rix.com>,
	<linux-kernel@...r.kernel.org>,
	xen-devel <xen-devel@...ts.xenproject.org>,
	Konrad Rzeszutek Wilk <konrad.wilk@...cle.com>,
	David Vrabel <david.vrabel@...rix.com>
Subject: Re: [GIT PULL] (xen) for-jens-3.20 for v3.20 Xen blk driver updates

On 02/10/2015 05:58 AM, Roger Pau Monné wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> Hello Jens,
>
> There are 3 fixes for Xen blk drivers. One adds myself as a maintainer
> (together with Konrad). There's also one to improve compatibility with
> older xen-blkfront versions that did not specify a protocol version.
> The remaining one is a fix for blkfront to correctly account the
> number of requests on the queue when resuming from suspension.
>
> Thanks, Roger.
>
> The following changes since commit
> e36f014edff70fc02b3d3d79cead1d58f289332e:
>
>    Linux 3.19-rc7 (2015-02-01 20:07:21 -0800)
>
> are available in the git repository at:
>
> git://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.git for-jens-3.20

This is based on 3.19-rc7, which means that when I pulled it into 
for-3.20/drivers, I get a ton of unrelated commits as well. That's no 
good. The pull request should be based on for-3.20/drivers or earlier. 
So please rebase the three patches on top of for-3.20/drivers. Konrad 
does the right thing here. Not sure if you are going to be sending pull 
requests going forward (or both of you), but it might not hurt to sync 
up with Konrad and ensure you guys are on the same page.

Secondly, this is a bit late. But since it's just two fixes (not 
features) and a maintainer update, it's not a huge deal. It's a bit 
annoying though since every pull request like this will push my pull 
request by a day or two, so generally I prefer to have pull requests at 
least the week before the merge window opens.

-- 
Jens Axboe

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/