On Thu, Sep 25, 2014 at 08:47:44PM +0100, Andrew Morton wrote: > On Thu, 25 Sep 2014 12:30:00 -0700 Peter Feiner <pfeiner@xxxxxxxxxx> wrote: > > > On Thu, Sep 25, 2014 at 05:47:30PM +1000, Stephen Rothwell wrote: > > > Hi Andrew, > > > > > > Today's linux-next merge of the akpm-current tree got a conflict in > > > include/asm-generic/pgtable.h between commit b766eafe6828 ("PCI: Add > > > pci_remap_iospace() to map bus I/O resources") from the pci tree and > > > commit 4cc2f91f2636 ("mm: softdirty: enable write notifications on VMAs > > > after VM_SOFTDIRTY cleared") from the akpm-current tree. > > > > > > I fixed it up (see below) and can carry the fix as necessary (no action > > > is required). > > > > > > -- > > > Cheers, > > > Stephen Rothwell sfr@xxxxxxxxxxxxxxxx > > > > > > diff --cc include/asm-generic/pgtable.h > > > index 977e545a64c3,f6acf1195182..000000000000 > > > --- a/include/asm-generic/pgtable.h > > > +++ b/include/asm-generic/pgtable.h > > > @@@ -249,10 -249,18 +249,22 @@@ static inline int pmd_same(pmd_t pmd_a > > > #define pgprot_writecombine pgprot_noncached > > > #endif > > > > > > +#ifndef pgprot_device > > > +#define pgprot_device pgprot_noncached > > > +#endif > > > + > > > + #ifndef pgprot_modify > > > + #define pgprot_modify pgprot_modify > > > + static inline pgprot_t pgprot_modify(pgprot_t oldprot, pgprot_t newprot) > > > + { > > > + if (pgprot_val(oldprot) == pgprot_val(pgprot_noncached(oldprot))) > > > + newprot = pgprot_noncached(newprot); > > > + if (pgprot_val(oldprot) == pgprot_val(pgprot_writecombine(oldprot))) > > > + newprot = pgprot_writecombine(newprot); > > > > For completeness of the arch-independent pgprot-modify, > > > > if (pgprot_val(oldprot) == pgprot_val(pgprot_device(oldprot))) > > newprot = pgprot_device(newprot); > > > > should be added here. > > Thanks for noticing this Peter! > > Tricky. I've moved > mm-softdirty-enable-write-notifications-on-vmas-after-vm_softdirty-cleared.paqtch > to be staged after linux-next, so I'll merge it after PCI has merged. > And added this: > > --- a/include/asm-generic/pgtable.h~mm-softdirty-enable-write-notifications-on-vmas-after-vm_softdirty-cleared-fix > +++ a/include/asm-generic/pgtable.h > @@ -261,6 +261,8 @@ static inline pgprot_t pgprot_modify(pgp > newprot = pgprot_noncached(newprot); > if (pgprot_val(oldprot) == pgprot_val(pgprot_writecombine(oldprot))) > newprot = pgprot_writecombine(newprot); > + if (pgprot_val(oldprot) == pgprot_val(pgprot_device(oldprot))) > + newprot = pgprot_device(newprot); > return newprot; > } > #endif > _ > Thanks for the patch Andrew! Best regards, Liviu -- ==================== | I would like to | | fix the world, | | but they're not | | giving me the | \ source code! / --------------- ¯\_(ツ)_/¯ -- To unsubscribe from this list: send the line "unsubscribe linux-next" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html